没有通过计数排序得到正确的排序数组

Question

我目前正在研究计数排序算法。我设置了两个临时数组C 和B。 C是统计原始数组中某个数字出现的次数。然后它使用C 中的元素将A（原始数组）中的元素放入B 中的正确位置。我有我的countingSort 函数在每个循环之后打印出C，以确保它具有正确的值（确实如此，我正在用小样本量对其进行测试）。当我在C 的帮助下将A 的元素插入B 时，就会出现问题。

这是我对countingSort 的函数：

注意：我将一个由 10 个整数组成的数组 2 0 3 2 5 4 3 6 10 传递给函数、临时数组 B、maximum 值（所以我知道要制作什么大小 C）和数组大小 @987654339 @

void countingSort(int A[], int B[], int k, int size){
    int C[k + 1];
    cout << "K: " << k << endl;

    for(int i = 0; i < k + 1; i++){
        C[i] = 0;
    }



    for(int i = 0; i < k + 1; i++){
        cout << C[i] << " ";
    }


    for(int i = 0; i < size; i++){
        C[A[i]] = C[A[i]] + 1;
    }

    cout << endl;


    for(int i = 0; i < k + 1; i++){
        cout << C[i] << " ";
    }


    for(int i = 0; i < k + 1; i++){
        C[i] = C[i] + C[i - 1];
    }


    cout << endl;
    for(int i = 0; i < k + 1; i++){
        cout << C[i] << " ";
    }



    for(int i = size + 1; i > 0; i--){
        B[C[A[i]]] = A[i];
        C[A[i]] = C[A[i]] - 1;
    }


    cout << endl;
    for(int i = 0; i < size; i++){
        cout << B[i] << " ";
    }


}

如您所见，我在每个循环之后打印出C，所以在第一个循环之后，它应该显示C 是0 0 0 0 0 0，它确实可以正确打印出来。在下一个 for 循环之后，C 应该是 2 1 2 2 1 1 1，它也可以正确打印出来。接下来它添加C 的元素，得到2 3 5 7 8 9 10，它也被正确打印出来。现在，当我尝试将A 的元素放入B 时，我的问题就出现了。它应该打印0 0 1 2 2 3 3 4 5 6，但它打印0 0 0 1 0 2 3 3 4 5。

我曾尝试在最后一个 for 循环中使用我的索引，但似乎无法弄清楚导致 B 不正确的原因。我该如何解决这个问题？我的总体目标是让计数排序适用于随机生成的大小为40 的数组，其数字在 1 到 25 之间。

编辑：我调用countingSort的主要功能：

int sizeCount1 = 10;
int countOne[10] = {2, 0, 3, 2, 5, 4, 3 ,6, 1, 0};

cout << "Counting Sort Version 1 (Pre Sort)" << endl;

for(int i = 0; i < sizeCount1; i++){
    cout << countOne[i] << " ";
}

cout << endl;


for(int i = 0; i < sizeCount1; i++){
    countTemp[i] = 0;
}



int max = 0;
for(int i = 0; i < sizeCount1; i++){
    if(countOne[i] > max){
        max = countOne[i];
    }
}

cout << "Max: " << max << endl;


countingSort(countOne, countTemp, max, sizeCount1);

cout << endl;

cout << "Counting Sort Version 1 (Post Sort)" << endl;


for(int i = 1; i < 10; i++){
    cout << countTemp[i] << " ";
}

cout << endl << endl;

Answer 1

for(int i = 1; i < k + 1; i++){
    C[i] = C[i] + C[i - 1];
}

否则你会得到未定义的行为。

同样在输出数组形成

for(int i = size-1; i >= 0; i--){
    B[C[A[i]]] = A[i];
    C[A[i]] = C[A[i]] - 1;
}

你的算法是对的。现在只是试运行一下。这样你就可以在你的代码中找到这些类型的错误。

_{因为 OP 使用了0-indexing。我在我的答案中使用相同的}

_{如果你不能使用向量..使用new分配内存。请检查一下参考。}

_{另一件事是，每当您编写计数排序代码时，总是试图证明您可以将范围保存在辅助数组中。这有帮助。}

计数排序代码：

void countingSort(int A[], int B[], int k, int size){
    int C[k + 1];
    for(int i = 0; i < k + 1; i++){
        C[i] = 0;
    }
    for(int i = 0; i < size; i++){
        C[A[i]] = C[A[i]] + 1;
    }
    for(int i = 0; i < k + 1; i++){
        C[i] = C[i] + C[i - 1];
    }
    for(int i = size-1; i >= 0; i--){
        B[C[A[i]]] = A[i];
        C[A[i]] = C[A[i]] - 1;
    }
}

主要代码

int sizeCount1 = 10;
int countOne[10] = {2, 0, 3, 2, 5, 4, 3 ,6, 1, 0};

cout << "Counting Sort Version 1 (Pre Sort)" << endl;

for(int i = 0; i < sizeCount1; i++){
    countTemp[i] = 0;
}
int max = 0;
for(int i = 0; i < sizeCount1; i++){
    if(countOne[i] > max){
        max = countOne[i];
    }
}

cout << "Max: " << max << endl;


countingSort(countOne, countTemp, max, sizeCount1);
cout << "Counting Sort Version 1 (Post Sort)" << endl;
for(int i = 0; i < 10; i++){
    cout << countTemp[i] << " ";
}

cout << endl << endl;