Question

我在遵循mergesort算法的Xcode上用C语言编写了这段代码。问题是，有时我得到EXC_BAD_ACCESS，我无法管理错误的位置！合并算法应该工作（我在mergesort函数之外尝试它并且工作！）。感谢您的帮助和耐心！

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#define DIM 6

void mymerge (int v[], int i1,int i2, int last); //mergesort core: merge two ordinated arrays in one bigger ordinated array
void mymergesort (int v[], int lower, int upper);//mergesort
void printv (int v[],int lower, int upper);

int main () {
    int i;
    srand((unsigned int)time(NULL));
    int v[DIM];
    for (i=0; i<DIM; i++)
        v[i]=rand()%15;
    printv(v, 0, DIM-1);
    getc(stdin);
    mymergesort(v, 0, DIM-1);
    printv(v, 0, DIM-1);
}
void printv (int v[],int lower, int upper){
    int i;
    for (i=lower; i<=upper; i++)
    printf("%d\t",v[i]);
}
void mymergesort (int v[], int lower, int upper){
    int mid=(upper+lower)/2;
    if (upper<lower) {
        mymergesort(v, lower, mid);
        mymergesort(v, mid+1, upper);
        mymerge(v,lower,mid+1,upper);
    }
}
void mymerge (int v[], int i1,int i2, int last){
    int i=i1,j=i2,k=i1,*vout;
    vout=(int*)malloc((last-i1+1)*sizeof(int));
    while (i<i2 && j<=last) {
        if (v[i]<=v[j]) {
            vout[k++]=v[i++];
        }else {
            vout[k++]=v[j++];
        }
    }
    for (;i<i2;i++) vout[k++]=v[i];
    for (;j<=last;j++) vout[k++]=v[j];
    for (k=i1; k<=last; k++) v[k]=vout[k];
free(vout);
}

编辑：非常感谢你！但我认为还有另一个问题，当我尝试排序一个更大的数组（200个元素）时，程序不起作用（我得到一个malloc错误：释放对象的错误校验和 - 对象可能在被释放后被修改）。但是，如果我从xCode调试器运行它一切正常

Answer 1

这：vout=(int*)malloc((last-i1)*sizeof(int));错了。

首先，您想要的元素数量是last-i1+1，而不是last-i1 - 经典的减1。这种错误是为什么C代码中的约定要使包含下限和上限独占的原因之一 - 您需要做的更少+1和-1，更少的机会搞砸了。 / p>

更严重的错误是您从vout开始索引i1。如果你这样做，你需要为last+1分配vout元素，而你永远不会使用第一个i1（索引0 .. i1-1）。

修复：首先，分配last-i1+1个元素。其次，在开始时将k初始化为0，而不是i1。第三，将最终副本更改为

for (k=i1; k<=last; k++) v[k] = vout[k-i1];

Answer 2

你有两个问题。首先，您对中点的计算不正确 - 您使用的是(upper - lower)/ 2，但不能保证在lower和upper之间。你真正想要的是lower + (upper - lower) / 2。如果要排序的区间中只有1个数字，也没有必要做任何工作 - 所以mymergesort()函数应该是这样的：

void mymergesort (int v[], int lower, int upper)
{
    if (upper > lower) {
        int mid = lower + (upper - lower)/2;

        mymergesort(v, lower, mid);
        mymergesort(v, mid+1, upper);
        mymerge(v,lower,mid+1,upper);
    }
}

第二个问题是Fabian Giesen已经指出的mymerge()函数中的问题。

Answer 3

#include<stdio.h>
#include<stdlib.h>

void merge(int *a, int n1, int *b, int n2, int *arr)
{
  int i=0, j=0, n=0;
  while(i<n1 && j<n2)
  {
    if (a[i] < b[j])
    {
      arr[n++] = a[i];
      i++;
    }
    else
    {
      arr[n++] = b[j];
      j++;
    }
  }
  while( i < n1)
    arr[n++] = a[i++]; 
  while( j < n2)
    arr[n++] = b[j++];
}
void merge_sort(int *a, int n)
{
  int left[n/2], right[n-n/2],i=0;
  if (n<=1)
    return ;
  while(i<n/2)
    left[i] = a[i++];
  while(i<n)
    right[i - n/2] = a[i++];
  merge_sort( left, n/2 );
  merge_sort( right, n-n/2);
  merge(left, n/2, right, n-n/2, a);
}  
void main()
{
  int a[] = { 6, 5, 3, 1,9, 8, 7, 2, 4},i;
  merge_sort(a,sizeof(a)/sizeof(a[0]));
  for(i=0;i<9;i++)
    printf("--%d",a[i]);
  printf("\n");
}


-- s.k

Answer 4

#include<stdio.h>
#include<conio.h>
#define max 20
/*** function  for merging the adjecent subarrays in sorted order ***/
void merge(int A[max],int n,int low,int high, int mid)
{
     int i=low,j=mid+1,k,temp;
     while((i<=j)&&(j<=high))     
     {
        if(A[i]>A[j])          /** if element of the second half is greater then exchg and shift **/
        {
           temp=A[j];
          for(k=j;k>i;k--)    /** shifting the elements **/
           {
             A[k]=A[k-1]; 
           }
           A[i]=temp;
           j++;
        }
        i++;
     }
}
/******* iterative function for merge sort ********/
void merge_sort(int A[max],int n,int low,int high)
{
     int mid;
     if(low<high)                     /** terminating condition  **/
     {
        mid=(high+low)/2;            /** calculating the mid point ***/
        merge_sort(A,n,low,mid);     /*** recursive call for left half of the array ***/
        merge_sort(A,n,mid+1,high);  /*** recursive call for right half of the array ***/
        merge(A,n,low,high,mid);    /** merging the both parts of the array **/
     }
}
/******* begening of the main function **********/
int main()
{
    int A[max],n,i;
    /** reading the inputs fro  users **/
    printf("\n enter the size of the array\n");
    scanf("%d",&n);
    printf("\n enter the array \n");
    for(i=0;i<n;i++)
    {
       scanf("%d",&A[i]);                 
    }
    /*** calling merge sort ***/
    merge_sort(A,n,0,n-1);
    /** printing the sorted array **/
    for(i=0;i<10;i++)
    {
       printf("\n\t%d",A[i]);                 
    }
    getch();
    return 0;
}

合并排序C实现

4 个答案: