c - 使用函数从文件中查找最大值

Question

我对编程还很陌生，所以请多多包涵。

我正在尝试创建一些代码来读取包含 3 个数字的文本文件。我想使用创建的函数来查找最大数量。编译时我没有收到任何错误，但是当我运行代码时程序崩溃(没有收到任何消息或任何消息，只是 file.exe 已停止工作)。

如果您能帮助我解决这个问题，我将不胜感激。我也想避免使用数组。

#include <stdio.h>
#include <stdlib.h>

int max(int a,int b,int c);


int main()
{

FILE *fpointer;
int a, b, c;
int maxNumber = max(a,b,c);

fpointer = fopen("marks.txt","r");


while(fscanf(fpointer,"%d %d %d",a,b,c)!=EOF) {
    printf("%d",max(a,b,c));

}


fclose(fpointer);

return 0;
}

int max(int a,int b,int c){
if((a>b)&&(a>c))
    return a;

 if((b>a)&&(b>c))
    return b;

 if((c>a)&&(c>b))
    return c;

}

Answer 1

I am fairly new to programming so bear with me.

好的，我们会的，但无论我们多么努力，我们都无法修复您调用的未定义行为:

int maxNumber = max(a,b,c);

a、b 和 c 的值在您调用 max 时尚未初始化。这会调用未定义的行为。 (试图访问未初始化对象的值)。

其次，同样容易导致未定义行为的是未能验证fopen 是否成功，以及未能验证fscanf 是否成功。测试 fscanf (...) != EOF 不会告诉您任何关于有效转换是否实际发生的信息。 fscanf 的返回 是发生的成功转换次数——基于格式中存在的转换说明符 的数量字符串(例如"%d %d %d" 包含3 转换说明符)。因此，要验证 a、b 和 c 都包含值，您必须比较 fscanf (...) == 3。

将这些部分放在一起，您可以执行类似于以下操作的操作:

#include <stdio.h>

int max (int a, int b, int c);

int main (int argc, char **argv) {

    int a, b, c, n = 0;
    FILE *fp = argc > 1 ? fopen (argv[1], "r") : stdin;

    if (!fp) {  /* validate file open for reading */
        fprintf (stderr, "error: file open failed '%s'.\n", argv[1]);
        return 1;
    }

    while (fscanf (fp, "%d %d %d", &a, &b, &c) == 3)
        printf ("line[%2d] : %d\n", n++, max (a, b, c));

    if (fp != stdin) fclose (fp);   /* close file if not stdin */

    return 0;
}

int max (int a, int b, int c) 
{
    int x = a > b ? a : b,
        y = a > c ? a : c;

    return x > y ? x : y;
}

示例输入

$ cat int3x20.txt
21 61 78
94 7 87
74 1 86
79 80 50
35 8 96
17 82 42
83 40 61
78 71 88
62 20 51
58 2 11
32 23 73
42 18 80
61 92 14
79 3 26
30 70 67
26 88 49
1 3 89
62 81 93
50 75 13
33 33 47

示例使用/输出

$ ./bin/maxof3 <dat/int3x20.txt
line[ 0] : 78
line[ 1] : 94
line[ 2] : 86
line[ 3] : 80
line[ 4] : 96
line[ 5] : 82
line[ 6] : 83
line[ 7] : 88
line[ 8] : 62
line[ 9] : 58
line[10] : 73
line[11] : 80
line[12] : 92
line[13] : 79
line[14] : 70
line[15] : 88
line[16] : 89
line[17] : 93
line[18] : 75
line[19] : 47

检查一下，如果您还有其他问题，请告诉我。