将此代码从cstringarray转换为std :: vector＆lt; cstring＆gt;？

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＆lt; cString实现相同的corme> perfermansshuffle和locateText方法？

我应该只留下cstringarray吗？

Answer 1

我知道MFC数组的1（一个！）优于std :: vector - 它们支持MFC风格的序列化。如果您使用它 - 您可能会被卡住。

但是，如果您不这样做 - 我将使用std :: vector＆lt; cstring＆gt;。您的locateText（那是冗长的）将变得过时 - 只需使用查找

另外，您的shufflearray非常效率（一次删除/插入一个项目）。使用vector将允许您执行从向量提取子向量的最佳方法？