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

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

fscanf使用指针参数，您传递变量值。当函数尝试访问地址a（实际上是一个未初始化的变量）时，它会导致分段错误（您尝试访问无效的内存地址）并且程序崩溃并退出。

您应该将变量地址传递给指针参数（例如&a - 变量a的地址），因此它将访问有效的内存地址。

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

其他未定义的行为可能会被避免，比如初始化变量并正确检查返回值，因为@ DavidC.Rankin的答案详细描述。

Answer 2

我对编程很新，所以请耐心等待。

好的，我们会，但无论我们如何努力，我们都无法修复您调用的未定义行为：

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

仔细看看，如果您有其他问题，请告诉我。

Answer 3

所有需要让它工作的是在这一行添加一个& ...

while(fscanf(fpointer,"%d %d %d",&a,&b,&c)!=EOF) { ... }