未找到文件时将文件名作为参数传递导致分段错误

Question

我想通过将文件名作为命令行参数然后将其传递给函数来打开文件。

打开文件时代码运行良好，但找不到文件时会抛出分段错误错误消息。当我将文件打开逻辑从 runFile 函数转移到 main 函数时，它也可以工作，但是当文件名作为参数传递给函数时会导致分段错误。

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

void runFile(char* fileName);

int main(int argc, char** argv)
{   
    if (argc != 2)
    {   
        printf("Usage: ./lexer [source]\n");
        exit(64);
    }
    else
    {   
        runFile(argv[1]);
    }
    return 0;
}

void runFile(char* file)
{
    FILE *fp;
    if (!filename)
    {
        printf("Error");
    }

    fp = fopen(file, "r");
    if ((fp = fopen(file, "r")) == NULL)
    {
        printf("File not opened!");
    }
    else
    {
        printf("File opened Successfully!\n");
    }



    fclose(fp);
}

Answer 1

像这样编写runFile() 函数：

void runFile(char* file) {
  FILE* fp = fopen(file, "r");

  if (fp == NULL) {
    printf("File not opened!\n");
    return;
  }

  // do something with the file
  printf("File opened!\n");

  fclose(fp);
}

您的代码的问题是即使fp 是NULL，您仍然调用fclose(fp)（因为它无法打开文件）； fclose(fp) 应该只被调用一次，并且只在 fp != NULL 时调用。