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

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

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

第二条路径是错误的!

最佳答案

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

可以减轻头痛，并使用CFileDialog::GetIFileOpenDialog()可以使用的Vista+ API。

这是一个工作代码示例:

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 );
}