c++ - 带有 OFN_ALLOWMULTISELECT 的 CFileDialog 快捷方式的错误结果

Question

有人可以让我知道我在这里做错了什么吗？

MFC 项目，我正在使用 CFileDialog让用户选择多个文件，例如:

CFileDialog fd(TRUE, NULL, NULL,
    OFN_HIDEREADONLY | OFN_OVERWRITEPROMPT | OFN_EXPLORER | OFN_ALLOWMULTISELECT, 
    NULL, this);

if(fd.DoModal() == IDOK)
{
    //Multi-selection
    CString strPaths;
    POSITION fileNamesPosition = fd.GetStartPosition();

    while(fileNamesPosition != NULL)
    {
        if(!strPaths.IsEmpty())
            strPaths += L"\n";

        strPaths += fd.GetNextPathName(fileNamesPosition);
    }  

    AfxMessageBox(strPaths);
}

所以如果说，有两个快捷方式文件:

shortcut_1.lnk 文件引用:"D:\Folder\Project_B\Release\Name of Project B.exe"

和 shortcut_2.lnk 引用 "D:\Folder\Project_A\Release\Name of Project A.exe"

如果我从上面代码生成的“文件打开”对话框中选择它们，我得到的 strPaths 将变为以下，这是不正确的:

D:\Folder\Project_A\Release\Name of Project A.exe
D:\Folder\Project_A\Release\Name of Project B.exe

第二条路径错误!

Answer 1

使用 GetStartPosition() 和 GetNextPathName() 函数是一团糟。一方面，他们使用旧式 API which depends on a correct return buffer size defined via OPENFILENAME struct . MFC 不会处理这个问题!正如您的问题所示，即使缓冲区大小足够大，它也存在链接问题。

使用 Vista+ API 避免让您头疼，可通过 CFileDialog::GetIFileOpenDialog() 获得。

这是一个工作代码示例:

CFileDialog fd( TRUE, NULL, NULL,
    OFN_HIDEREADONLY | OFN_OVERWRITEPROMPT | OFN_EXPLORER | OFN_ALLOWMULTISELECT,
    NULL, nullptr );

if (fd.DoModal() == IDOK)
{
    //Multi-selection
    CString strPaths;

    CComPtr<IFileOpenDialog> piod = fd.GetIFileOpenDialog();
    ASSERT( piod );

    CComPtr<IShellItemArray> pResults;
    if( SUCCEEDED( piod->GetResults( &pResults ) ) )
    {
        DWORD count = 0; pResults->GetCount( &count );
        for( DWORD i = 0; i < count; ++i )
        {
            CComPtr<IShellItem> pItem;
            if( SUCCEEDED( pResults->GetItemAt( i, &pItem ) ) )
            {
                CComHeapPtr<wchar_t> pPath;
                if( SUCCEEDED( pItem->GetDisplayName( SIGDN_FILESYSPATH, &pPath ) ) )
                {
                    if( !strPaths.IsEmpty() )
                        strPaths += L"\n";
                    strPaths += pPath;
                }
            }
        }
    }

    AfxMessageBox( strPaths );
}