在C ++中合并排序 - 程序的奇怪行为

时间:2013-01-31 08:25:12

标签: c++ mergesort

更新:合并功能中的修改条件,现在一切正常

我正在尝试在c ++中实现合并排序算法,该算法适用于整数向量。所以这是我的代码:

#include <vector>
#include <iostream>

using namespace std;

void Vout (vector <int> V){
    for (int i = 0; i<V.size(); i++)
        cout << V[i] << "\t";
    cout << endl;
}

vector <int> merge (vector <int> B, vector <int> C){
    vector <int> D;
    int n = B.size() + C.size();
    int i=0, j=0;

    for (int k = 0; k<n; k++){
        if( ( B[i]<C[j] || j == C.size() ) && i<B.size() ){  //was like this: if( B[i]<C[j] ){
            D.push_back(B[i]);
            i++;
        }
        else{
            D.push_back(C[j]);
            j++;
        }
    }
    return D;
}

vector <int> merge_sort(vector <int> A){
    int n = A.size();
    if (n<=1) return A;
    vector <int> B, C;
    for (int i = 0; i<n/2; i++)
        B.push_back(A[i]);
    for (int i = n/2; i<n; i++)
        C.push_back(A[i]);

    B = merge_sort(B);
    C = merge_sort(C);
    vector <int> D = merge(B, C);
    return D;
}


int main() {
    vector <int> data;
    freopen("input.txt", "rt", stdin);
    int n;
    while (cin >> n){
        data.push_back(n);}
    Vout (data);
    Vout (merge_sort(data));
}

然而,这个输入

  

5 4 6 7 9 1 2 8 3 12 10

它提供以下输出:

  

1 2 3 0 4 5 6 7 8 9 0

我不太明白所有这些零来自哪里。请帮忙。

2 个答案:

答案 0 :(得分:3)

您的merge()函数无法处理BC向量的结尾;当它到达任何一个结尾时,你的比较(B[i]<C[j])将与数组末尾之后的比较进行比较。

答案 1 :(得分:0)

我修改了下面给出的合并功能并且它有效。

vector <int> merge (vector <int> B, vector <int> C){
    vector <int> D;
    int n = B.size() + C.size();
    int i=0, j=0;

    for (int k = 0; k<n; k++){
        if(B[i]<C[j]){
            D.push_back(B[i]);
            i++;

            if(i == B.size())
                break;
        }
        else{
            D.push_back(C[j]);
            j++;

            if(j == C.size())
                break;
        }
    }

    if(i == B.size())
    {
        while(j < C.size())
            D.push_back(C[j++]);
    }

    if(j == C.size())
    {
        while(i < B.size())
            D.push_back(B[i++]);

    }
    return D;
}