簡體   English   中英

mergesort C實現

[英]mergesort C implementation

我在遵循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調試器運行它一切正常

這個: vout=(int*)malloc((last-i1)*sizeof(int)); 是錯的。

首先,你想要的元素數量是last-i1+1 ,而不是last-i1 - classic off-by-1。 這種錯誤是C代碼中的約定是包含下限和上限的唯一原因之一 - 你需要做的更少+1-1 ,更少的機會搞砸了。

更嚴重的錯誤是你從i1開始索引vout 如果你這樣說,你需要分配last+1元為vout ,而且絕不使用第一i1 (索引為0 ... i1-1 )。

修復:首先,分配last-i1+1元素。 其次,在開始時將k初始化為0,而不是i1 第三,改變最終副本

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

你有兩個問題。 第一個是你的中點的計算是不正確的-使用(upper - lower)/ 2 ,但並不保證之間說謊lowerupper 你真正想要的是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()函數中的問題。

#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
#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;
}

暫無
暫無

聲明:本站的技術帖子網頁,遵循CC BY-SA 4.0協議,如果您需要轉載,請注明本站網址或者原文地址。任何問題請咨詢:yoyou2525@163.com.

 
粵ICP備18138465號  © 2020-2024 STACKOOM.COM