Question

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

void merge(int a[],int l,int m,int r)
{
    int i,j,k;
    int n1= m - l + 1;
    int n2= r - m;
    int L[n1],R[n2];
    for(i=0;i<n1;i++)
        L[i]=a[l+i];
    for(j=0;j<n2;j++)
        R[j]=a[m + 1+ j];
    i=0;j=0;k=l;
    while(i<n1 && j<n2)
    {
        if(L[i]<R[j])
        {
            a[k]=L[i];
            i++;
        }
        else
        {
            a[k]=R[j];
            j++;
        }
        k++;
    }
    while(i<n1)
    {
        a[k]=L[i];
        i++;
        k++;
    }
    while(j<n2)
    {
        a[k]=R[j];
        j++;
        k++;
    }
}

void mergeSort(int a[],int l,int r)
{
    if(l<r)
    {
        int m = l+(r-1)/2;
        mergeSort(a,l,m);
        mergeSort(a,m+1,r);
        merge(a,l,m,r);
    }   
}

void printArray(int a[],int n)
{
    int i;
    for(i=0;i<n;i++)
        printf("%d   ",a[i]);
    printf("\n");
}

int main()
{
    int a[]= {
                12,76,34,45,63,98
             };
    int n = sizeof(a)/sizeof(a[0]);
    printf("The element entered in array   ");
    printArray(a,n);
    mergeSort(a,0,n-1);
    printf("The element after sorting    ");
    printArray(a,n);
    return 0;
}

这是一个合并排序程序。如果有人能解释，为什么它会给出运行时错误？该程序没有错误

我试了10次。而且无法找到解决方案。

Answer 1

当您致电mergesort(a, 3, 5)时，会发生以下情况：

int m = l+(r-1)/2;  // 3 + (5-1)/2 -> 3 + 4/2 -> 3 + 2 -> 5
mergeSort(a,l,m);   // So this will call: mergesort(a, 3, 5) again

换句话说：无限循环。

也许你想要这个：

int m = (l+(r-1))/2;

        ^       ^
         notice

那我是怎么找到这个错误的？

非常简单 - 只需使用一些printf-debug。

首先，我在mergesort的开头添加了一个打印件，如：

void mergeSort(int a[],int l,int r)
{
    printf("mergesort %d %d\n", l, r);  // Debug print
    if(l<r)
    {
        int m = (l+(r-1))/2;
        mergeSort(a,l,m);
        mergeSort(a,m+1,r);
        merge(a,l,m,r);
    }   
}

得到了输出：

mergesort 3 5
mergesort 3 5
mergesort 3 5
...
...

告诉我输入值3和5有一个无限循环。

然后我添加了m

的印刷品

void mergeSort(int a[],int l,int r)
{
  printf("mergesort %d %d\n", l, r);
    if(l<r)
    {
        int m = (l+(r-1))/2;
        printf("m %d\n", m);
        mergeSort(a,l,m);
        mergeSort(a,m+1,r);
        merge(a,l,m,r);
    }   
}

得到了输出：

mergesort 3 5
m 5
mergesort 3 5
m 5
mergesort 3 5
m 5

所以很明显错误地计算了m。

近距离观察

int m = l+(r-1)/2;

很明显，在分割之前，添加应该是。缺少一组(....)。

希望您可以将此调试示例用于您自己的调试。

合并排序程序

1 个答案: