Question

该程序应该交换没有公共分母的相邻元素，并且一个元素只能交换一次。当我运行程序时，几乎所有输入都可以正常工作。除了这一点：

100 //输入元素数量

48 92 76 93 17 38 59 34 53 99 58 20 50 0 38 37 16 36 91 12 59 1 76 82 20 76 7 72 13 70 64 23 81 70 41 69 11 0 16 41 37 83 41 99 73 79 4 38 24 32 87 38 95 24 77 30 61 13 89 67 87 76 22 31 67 31 25 90 6 76 21 43 40 55 72 91 91 28 18 58 72 71 83 22 99 23 86 58 75 53 69 29 5 55 46 8 98 55 19 46 //元素

对于此输入，程序将挂起并且不打印任何内容。有人知道在这种情况下发生了什么吗？

#include <stdio.h>
int nzd(int a, int b)
{
    if(a==b || b==0) 
        return a;
    if(a>b) 
        return nzd(a-b, b);
    return nzd(a, b-a);
}
int swap(int *niza, int i)
{
    int temp;
    temp=*(niza+i);
    *(niza+i)=*(niza+i+1);
    *(niza+i+1)=temp;
}
int main()
{
    int a[100], n, i;
    scanf("%d", &n);
    for(i=0; i<n; i++)
    {
        scanf("%d", &a[i]);
    }
    for(i=0; i<n; i++)
    {
        if(i+1==n) continue;
        if(nzd(a[i], a[i+1])==1)
        {
            swap(a, i);
            i++;
        }
    }
    for(i=0; i<n; i++)
    {
        printf("%d ", a[i]);
    }
    return 0;
}

Answer 1

您的gcd函数检查b==0的大小写，而不检查a==0的大小写。由于您跳过了该检查，因此最终呼叫了nzd(0, b-0);，该呼叫与先前的呼叫完全相同。这使您陷入无限递归循环，最终将导致堆栈溢出。

在您的函数中添加针对这种情况的检查：

if(a==b || b==0 || a == 0)

另外，称为Euclid算法的gcd的更快实现如下：

int gcd(int a, int b)
{
    if (b==0) {
        return a;
    } else {
        return (b, a%b);
    }
}

Answer 2

您的函数nzd()无法正确处理案件a == 0，并陷入了无限循环。您也需要处理这种情况：

int nzd(int a, int b)
{
    if(a==b || a==0 || b==0) 
        return a;
    if(a>b) 
        return nzd(a-b, b);
    return nzd(a, b-a);
}

我的代码中某个测试用例的奇怪行为

2 个答案: