c - 为什么每次循环我的 sum 和 count 值都会丢失？

Question

即使每个循环都应该填充变量，我仍然得到一致的除以零错误。代码如下:

#include <stdio.h>

 void calculateAverage()
{
int grade, count, sum;
double average;


sum = 0;
count = 0;
grade = 0;
average = 0.0;
int coolvalue = 0;

while (coolvalue==0)
{
    scanf("%d", &grade);
    if (grade == -1)
    {
        sum, sizeof(double);
        count, sizeof(double);
        average = (sum / count);
        printf("%lf", &average);
        break;
    }
    else
    {
        if ((grade > 100) || (grade < -1))
        {
            printf("Error, incorrect input.\n");
            break;
        }
        else
        {
            sum = +grade;
            count = count + 1;
            return count;
            return sum;
        }
    }
}
coolvalue = 1;
}

int main(void)
{
while (1)
calculateAverage();

while (1) getchar();
return 0;
}

即使使用 return，我也无法正确增加 sum 或 count 的值。

Answer 1

您的代码中存在多个问题。

scanf("%d", &grade); - 您不检查 scanf() 返回的值。它返回成功读取的值的数量。如果您输入一串字母而不是数字，scanf("%d") 将返回 0 并且不会更改 grade 的值>。因此，代码将使用 grade 的先前值执行循环的其余部分。如果 scanf() 返回的值不是 1，您应该重新启动循环:

if (scanf("%d", &grade) != 1) {
    continue;
}

假设您为 grade 输入 10，此代码块将执行:

sum = +grade;
count = count + 1;
return count;
return sum;

• sum = +grade 与 sum =grade 相同。 grade 前面的 + 符号没有任何作用。它与0 + 等级相同。

  您想要将grade的值添加到sum中，它应该是sum +=grade。这是 sum = sum +grade 的快捷方式。

• return count 使函数完成并将 count 的值(此时为 1)返回给调用者。调用者是函数main()，但它不以任何方式使用返回值。更重要的是，你的函数被声明为返回void(即什么都没有)，这会导致返回计数不正确(编译器应该警告你这一点)。

• return sum 永远不会被执行(编译器应该警告您它是死代码)，因为函数完成并且由于 return 而执行被传递回调用者count 语句位于其上方。

  删除两个 return 语句。他们不能留在这里。

如果您为 grade 输入 -1，则会执行此代码块:

sum, sizeof(double);
count, sizeof(double);
average = (sum / count);
printf("%lf", &average);
break;

• sum, sizeof(double) 是一个没有任何效果的表达式；它获取 sum 的值，然后丢弃它，然后获取 sizeof(double) 的值(这是一个常量)并丢弃它。编译器甚至不为其生成代码。

• 与上面的 count, sizeof(double) 相同；

• 平均值=(总和/计数)；:

  • 括号没有用；

  • 因为sum和count都是整数，所以sum/count也是一个整数( sum/count，余数被忽略)。

    您将average声明为double；要获得 double 结果，您必须在除法上将其中一个值转换为 double:average = (double)sum/count;

  • 如果在程序启动时输入 -1 作为第一个值，则执行此代码时 count 为 0，并且除法失败(除以零)。

• printf("%lf", &average); - 您想要打印 average 的值，但打印的是它在内存中的地址。删除&运算符； scanf() 需要它(以知道将读取的值放在哪里)。 printf() 不需要它；编译器生成代码，将要打印的值传递给 printf()。

• break; - 它在最里面的 switch 或循环语句(do、while 或 for)。这里是正确的，并且使变量 coolvalue 无用。您只需删除 coolvalue 并使用 while (1) 即可。

总而言之，您的函数应该如下所示:

void calculateAverage()
{
    int sum = 0;
    int count = 0;
    int grade = 0;
    double average = 0.0;

    while (1) {
        if (scanf("%d", &grade) != 1) {
            // Invalid value (not a number); ignore it
            continue;
        }

        // A value of -1 signals the end of the input
        if (grade == -1) {
            if (count > 0) {
                // Show the average
                average = (double)sum / count;
                printf("Average: %lf\n", average);
            } else {
                // Cannot compute the average
                puts("You didn't enter any value. Cannot compute the average.\n");
            }
            // End function
            return;
        }

        if ((grade < -1) || (100 < grade)) {
            puts("Error, incorrect input.\n");
            // Invalid input, ignore it
            continue;
        }

        sum += grade;
        count ++;
    }
}