Question

我正在尝试实现合并排序，但我的代码似乎缺少某些东西。代码似乎是为最终结果错误地附加每个子排序数组，在某些情况下，添加元素的副本。

例如，如果我通过函数运行A=[1,3,5,2,4,6]，我会A=[1,2,3,2,4,5,6];在1,2,3和4,5,6之间似乎有一个额外的2。另一个例子是A=[4,2,9,8,11,3]。当'排序'时，我得到A=[3,4,2,8,3,9,11]。为什么会这样？

vector<int> mergeSort(vector<int> A)
{
    if(A.size()<=1)
    {
        //print(A);
        return A;
    }
    vector<int> left;
    vector<int> right;
    for(unsigned int i=0; i < A.size()/2; i++)
        left.push_back(A[i]);
    for(unsigned int i=A.size()/2; i < A.size(); i++)
        right.push_back(A[i]);
    vector<int> sortedLeft = mergeSort(left);
    vector<int> sortedRight = mergeSort(right);
    vector<int>::iterator iterL=left.begin();
    vector<int>::iterator iterR;
    vector<int> output;
    while(iterL != left.end())
    {
        iterR=right.begin();
        while(iterR != right.end())
        {
            if(*iterR < *iterL)
            {
                //cout << "Pushing back " << *iterR << endl;
                output.push_back(*iterR);
            }
            iterR++;
        }
        //cout << "Pushing back " << *iterL << endl;
        output.push_back(*iterL);
        iterL++;
    }
    iterR=right.begin();
    while(iterR != right.end())//appends left-over elements in right to output
    {
        if(find(output.begin(), output.end(), *iterR) == output.end())
        {
            //cout << "Pushing back " << *iterR << endl;        
            output.push_back(*iterR);
        }
        iterR++;
    }
    return output;

下面是更新的while循环：

while(iterL != left.end())
    {
        iterR=right.begin();
        while(iterR != right.end() && *iterR != -1)
        {
            if(*iterR < *iterL)
            {
                //cout << "Pushing back " << *iterR << endl;
                output.push_back(*iterR);
                *iterR = -1;
            }
            iterR++;
        }
        //cout << "Pushing back " << *iterL << endl;
        output.push_back(*iterL);
        iterL++;
    }
    iterR=right.begin();
    while(iterR != right.end())//appends left-over elements in right to output
    {
        if(*iterR != -1)
        {
            output.push_back(*iterR);
        }
        iterR++;
    }

Answer 1

你的主要错误在于这个循环的设计：

while(iterL != left.end())
{
    iterR=right.begin();
    while(iterR != right.end())
    {
        if(*iterR < *iterL)
        {
            //cout << "Pushing back " << *iterR << endl;
            output.push_back(*iterR);
        }
        iterR++;
    }
    //cout << "Pushing back " << *iterL << endl;
    output.push_back(*iterL);
    iterL++;
}

如果左边的范围是3, 4而右边的范围是1, 2，请考虑会发生什么。

在第一次迭代中，它会将两个正确的元素添加到您的输出中。
在第二次迭代中，它将重置iterR ，然后再次重复。

您的输出为1, 2, 3, 1, 2, 4。

如果你设法正确地保持处理右向量的哪个元素的状态，那么关于剩余元素的问题将自行消失。

进一步提示：

::std::merge可能会更好。

您始终按值传递向量。

您的代码仅适用于vector<int>，这是C ++中不必要的限制。模板适合晴天。

合并排序不正确排序（C ++）

1 个答案:

进一步提示：