为什么我的作业出现逻辑错误？

Question

我是一名IT专业的大三学生。 我面临程序输出的问题。 该程序的想法是，我应该使用函数读取 10 个元素的数组，然后获取元素的平均值，然后获取最大值和最小值。 我的最大值和最小值是正确的，但平均值显示出奇怪的东西。 请检查代码并告诉我应该做什么或以某种方式帮助我。

输出是（请注意，请求的是 11 个数字而不是 10 个数字，如果我更改循环参数以使其只需要 10 个数字，那么它会

enter the group of integers numbers
1
2
3
4
5
6
7
8
9
0
9
 1 2 3 4 5 6 7 8 9 0the avg is 3.500000
9
1Press any key to continue . . .

// func-sortarray.cpp : Defines the entry point for the console application.
//

#include "stdafx.h"
#define size  10

void readarray(int []);
void average(int []);
void printArray(int []);
void max(int []);
void min(int []);

int _tmain(int argc, _TCHAR* argv[])
{
    int sarray[size];
    readarray(sarray);
    printArray(sarray);
    average(sarray);
    max(sarray);
    min(sarray);
    return 0;
}

void readarray(int a[])
{
    printf("enter the group of integers numbers\n");
    for (int i=0; i<=size-1 ;i++)
        scanf("%d\n",&a[i]);
}

void average(int a[])
{
    int i;
    double avg;
    double total = 0;
    for (i=0; i <= size-1; i++)
    {
        total = total + a[i];
    }

    avg = total /size-1;
    printf("the avg is %f\n",avg);
}

void printArray(int a[])
{
    int j;
    for (j = 0; j <= size - 1; j++) 
        printf( "%2d", a[ j ]);
}

void max(int a[])
{
    int ma =a[0];

    for (int j=0;j<size-1;j++)
    {
        if (ma<a[j])
            ma=a[j];
    }
    printf("%d",ma);
}

void min(int a[])
{
    int mi =a[0];

    for (int j=0;j<size-1;j++)
    {
        if (mi>a[j])
            mi=a[j];
    }
    printf("\n%d",mi);
}

提前显示奇怪的内容

Answer 1

您在从零开始计数和先乘后加规则方面遇到了一些问题。 好吧，让我们先开始吧。 最常见的是，当你从 0 开始计数时，你会这样做：

for(i=0; i < count; i++)
    /* ... */;

如果你从 1 开始计数，你会这样做：

for(i=1; i <= count; i++)
    /* ... */

如果你混合这些，你会得到相同的结果，但这可能会让你自己和其他阅读的人感到困惑。代码：

for(i=0; i <= count-1; i++) /* same, but better don't do this */
    /* ... */;

在计算平均值的代码中，有两个错误。 首先，由于数学原因，您应该使用括号：

avg = total / (size-1); /* there is still one bug */

其次，您有 size 元素。 因此，您必须除以 size，而不是除以 size-1：

avg = total / size; /* right ! */

Answer 2

这行可能是问题所在：

avg = total /size-1;

您可能想要改为：

avg = total / size;

另外，您的 max() 和 min() 函数有一个像这样的循环：

for (int j=0;j<size-1;j++)

这可能会少检查一个数量超出您的预期。

上述类型的错误通常称为“栅栏错误”。 这个名字涉及到以下问题：如果你想建一个100m长的栅栏，并且每隔1m需要一个栅栏柱，那么你需要多少个栅栏柱？ 答案不是 100，而是 101。此类错误的另一个名称是“相差一个错误”。

Answer 3

由于 scanf() 字符串中的“\n”，它会跳过第一个输入数字。 要修复此问题，请从 scanf 字符串中删除 '\n'，如下所示：

scanf("%d", &a[i]);