C 程序将在 Linux 而非 Windows 中编译和运行

Question

我正在为一门编程课做作业，但我遇到了一个程序问题。目标是获取一组用户定义的值，将它们存储在一个数组中，然后找到最接近的一对数字（差异最小的数字）。这些数字不必有连续的索引。此外，教授在作业中定义了 50 的数组大小。

我遇到的问题是该程序将在 Linux（Ubuntu 14.xx）和 Windows 10 中编译，但是，当我在 Linux 中运行结果时，它工作正常，但在 Windows 中它什么也不输出。

这是我第一次遇到这个问题，据我所知，我没有使用任何系统特定的命令。提供的任何帮助将不胜感激。

代码：

#include <stdio.h>

int main()
{
    int i, j, a, b, temp, mindiff, count;
    int numarray[50];

    count = 0;

    for (i = 0; i < 50; i++)
    {
        scanf("%d", &numarray[i]);
        count++;
        if (numarray[i] == -1)
        {
            numarray[i] = 0;
            count--;
            break;
        }
    }
    mindiff = 100;
    for (i = 0; i < count; i++)
    {
        for (j = 0; j < count; j++)
        {
            a = numarray[i];
            b = numarray[j];
            if (a != b)
                if (a > b)
                    temp = a - b;
                else    
                    temp = b - a;

            if (temp < mindiff)
                mindiff = temp;
        }
    }

    for (i = 0; i < count; i++)
    {
        for (j = 0; j < count; j++)
        {
            a = numarray[i];
            b = numarray[j];
            if (a != b) 
            {
                if (a > b && (a - b) == mindiff)
                {
                    printf("Closest pair: %d and %d, Difference: %d\n", a, b, mindiff);
                    return 0;
                }
            }
        }
    }

    return 0;
}

Answer 1

至少有两个主要问题：

• 你的代码有未定义的行为，因为你使用了temp，即使它没有被设置（你应该在if (a != b)之后开始一个块）。 未定义的行为意味着任何事情都可能发生，包括在 Linux 上的明显成功和在 Windows 上的失败。

• 您将mindiff 初始化为100。如果所有数字彼此相距较远，mindiff 将不会更改，第二个循环也不会打印任何内容。

这是一个更简单的版本：

#include <stdio.h>

int main(void) {
    int count, i, j, mina, minb, mindiff;
    int numarray[50];

    for (count = 0; count < 50; count++) {
        if (scanf("%d", &numarray[count]) != 1 || numarray[count] == -1)
            break;
    }
    mindiff = mina = minb = 0;
    for (i = 0; i < count; i++) {
        for (j = 0; j < count; j++) {
            int a = numarray[i];
            int b = numarray[j];
            if (a > b) {
                int diff = a - b;
                if (mindiff == 0 || mindiff > diff) {
                    mindiff = diff;
                    mina = a;
                    minb = b;
                }
            }
        }
    }
    if (mindiff == 0) {
        if (count == 0) {
            printf("No numbers input\n");
        } else {
            printf("The numbers are all identical\n");
        }
    } else {
        printf("Closest pair: %d and %d, Difference: %d\n",
               mina, minb, mindiff);
    }
    return 0;
}

Answer 2

您有以下问题：
注意：temp被认为是main函数下的堆栈变量，由于你没有初始化它，它变成了垃圾（例如一个很大的负数）：

int i, j, a, b, temp, mindiff, count;

那么如果temp是一个很大的负数，mindiff = mindiff的垃圾值，

        if (temp < mindiff)
            mindiff = temp;

并且下面的 if 语句总是错误的。 if (a > b && (a - b) == mindiff)

什么都没有打印出来。