Question

我正在尝试使用向量而不是C样式数组来获取以下代码进行合并排序工作，并且我很难弄清楚为什么它会因为分段错误而崩溃。有人可以帮助我理解这里的问题吗？

#include <bits/stdc++.h>
using namespace std;

void merge(vector<int>& a, int l, int m, int r) {
    vector<int> L;
    vector<int> R;

    for (int i = 0; i <= m; i++)
        L.push_back(a[i]);
    for (int i = m+1; i <= r; i++)
        R.push_back(a[i]);

    int i = 0, j = 0; // Initial index of first and second subarray
    int k = l;  // Initial index of merged subarray
    while (i < L.size() && j < R.size()) {
        if (L[i] <= R[j]) {
            a[k] = L[i];
            i++;
        }
        else {
            a[k] = R[j];
            j++;
        }
        k++;
    }

    // Filling leftovers
    while (i < L.size()) {
        a[k] = L[i];
        k++;
        i++;
    }
    while (j < R.size()) {
        a[k] = R[j];
        k++;
        j++;
    }
}

void merge_sort(vector<int>& a, int l, int r) {
    if (l < r) {
        int m = l + (r-l) / 2; // Avoids integer overflow.
        merge_sort(a, l, m);
        merge_sort(a, m+1, r);
        merge(a, l, m, r);
    }
}

int main()
{
    vector<int> a = {2, 4, 1, 5, 3, 9};
    int size = a.size();    
    merge_sort(a, 0, size-1);
    for (int i = 0; i < size; i++)
        cout << a[i] << ' ';
    cout << endl;
    return 0;
}

Answer 1

填充向量l时，您需要从m转到L：

void merge(vector<int>& a, int l, int m, int r) {
    vector<int> L;
    vector<int> R;

    for (int i = l; i <= m; i++)
        L.push_back(a[i]);
    ...
}

否则L包含感兴趣范围之外的其他元素。然后，当迭代k的大小时，索引a会从向量L的末尾开始。

通过引用传递向量：分段错误

1 个答案: