忽略 fscanf 的返回值和分段错误

Question

我想知道如何在我的 C 代码上解决 Core dumped issue。

当我用：g++ -g MatSim.cpp -o MatSim -lstdc++ -O3 编译它时，我收到三个警告，这是一个（其他两个类似，仅通过字符串变量名区分）：

MatSim.cpp: In function ‘int main()’:
MatSim.cpp:200037:27: warning: ignoring return value of ‘int fscanf(FILE*, const char*, ...)’, declared with attribute warn_unused_result [-Wunused-result]
 fscanf(TM,"%255s",string2);

我的代码的主要方面以及编译器报告的相关部分：

#include <stdio.h>
#include <stdlib.h>
#include <malloc.h>
#include <iostream>
#include <fstream>
#include <string.h>

using namespace std;

int to_int(char string[256])
{
    if( strcmp(string,"0") == 0 )
    {
        return 0;
    }

...

    else if( strcmp(string,"50000") == 0 )
    {
        return 50000;
    }
    return -1;
}   

int main()
{
int a,b,div,value,k,i,j,tm,ler; 
    char string[256];
    char string1[256];
    char string2[256];

FILE *TM;
TM = fopen("TM","r");
if( (TM = fopen("TM","r")) == NULL)
{  
    printf("Can't open %s\n","TM");
    exit(1);
}
fscanf(TM,"%255s",string2);
tm = to_int(string2);
fclose(TM);

...

}

我已经尝试了here 中报告的建议，并试图了解here 中发布的内容。但是，我没有在我的代码中看到它的应用程序。

最后，当我运行exe文件时，它返回：

分段错误（核心转储）`。

Answer 1

在您的代码中，您 fopen()ing 该文件两次。只需摆脱

 TM = fopen("TM","r");

在if 语句之前。

也就是说，您应该检查fscanf() 的值以确保成功。否则，您最终会读取一个未初始化的数组 string2，它不是以空值结尾的，它反过来会调用 undefined behaviour。

请注意，几乎所有与string 相关的函数都需要一个以空字符结尾的字符数组。如果你的数组不是空终止的，就会有 UB。此外，初始化您的自动局部变量是一个很好的做法，以避免在后面的代码部分中出现可能的 UB。

Answer 2

您打开文件两次。

你只需要这个：

FILE *TM = fopen("TM","r");
if (TM == NULL)  { /* file was not opened */ }