4 个变量函数的平均值打印“4.00000”（除以 4）而不是实际平均值

Question

我想知道我是否使用了正确的变量类型。据我所知，我应该只使用浮点数和整数。我知道它打印“除以 4”部分，因为在函数中我将其切换为除以 5 并打印“5.0000”。如果我将“结果”设为整数，它显然会去掉小数点的答案，但它仍然给我正确的平均值而不是“4”。 为什么将“4.0000”存储在结果变量而不是答案中？

void main(void)
{
WDT_A->CTL = WDT_A_CTL_PW | WDT_A_CTL_HOLD;
// stop watchdog timer\

srand(time(NULL));
int val1 = (rand()%100);
int val2 = (rand()%100);
int val3 = (rand()%100);
int val4 = (rand()%100);
printf("%d\n",val1);
printf("%d\n",val2);
printf("%d\n",val3);
printf("%d\n",val4);

printf("%f\n",average4(val1, val2, val3, val4));
}


float average4(int val1, int val2, int val3, int val4){
float result;
result = (val1 + val2 + val3 + val4)/4;
return result;
}

Answer 1

两个问题：

1. (val1 + val2 + val3 + val4)/4; 以整数运算 计算，因为这些项是整数类型。剩余部分被丢弃。

   一种解决方法是编写1.f * (val1 + val2 + val3 + val4) / 4; 或稍微混淆的0.25f * (val1 + val2 + val3 + val4);

2. 您缺少average4 的原型。正式地，这样做的行为是未定义（自 C99 起并包括 C99），您可能会在调用站点观察到到 int 的转换。

Answer 2

以下建议的代码：

1. 没有完全编译，因为三个字段：WDT_A WDT_A_CTL_HOLD 和 WDT_A_CTL_PW 没有声明。

现在是建议的代码，带有注释

#include <stdio.h>     // printf()
#include <stdlib.h>    // rand(), srand()
#include <time.h>      // time()

// prototypes
float average4(int val1, int val2, int val3, int val4);


int main(void)                       // corrected signature
{
    WDT_A->CTL = WDT_A_CTL_PW | WDT_A_CTL_HOLD;
    // stop watchdog timer           // removed trailing line continuation 'slash'

    srand( (unsigned int)time(NULL)); // cast to expected type
    int val1 = (rand()%100);
    int val2 = (rand()%100);
    int val3 = (rand()%100);
    int val4 = (rand()%100);
    printf("%d\n",val1);
    printf("%d\n",val2);
    printf("%d\n",val3);
    printf("%d\n",val4);

    printf("%f\n",average4(val1, val2, val3, val4));
}


float average4(int val1, int val2, int val3, int val4)
{
    float result;
    // cast the 'integer' sum to 'float',
    // properly declared the '4' as a 'float' literal
    // so division is 'float' rather than `'integer' division
    result = (float)(val1 + val2 + val3 + val4)/4.0f;
    return result;
}

为了测试，我注释掉了试图弄乱看门狗定时器的那行。

我使用gcc 运行上述代码来编译/链接，然后在我的 ubuntu linux 系统上打开一个终端，这里有几个典型的执行。

32
13
74
29
37.000000

和

77
53
90
49
67.250000