在C中逐字符读取文件

Question

我正在用 C 编写一个 BF 解释器，但在读取文件时遇到了问题。我曾经使用scanf 来读取第一个字符串，但是你的BF 代码中不能有空格或cmets。

现在这就是我所拥有的。

char *readFile(char *fileName)
{
  FILE *file;
  char *code = malloc(1000 * sizeof(char));
  file = fopen(fileName, "r");
  do 
  {
    *code++ = (char)fgetc(file);

  } while(*code != EOF);
  return code;
}

我知道问题出在我如何将文件中的下一个字符分配给代码指针，但我只是不确定那是什么。
我缺乏指针知识，这是本练习的重点。 解释器工作正常，全部使用指针，我只是在读取文件时遇到问题。

（稍后我将实现仅将+-><[]., 读取到文件中，尽管如果有人有好的方法可以做到这一点，如果你能告诉我会很棒！）

Answer 1

您的代码有很多问题：

char *readFile(char *fileName)
{
    FILE *file;
    char *code = malloc(1000 * sizeof(char));
    file = fopen(fileName, "r");
    do 
    {
      *code++ = (char)fgetc(file);

    } while(*code != EOF);
    return code;
}

1. 如果文件大于 1000 字节怎么办？
2. 每次读取字符时都会增加code，并且将code返回给调用者（即使它不再指向内存块的第一个字节，因为它是由malloc返回的)。
3. 您将fgetc(file) 的结果转换为char。在将结果转换为 char 之前，您需要检查 EOF。

维护malloc返回的原始指针很重要，以便以后可以释放它。如果我们忽略文件大小，我们仍然可以通过以下方式实现：

char *readFile(char *fileName)
{
    FILE *file = fopen(fileName, "r");
    char *code;
    size_t n = 0;
    int c;

    if (file == NULL)
        return NULL; //could not open file

    code = malloc(1000);

    while ((c = fgetc(file)) != EOF)
    {
        code[n++] = (char) c;
    }

    // don't forget to terminate with the null character
    code[n] = '\0';        

    return code;
}

有多种系统调用可以为您提供文件的大小；一个常见的是stat。

Answer 2

从@dreamlax 扩展上述代码

char *readFile(char *fileName) {
    FILE *file = fopen(fileName, "r");
    char *code;
    size_t n = 0;
    int c;

    if (file == NULL) return NULL; //could not open file
    fseek(file, 0, SEEK_END);
    long f_size = ftell(file);
    fseek(file, 0, SEEK_SET);
    code = malloc(f_size);

    while ((c = fgetc(file)) != EOF) {
        code[n++] = (char)c;
    }

    code[n] = '\0';        

    return code;
}

这会给你文件的长度，然后逐个字符地读取它。

Answer 3

这是一种简单的方法，可以忽略除有效的脑残字符之外的所有内容：

#define BF_VALID "+-><[].,"

if (strchr(BF_VALID, c))
    code[n++] = c;

Answer 4

每次调用函数时，文件都被打开而不是关闭

Answer 5

我认为最重要的问题是你在读入内容时递增code，然后返回code 的最终值，即你将返回一个指向end 的字符串。您可能想在循环之前复制code，然后将其返回。

此外，C 字符串需要以空值结尾。您需要确保将'\0' 直接放在您读入的最后一个字符之后。

注意：您可以只使用fgets() 一次性获得整行。

Answer 6

两者中的任何一个都可以解决问题 -

char *readFile(char *fileName)
{
  FILE *file;
  char *code = malloc(1000 * sizeof(char));
  char *p = code;
  file = fopen(fileName, "r");
  do 
  {
    *p++ = (char)fgetc(file);
  } while(*p != EOF);
  *p = '\0';
  return code;
}

char *readFile(char *fileName)
{
  FILE *file;
  int i = 0;
  char *code = malloc(1000 * sizeof(char));
  file = fopen(fileName, "r");
  do 
  {
    code[i++] = (char)fgetc(file);
  } while(code[i-1] != EOF);
  code[i] = '\0'
  return code;
}

就像其他发帖人指出的那样，您需要确保文件大小不超过 1000 个字符。另外，请记住在使用完毕后释放内存。

Answer 7

这里的问题是双重的

• a) 在检查读入的值之前增加指针，并且
• b) 你忽略了 fgetc() 返回一个 int 而不是 char 的事实。

第一个很容易解决：

char *orig = code; // the beginning of the array
// ...
do {
  *code = fgetc(file);
} while(*code++ != EOF);
*code = '\0'; // nul-terminate the string
return orig; // don't return a pointer to the end

第二个问题更微妙 -fgetc 返回一个 int 以便可以将 EOF 值与任何可能的 char 值区分开来。解决此问题时使用临时 int 进行 EOF 检查，并且可能使用常规 while 循环而不是 do / while。