需要帮助解决 C 中文件 i/o 的分段错误

Question

我读取了有 911 行的输入文件，并希望在对每一行进行一些更改后将这 911 行复制到输出文件中。

当我运行此文件时，我遇到了分段错误。我不知道为什么。谁能帮忙..

#include<stdio.h>
void main()
{
    int i;
    FILE *fin,*fop;char* str;
    fin=fopen("atk561011.txt","r");
    if(fin=NULL) printf("ip err");
    fop=fopen("svmip.txt","w");
    if(fop=NULL) printf("op err");
    for(i=1;i<=911;i++)
    {
        fgets(str,150,fin);
        if((i>300&&i<=360)||(i>600&&i<=660)) 
            str[7]='1';
        else 
            str[7]='0';
        fputs(str+7,fop);
        putc('\n',fop);
    }
    fclose(fin);
    fclose(fop);
}

Answer 1

首先，这是错误的：

if(fin=NULL)

应该是：（

if (fin == NULL)

当然，fop 也是如此）。如果您没有成功打开文件 - 不要只是打印错误，退出，因为您要读取什么？请记住，printf 的输出是缓冲的，如果出现段错误，您根本不会看到它，即使它在故障之前运行。

另一件事：您不为 str 分配内存，而是使用 fgets 写入它。

另一件事：从文件中读取预定义数量的行可能是一个坏主意。最好从输入中读取，直到文件结尾，或者直到读取了所需的行数。

Answer 2

您没有为指针 str 分配任何空间。

将其更改为 char str[/*Max Length 您期望 150？ */] 或分配缓冲区。

与此同时，您的代码正在遍历内存 - 因此出现分段错误。

Answer 3

这是应该执行此操作的更正代码。

#include
#define MAX_BUF   150
void main()
{
    int i;
    FILE *fin,*fop;char* str;
    str = malloc((MAX_BUF * sizeof(char)) + 1);
    if (str == NULL){
       printf("Out of memory\n");
       exit(-1);
    }
    fin=fopen("atk561011.txt","r");
    if(fin == NULL){
        printf("ip err");
        exit(-2);
    }
    fop=fopen("svmip.txt","w");
    if(fop == NULL){
        printf("op err");
        exit(-3);
    }
    for(i=1;i<=911;i++)
    {
        fgets(str,150,fin);
        if((i>300&&i<=360)||(i>600&&i<=660)) 
            str[7]='1';
        else 
            str[7]='0';
        fputs(str+7,fop); 
        // What is that for? should it be
        // fputs(str, fop); ????? 
        // since you're outputting the 7'th character (1/0)?
        putc('\n',fop);
    }
    fclose(fin);
    fclose(fop);
    if (str != NULL) free(str);
}

我添加了逻辑检查以确保文件是否存在，以继续处理。如果不这样做将会导致代码崩溃。因为在原始代码中，如果输入失败或输出失败，您将打印“ip err”，但继续进入 for 循环，在这种情况下，这将导致执行失败，如下所示失败时它仍然会尝试从不存在的文件句柄中读取。

编辑：请参阅代码中的上述注释。您是否尝试根据 300-360 和 600-660 范围内的 i 条件值将 1/0 输出到输出文件中。你能澄清一下吗？ 希望这会有所

fputs(str[7], fop);

帮助，
此致，
汤姆.

Answer 4

fin=NULL 和 fop=NULL 都应该使用“等于”运算符。您将 fin 和 fop 设置为 NULL，而不是检查错误的返回值。