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，还有例如return; mergeSort()应该返回int*和未初始化的x。关于算法的另一个重要事项是：你似乎假设n是2的幂：你递归地除以2并假设它总是一个未被截断的整数。

Answer 3

首先，代码不能在C ++编译器中编译（我知道它是C代码，但仍然......），

有几个问题：

第11行：

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

为什么要转换指向int的指针？ a是int *

第38行：

return;

你从mergeSort返回没有返回值......这可能不是很好

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

程序在输入数组后挂起

3 个答案: