Question

我尝试实现合并排序算法，我得到了一个分段错误。为什么？错误似乎在MergeSort函数中。合并排序函数（在第二次调用时）应该只检查4个数字的数组（长度应为4）显示长度= 27.为什么？（在具有8个元素的阵列上测试）

#include<iostream>
using namespace std;

int n, A[1000];

void citire(int lungime) {
    for (int i = 0; i < lungime; i++) cin >> A[i];
}

void afisare(int lungime) {
    for (int i = 0; i < lungime; i++)
        cout << A[i] << " ";
    cout << '\n';
}

int lungime(int A[]) {
    int i = 0;
    while (A[i]) i++;
    return i;
}

void Merge(int L[], int R[], int A[]) {
    int nL = lungime(L);
    int nR = lungime(R);
    int i = 0, j = 0, k = 0;

    while (i < nL && j < nR) {
        if (L[i] <= R[j]) {
            A[k] = L[i];
            i++;
        }
        else {
            A[k] = R[j];
            j++;
        }
        k++;
    }

    while (i < nL) {
        A[k] = L[i];
        i++;
        k++;
    }

    while (j < nR) {
        A[k] = R[j];
        j++;
        k++;
    }
}


void MergeSort(int A[]) {
    int n1 = lungime(A);
    if (n1 < 2) return;
    else
    {
        int mid = (int)n1 / 2;
        int L[mid];
        int R[n - mid];

        for (int i = 0; i < mid; i++)
            L[i] = A[i];
        for (int i = mid; i < n; i++)
            R[i - mid] = A[i];
        MergeSort(L);
        MergeSort(R);
        Merge(L, R, A);
    }
}


int main() {
    cin >> n;
    citire(n);
    MergeSort(A);
    afisare(n);
    return 0;
}

Answer 1

“lungime函数是字符串的长度，这个函数运行良好。我在不同的数组上测试过它”。嗯，这纯粹是偶然的;未初始化的内存可以包含零，并偶然提供数组终止符。如果你想保持当前的设计，你应该：

将A初始化为零
确保输入流中的元素不超过999个
没有元素的值为零，因为保留为零，并用作终结符，
将L和R（在MergeSort中）更长一个元素，并将最后一个元素初始化为零。

除非有“推出自己的”排序解决方案的压倒性原因，否则您可以查看预制排序支持。 C ++中的vector类就是这样。

Answer 2

此示例中所做的更改。 A []，L []，R []使用new分配。 A []作为参数传递。 L []和R []在Merge（）中分配。作为参数传递的大小和/或索引，不再使用lungime（）来获取大小。评论中注明了其他变化。

#include<iostream>
using namespace std;

void citire(int A[], int lungime) {     // A is parameter
    for (int i = 0; i < lungime; i++) cin >> A[i];
}

void afisare(int A[], int lungime) {    // A is parameter
    for (int i = 0; i < lungime; i++)
        cout << A[i] << " ";
    cout << '\n';
}

// A, low, mid, end are parameters
// L and R allocated here
void Merge(int A[], int low, int mid, int end) {
    int sizeL = mid-low;
    int sizeR = end-mid;
    int *L = new int[sizeL];
    int *R = new int[sizeR];

    for(int i = 0; i < sizeL; i++)
        L[i] = A[low+i];                // A[low+i]
    for(int i = 0; i < sizeR; i++)
        R[i] = A[mid+i];                // A[mid+i]

    int i = 0, j = 0, k = low;          // k = low
    while (i < sizeL && j < sizeR) {
        if (L[i] <= R[j]) {
            A[k] = L[i];
            i++;
        }
        else {
            A[k] = R[j];
            j++;
        }
        k++;
    }

    while (i < sizeL) {
        A[k] = L[i];
        i++;
        k++;
    }

    while (j < sizeR) {
        A[k] = R[j];
        j++;
        k++;
    }
    delete[] R;
    delete[] L;
}

// A, low, end are parameters
void MergeSort(int A[], int low, int end) {
    int sizeA = end - low;
    if(sizeA < 2)
        return;
    int mid = low + (sizeA / 2);        // mid = low + ...
    MergeSort(A, low, mid);
    MergeSort(A, mid, end);
    Merge(A, low, mid, end);
}

int main() {
    int n;
    cin >> n;
    int *A = new int[n];                // A is allocated
    citire(A, n);                       // A, n are parameters
    MergeSort(A, 0, n);                 // A, 0, n are parameters
    afisare(A, n);                      // A, n are parameters
    delete[] A;
    return 0;
}

分段错误：11 MergeSort

2 个答案: