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

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

好的，我们会，但无论我们怎么努力，我们都无法解决您调用的未定义行为：

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

a, b & c时，max的值尚未初始化。这会调用 Undefined Behavior 。（尝试访问未初始化对象的值）。

其次，也很容易导致未定义行为，无法验证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 2

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

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

&a

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

Answer 3

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

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

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

3 个答案: