将此代码从 CStringArray 转换为 std::vector<CString> 会更简单吗？

Question

鉴于此代码：

void CSelectNamesDlg::ShuffleArray(CString strName, CStringArray *pAryStrNames)
{
    if (pAryStrNames == nullptr)
        return;

    const auto iSize = pAryStrNames->GetSize();
    if (iSize > 1)
    {
        // First, we must locate strName in the array
        auto i = CSelectNamesDlg::LocateText(strName, pAryStrNames);
        if (i != -1)
        {
            const auto iName = i;

            // We must now shuffle the names from the bottom to the top
            const auto iCount = gsl::narrow<int>(iSize) - iName;
            for (i = 0; i < iCount; i++)
            {
                CString strTemp = pAryStrNames->GetAt(iSize-1);
                pAryStrNames->RemoveAt(iSize-1);
                pAryStrNames->InsertAt(0, strTemp);
            }
        }
    }
}

int CSelectNamesDlg::LocateText(CString strText, const CStringArray *pAryStrText)
{
    bool bFound = false;
    int i{};

    if (pAryStrText != nullptr)
    {
        const auto iSize = pAryStrText->GetSize();
        for (i = 0; i < iSize; i++)
        {
            if (pAryStrText->GetAt(i) == strText)
            {
                // Found him!
                bFound = true;
                break;
            }
        }
    }

    if (!bFound)
        i = -1;

    return (int)i;
}

如果我将我的 CStringArray 转换为 std::vector<CString，实现相同的 PerformShuffle 和 LocateText 方法是否会更简单？

我应该和 CStringArray 待在一起吗？

Answer 1

我知道 MFC 数组超过 std::vector 的 1（一个！）好处 - 它们支持 MFC-style 序列化。如果你使用它 - 你可能会被卡住。

但是，如果您不这样做，我会使用 std::vector<CString>。您的 LocateText（过于冗长）将过时 - 只需使用 find

另外，您的 ShuffleArray 效率很低（remove/insert 一次一个项目）。使用 vector 将允许您执行类似 Best way to extract a subvector from a vector?

的操作

将此代码从 CStringArray 转换为 std::vector<CString> 会更简单吗？

Will it be simpler to convert this code from CStringArray to std::vector<CString>?

visual-c++

mfc

stdvector