输入数组后程序挂起

Question

#include<stdio.h>
#include<conio.h>
//#include<alloc.h>

int* mergeSort(int*,int);
int* merge(int*,int*,int);

void main()
{
  int n;int i=0;
  int *a,*b;

  scanf("%d",&n);
  a=(int)malloc(n*sizeof(int));
  for(;i<n;i++)
     scanf("%d",&a[i]);

  b=mergeSort(a,n);
  for(i=0;i<n;i++)
     printf("%d ",b[i]);

}

int* mergeSort(int *b,int n)
{
  int temp,*s;

  if(n>2)
  {
     mergeSort(b,n/2);
     mergeSort(b+n/2,n-n/2);
     s=merge(b,b+n/2,n);

     return s;
  }
  else if(n==2)
  {
     if(b[0]>b[1])
     {
         temp=b[0];
         b[0]=b[1];
         b[1]=temp; 
     }
     return;
  }
}

int* merge(int* a,int* c,int n)
{
  int i=0,j=0,k=0,
  int* x;

  while( (j ! =n/2) && (k != (n-n/2)) && (i < n))
  {
       if(a[j]<c[k])
       {
             x[i]=a[j];
             j++; 
             i++;
       }
       else
       {
             x[i]=c[k];
             k++;
             i++;
       }
   }
   for( ; j<n/2; j++,i++)
      x[i]=a[j];

   for( ; k < (n-n/2); k++,i++)
      x[i]=c[k];

   return x;
}

当我运行此代码时，它在第一个 for 循环中输入数组的所有元素后挂起。请帮助我，我该如何纠正它以使其成功运行？它挂起从 main() 函数调用 mergeSort 函数。

Answer 1

it hangs after inputting all the elements of the array in first for loop.

挂起？你确定......考虑到你的合并代码声明了一个指向 int 的指针，这很好：

int *x;

并且从不初始化它，然后尝试跳转到过去的偏移量 (i)：

x[i]=a[j];

将您的代码更改为：

int *x = malloc(n * sizeof(int));

它应该停止崩溃/挂起。

仅供参考，无论您何时malloc()，您现在都应该free() 出现内存泄漏。

Answer 2

有些地方出了问题，不仅是基本的东西，比如void main 而不是int main，还有例如mergeSort() 中的return; 应该返回int* 和未初始化的x。关于算法还有一件重要的事情：您似乎假设 n 是 2 的幂：您递归地除以 2 并假设它始终是未截断的整数。

Answer 3

对于初学者，代码不能在 C++ 编译器中编译（我知道它是 C 代码，但仍然...），

有几个问题：

第 11 行：

a=(int)malloc(n*sizeof(int));

你为什么要转换一个指向 int 的指针？ a 是一个整数 *

第 38 行：

return;

你从没有返回值的mergeSort返回......这不好

我建议您修复所有编译器错误和警告，然后重试