为什么我的代码返回 0.00？

Question

#include <stdio.h>
#include <math.h>

float math(int, int, int, int, int, float, float, float);

main() {
  int a, b, c, d, e;
  float sum, avg, sd;
  printf("Enter Five Integers->");
  scanf("%d%d%d%d%d", &a, &b, &c, &d, &e);
  math(a, b, c, d, e, sum, avg, sd);
  printf("Sum=%.2f\nAverage=%.2f\nStandard Deviation=%.2f", sum, avg, sd);
}

float math(int a, int b, int c, int d, int e, float sum, float avg, float sd) {
  sum = a + b + c + d + e;
  avg = (sum) / 5;
  sd = pow(
      ((pow(a - avg, 2) + pow(b - avg, 2) + pow(c - avg, 2) + pow(d - avg, 2),
        pow(e - avg, 2)) /
       5),
      0.5);
  return sum, avg, sd;
}

我的程序总是返回答案 0.00。谁能解释我的代码有什么问题？

Answer 1

TL;DR因为，您的代码调用了undefined behavior，无法证明输出的合理性。

首先要详细说明声明

 return sum, avg, sd;

没有做你认为它正在做的事情。它不会一起返回三个值，而是由于使用了comma operator，它只返回sd。

也就是说，您没有在代码中的任何地方收集函数调用的返回值，因此您无法在调用者那里获得从函数调用返回的任何有效输出。

在那之后，你最终会使用

 printf("Sum=%.2f\nAverage=%.2f\nStandard Deviation=%.2f", sum, avg, sd);

其中，提供的变量（自动、局部变量）保持未初始化状态，本质上是在尝试使用调用 undefined behavior 的不确定值。

最后，对于托管环境，main() 的一致性签名至少应该是 int main(void)。

解决方案：您需要

• 将指针传递给那些你想在其中存储被调用函数结果的变量，然后在调用者中你可以使用它们来检索更新的值。

• 形成一个结构，其中包含您希望为其返回计算值的所有变量，填充并返回该结构。然后，在调用者中，将返回的值收集到另一个结构类型变量中，然后使用单个成员元素打印该值。

Answer 2

您的代码中的取点很少。

1. 在 C 语言中，您不能从函数返回超过 1 个值。您正在尝试返回 3 个值，因为它们使用“逗号（，）”分隔，只返回最后一个值。
2. 两个函数的局部变量之间存在差异。如果没有适当的参考，您不能在 1 个函数中分配它们并在其他函数中使用它。
3. 您将函数设为“call by value”（在链接中了解更多信息）。如果您希望更改值，则应使用“call by refernce”，它使用指针。

所以对您来说最简单的解决方法是使用全局变量。

void math(int, int, int, int, int);     //Changed function declareation
 double sum,avg,sd; //New global variables

 int main()
 {
    int a, b, c, d, e;
    printf("Enter Five Integers->");
    scanf("%d%d%d%d%d", &a, &b, &c, &d, &e);
    math(a, b, c, d, e);        //Changed function calling
    printf("Sum=%.2f\nAverage=%.2f\nStandard Deviation=%.2f\n", sum, avg, sd);
    return 0;
}

 void math(int a, int b, int c, int d, int e)
 {
    sum = a + b + c + d + e;
    avg = (sum) / 5;
    double result = (pow(a - avg, 2) + pow(b - avg, 2) + pow(c - avg, 2) +  pow(d - avg, 2) + pow(e - avg, 2) ) / 5;
    sd = pow(result , 0.5);
 }