循环重复 scanf 和 printf

Question

我编写了一个程序，它将在 printf 之前两次 scanf 并输出两个应该是单个 printf 的内容。该问题似乎从要求用户输入 1 到 4 之间的数字以查看输入天数的平均温度时开始出现。

我不确定是什么导致了这种双重输入和输出以及偶尔的延迟。这是我的代码：

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

int main (void) {
    int i;
    int limit;
    int day[] = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 };
    int high[10], low[10];

    printf("---===IPC Temperature Analyzer V2.0===---\n");

    printf("Please enter the number of days between 3 and 10, inclusive: ");
    scanf("%d", &limit);
    while (limit <= 2 || limit >= 11) {
        printf("Invalid entry, please enter a number between 3 and 10, inclusive: ");
        scanf("%d", &limit);
    }

    for (i = 0; i < limit; i++) {
        printf("Day %d - High: ", day[i]);
        scanf("%d", &high[i]);
        printf("Day %d - Low: ", day[i]);
        scanf("%d", &low[i]);
    }

    printf("\nDay Hi Low\n");
    for (i = 0; i < limit; i++) {
        printf("%d   %d    %d\n", day[i], high[i], low[i]);
    }

    int max = 0;
    int min = 0;

    for (i = 0; i < limit; i++) {
        if (high[max] < high[i])
            max = i;
        if (low[min] > low[i])
            min = i;
    }

    printf("\nHighest temperature was: %d on day %d\n", high[max], day[max]);

    printf("Lowest temperature was: %d on day %d\n", low[min], day[min]);

    int n;

    do {
        printf("\nEnter a number between 1 and 4 to see the average temperature "
               "for the entered number of days, enter a negative number to exit:");
        scanf("%d\n", &n);

        while (n > 4) {
            printf("Invalid entry, please enter a number between 1 and 4, inclusive: ");
            scanf("%d", &n);
        }

        while (n < 0) {
            printf("Goodbye!\n");
            exit(0);
        }

        float avgSum = 0.0;
        for (i = 0; i < n; i++) {
            float avgOfDay = (high[i] + low[i]) / 2.0;
            avgSum += avgOfDay;
        }
        float overallAvg = avgSum / n;
        printf("The average temperature up to day %d is: %.2f\n", day[n - 1], overallAvg);

    } while (n > 0 || n < 4);

    return 0;
}

示例输出：

Enter a number between 1 and 4 to see the average temperature for the entered number of days, enter a negative number to exit:5
5
Invalid entry, please enter a number between 1 and 4, inclusive: Invalid entry, please enter a number between 1 and 4, inclusive: 3
The average temperature up to day 3 is: 2.50

Enter a number between 1 and 4 to see the average temperature for the entered number of days, enter a negative number to exit: 2
2
The average temperature up to day 2 is: 2.75

Enter a number between 1 and 4 to see the average temperature for the entered number of days, enter a negative number to exit: -1
The average temperature up to day 2 is: 2.75

Enter a number between 1 and 4 to see the average temperature for the entered number of days, enter a negative number to exit: -1
Goodbye!

Answer 1

while (n > 0 || n < 4); => while (n > 0 && n < 4);

Answer 2

我能够确定我的问题的解决方案。我的代码的某些部分的 scanf 和 printf 旁边的新行函数导致程序重复某些 scanf 和 printf 函数。

scanf("%d\n", &n) => 
printf("%d", &n)

printf ("The average temeprature up to day %d is: %.2f\n", day[n-1], overallAvg); =>  
printf ("The average temeprature up to day %d is: %.2f", day[n-1], overallAvg);

Answer 3

您描述的问题可以追溯到do循环开头的scanf()语句：

    scanf("%d\n", &n);

格式字符串末尾的换行符很麻烦。当scanf() 在格式字符串中遇到空白字符时，它会匹配输入流中的空白字符，直到遇到非空白字符。问题是当你按回车键输入你的号码时，这只是另一个空白字符，所以scanf() 贪婪地继续等待更多输入，直到非空白字符或EOF 是遭遇。此时用户输入一个非空白字符，scanf()空白匹配失败，输入的字符留在输入流中，scanf()最终返回调用函数，最后下一个scanf() 拿起刚刚被拒绝的角色。这就是您观察到的零星反应的原因。

解决这个问题很简单。只需从格式字符串的末尾删除\n。通常情况下，格式字符串末尾的空白字符是错误的。

代码中还有其他问题。 do 循环结束时的测试应该是：

while (n > 0 && n < 4);

退出值的测试最好使用if 语句而不是while 循环，并且测试应该是n < 1 而不是n < 0 以避免除以零错误：

if (n < 1) {
    printf("Goodbye!\n");
        exit(0);
}

看来你应该把输入提示改为：

printf("\nEnter a number between 1 and 3 to see the average temperature for the entered number of days, enter a negative number to exit:");

如果用户在这里选择4，但只输入了3天的数据，则计算将访问high[]和low[]数组中未初始化的值。您还需要将此输入循环更改为：

while (n > 3) {
            printf("Invalid entry, please enter a number between 1 and 3, inclusive: ");
            scanf("%d", &n);
        }

可能还有其他问题，但这应该可以正常运行。