求未知数量输入的平均值

Question

我需要编写一个 C 程序，使用函数计算用户输入的成绩的平均值。输入的等级数未知，所以我必须使用标记值 -1。

这是我的代码，它编译得很好，但是它没有告诉我平均值，并且 for 循环没有调用该函数四次。我错过了什么？

 #include <stdio.h>

 float calculateAverage (int sumofGrades, int numberofGrades);

 int main(void) {

 int i;
 int sumofGrades = 0;
 int numberofGrades = 0;
 int grade = 0;
 float average;

 for (i = 1; i <= 4; ++i) {

    calculateAverage(sumofGrades, numberofGrades);

 }
 return 0;
 }

float calculateAverage (int sumofGrades, int numberofGrades) {

float average;
int grade;

printf("Enter grades and -1 to stop: ");
scanf("%d", &grade);
while (grade <= 100 && grade != -1) {
    numberofGrades++;
    sumofGrades += grade;
    average = sumofGrades / numberofGrades;

}
printf("The average is %.2f", average);

return average;

}

更新

感谢您的帮助！但我还不能做对。对于作业，我需要程序要求我输入 4 次未知的分数，并且每次计算平均值。

另外，为什么在我输入 -1 后它不停止并给出平均值？

#include <stdio.h>

float calculateAverage(int sumofGrades, int numberofGrades);

int main(void) {
    int i;
    int sumofGrades = 0;
    int numberofGrades = 0;

    for (i = 1; i <= 4; ++i) {
        calculateAverage(sumofGrades, numberofGrades);
    }
    return 0;
}

float calculateAverage (int sumofGrades, int numberofGrades) {

    float average;
    int grade = 0;

    while (grade != -1 && grade >= 0 && grade <= 100) {
        printf("Enter grades and type -1 to stop: ");
        scanf("%d", &grade);

        if (grade != -1) {
            numberofGrades++;
            sumofGrades += grade;
        }
    }

    average = sumofGrades / numberofGrades;
    printf("The average is %.2f\n", average);

    return average;
}

更新

我仍然有问题！我无法让程序要求我输入四次成绩并每次计算平均值！我试过了，我试过了，这太令人沮丧了！！我做错了什么？？

Answer 1

输入一个有效成绩后，您的代码不会尝试读取另一个成绩：

printf("Enter grades and -1 to stop: ");
scanf("%d", &grade);
while (grade <= 100 && grade != -1) {
    numberofGrades++;
    sumofGrades += grade;
    average = sumofGrades / numberofGrades;
}

如果成绩有效，则进入无限循环。它还计算每个周期的平均成绩，这是不必要的。一次，在循环之后，就足够了。

也许你应该使用：

printf("Enter grades and -1 to stop: ");
while (scanf("%d", &grade) == 1 && grade <= 100 && grade >= 0)
{
    numberofGrades++;
    sumofGrades += grade;
}
if (numberofGrades == 0)
{
    fprintf(stderr, "Error: zero valid grades entered\n");
    average = 0.0;
}
else
{
    average = sumofGrades / numberofGrades;
    printf("The average is %.2f", average);
}
return average;

请注意，这会在使用成绩之前检查输入是否成功。

您也真的不需要将变量传递给函数。您最好让函数不接受任何参数，并将参数作为常规局部变量正确初始化为零。

您的主程序应该在返回后捕获平均值，而不是忽略它。或者也许函数毕竟不应该返回平均值。

Answer 2

首先，您要循环 4 次 calculateAverage 函数。你是故意这样做的吗？我认为你应该只做一次。

其次，在你的 while 循环中

while (grade <= 100 && grade != -1) {
    numberofGrades++;
    sumofGrades += grade;
    average = sumofGrades / numberofGrades;

}

你在无限循环。如果输入了 -1 以外的值，这将永远循环。这应该改为 if 语句。

if (grade <= 100 && grade != -1)

但是有更好的方法来编写这段代码（如果我猜你真正想要它做什么）

#include <stdio.h>

float calculateAverage (int sum, int size);

int main(void) 
{

    int input;
    int sum = 0;
    int count = 0;
    while (input != -1 && input >= 0 && input <= 100)
    {
       printf ("Enter a Grade: ");
       scanf ("%d", &input);

       if (input != -1)
       {
           sum += input;
           count++;
       }
    }

    float average = calculateAverage(sum, count);
    return 0;
}

float calculateAverage (int sum, int count) {

   // Do your average math here....
}

Answer 3

鉴于发布的代码，子函数无法更新 main() 函数中的变量。

建议将指针传递给变量并修改子函数以期望（并正确处理）这些指针