合并排序与递归

merge sort with recurrsion

本文关键字：递归 排序 合并      更新时间：2023-10-16
/*
This code is Written by Shankhadeep Dey
(15th Aug 2018)
*/
#include<iostream>
using namespace std;
void merge(int a[],int p,int q,int r)
{
int n1,n2,j,i,k;
n1= q-p+1;
n2= r-q;
int L[n1],R[n2];
for (i = 0; i < n1-1; ++i)
L[i]=a[p+i-1];
for (j = 0; j < n2-1; ++j)
R[j]=a[q+j];
L[n1]=1e9;
R[n2]=1e9;
i=0;
j=0;
for (k = p; k < r; ++k)
{
if (L[i]<=R[j])
{
a[k]=L[i];
i++;
}
else
{
a[k]=R[j];
j++;
}
}
while (i < n1)
{
a[k] = L[i];
i++;
k++;
}
/* Copy the remaining elements of R[], if there
are any */
while (j < n2)
{
a[k] = R[j];
j++;
k++;
}
}
void mergeSort(int a[],int p,int r)
{
int q;
if (p<r)
{
q= (p+r)/2;
mergeSort(a,p,q);
mergeSort(a,q+1,r);
merge(a,p,q,r);
}
}
int main()
{   int w[4]={4,3,21,5};
mergeSort(w,0,3);
for (int i = 0; i < 4; ++i)
{
cout<<w[i]<<" ";
}
cout<<endl;
}

谁能告诉我为什么这种排序不起作用？我从"算法简介"一书中尝试了这个算法，但我不太确定为什么这个代码没有运行。我尝试在线搜索合并排序并找到很多东西，但我确实想知道我的代码出了什么问题。 输出为：-0 1000000000 1 1000000000(这不是正确的 ans(。
这是我收到的警告

mergeSort.cpp: In function ‘void merge(int*, int, int, int)’:
mergeSort.cpp:12:10: warning: ISO C++ forbids variable length array ‘L’ [-Wvla]
int L[n1],R[n2];
^
mergeSort.cpp:12:16: warning: ISO C++ forbids variable length array ‘R’ [-Wvla]
int L[n1],R[n2];

合并逻辑中存在问题。想想当你想合并数组中的前两个元素时会发生什么。在本例中，p=0，q=0，r=1。在这种情况下n1=0，并且以下循环永远不会执行：

for (i = 0; i < n1-1; ++i)
L[i]=a[p+i-1];

与下一个循环相同。更糟糕的是，如果您要通过将n1-1更改为n1来修复它，那么在第一次迭代中，p+i-1将是 -1，这将导致更多问题。

如果您想通过与工作解决方案进行比较来作弊，我相信Cormen和Co的"算法简介"中的"如何实现合并排序"中有一个

更一般地说，我建议使用逐行调试器(gdb 或使用 Eclipse 等 IDE(将允许您逐行执行代码，检查执行时间以及每一步的变量值。这使得诊断此类错误变得更加容易。

(注意：此问题已在问题编辑中修复(。

这些行存在问题：

for (int i = 0; i < n2-1; ++i)
R[j]=a[q+j];

要么使用j要么使用i，但不能两者兼而有之。目前，您正在使用未初始化的值j。

附带说明一下，您将变量声明为i并j两次(一次在函数顶部附近，另一次在for循环中(。这不会阻止程序工作，但是在不同的范围内有两个同名的变量并不是很好的做法，因为它可能会导致混淆。

另外，在这一行中：

q= (int)(p+r)/2;

(int)部分没有任何作用，可能会误导读者。p和r已经是整数，因此编译器使用整数除法生成整数结果。

相关文章：