从一个txt文件逐个字符地读取，并写入C中的另一个文件？

Question

（C、visual studio 2022环境）

我必须编写一个程序来读取 a.txt 文件（逐个字符）并将每个字符放在另一个.txt 文件中（在由 Visual Studio 创建的同一目录中）。 initial.txt 文件太长，所以在这个问题中我写了一小部分数据。

这是我的初始文件.txt：

3.456789 3.456789 3.5671234566 3.450934759435735738457 3.3298017238973289742398074238 3.456789 3.456789 3.5671234566 3.450934759435735738457 3.3298017238973289742398074238 3.456789 3.456789 3.5671234566 3.450934759435735738457 3.3298017238973289742398074238 3.456789 3.456789 3.5671234566 3.450934759435735738457 3.3298017238973289742398074238

这是代码：（我只使用了 main.c 因为它只是测试 C 输入输出的测试）：

代码应该做这些事情：

• 打开一个只读文件。 （由 f 指向）。
• （空指针检查。我检查了 f）。
• 我用名称 x 定义了数组。 （它是一个指向数组第 0 个元素的指针）。
• 我定义了 CHAR COUNT 因为我在第一个 while 循环（读取部分）中使用了它。
• （空指针检查。我检查了 x）。
• 阅读由这三个部分组成的无限循环组成：阅读、检查、使用。

阅读部分：我分配了计算 fgetc 的值 [fgetc() 返回值是一个字符]。 fgetc 采用 f stream 的字符。

检查部分：我检查了它是否等于 EOF 宏。 如果是这种情况，则在第一个 while 循环之外“中断”和 go。 {在写这个问题时，我正在考虑这个阶段可能出现的一个错误：如果这个检查条件成立，那么第二个数组就会被执行，但是在第二个 while 循环中我会遇到另一个错误，因为我没有阅读第一个文件完全}。

使用部分：如果 if 检查为假，则执行 calloc 并分配足够的 memory (i, sizeof(double)) 来存储使用 fgetc 读取的值。

（另一个 null 指针检查。我再次检查了 x，因为我已经分配了 x）。 {在写这个问题时，我在想是否必须检查 calloc，因为它不像 malloc。 malloc 用随机的东西填充分配的 memory，但 calloc 将 memory 初始化为零}。

我分配了值（x[i] = count）。

当然，我将 i 增加了 1。

• 在第一个循环之外，我打开了另一个文件（由 g 指向），但这个文件是一个只写文件。
• （空指针检查。我检查了 g）。
• 第二个无限while循环，但我使用了fputc（它在文件上写入一个字符）。
• （EOF检查。我检查了x[j]是否等于EOF宏。-在第二个while循环之外，我释放了分配的memory，并关闭了txt文件。（在return 0语句之前）。

总结一下我的问题：

• “在写这个问题的时候，我正在考虑这个阶段可能出现的一个错误：如果这个检查条件成立，那么第二个数组就会被执行，但是在第二个 while 循环中我会遇到另一个错误，因为我没有阅读第一个文件完全“你怎么看？

• "while writing this question, I'm thinking if calloc have to be checked or not, because it's not like malloc. malloc fills up allocated memory with random stuff, but calloc initializes the memory to zero". 你怎么看？

和一般问题：以这种方式编写这个程序是否合法，或者你认为有更明智的方式来完成同样的任务？ （注意：我知道我可以使用 fprintf() 和 fscanf()，但我想测试我编写程序的技能，该程序执行相同的任务但逐个字符（在长 txt 文件上））。

（我有两个错误和三个警告：

第一个错误：使用了未初始化的局部变量 f（为什么？我已经初始化它，因为它指向文件的开头）

第二个错误：使用了未初始化的局部变量 x（为什么？x 是一个指针，我已经检查它是否是 null 指针，所以我不必初始化它）

第一个警告：“function”：不兼容的类型（从“FILE *”到“const char *”。

其余的警告就像错误一样，调试器也将其复制到警告列表中。

   #define _CRT_SECURE_NO_WARNINGS
    #include <stdio.h>
    #include <stdlib.h>
    int main(void) {
    
        FILE* f = fopen(f, "r"); 
        if (f == NULL) {
            return NULL; 
        }
        int* x; char count = 0; 
        if (x == NULL) {
            return NULL; 
        }
        unsigned int i = 1; 
        while (1) {
            count = fgetc(f); 
            if (count == EOF) {
                break; 
            }
             
            int* x = calloc(i, sizeof(double)); 
            if (x == NULL) {
                return -3; 
            }
            x[i] = count;
            i++; 
        } 
        FILE* g = fopen("otherfile.txt", "w"); 
        if (g == NULL) {
            return -2; 
        }
        int j = 1; 
        while (1) {
            fputc(x[j], g); 
            if (x[j] == EOF) {
                break; 
            }
            j++; 
        }
    
    
        free(x); 
        fclose(f); 
        fclose(g); 
        return 0; 
    }
        

Answer 1

FILE* f = fopen(f, "r"); 

这是明显的错别字。 它应该是fopen("inputfile", "r") 。

int* x; 
if (x == NULL) { return NULL;  }

x未初始化。 这意味着它可以是NULL （零），也可以是任何其他随机值。 此时您无法对其进行测试。

while (1) {
    ...
    int* x = calloc(i, sizeof(double)); 
}

在这里，您声明了一个具有相同名称的不同指针。 第二个x是一个不同的变量，因为它位于不同的 scope 中（当您将变量放入{}时，scope 会发生变化）

第二个x分配 memory 然后它在循环的下一次迭代中丢失。 这会导致严重的 memory 泄漏。

sizeof(double)不适用。 如果您打算存储在char缓冲区中，首先您必须知道文件中有多少个字符。 在阅读文件之前不可能知道这一点。

为了解决这个问题，您可以通过文件一次 go 并计算字符数，或者您可以检查文件大小。

一旦分配了足够的 memory，就可以使用fread / fwrite读取整个文件。

int main(void)
{
    FILE* fin = fopen("input.txt", "r");
    if (!fin) { perror("input error"); return 0; }
    FILE* fout = fopen("output.txt", "w");
    if (!fout) { perror("fout error"); return 0; }

    fseek(fin, 0, SEEK_END); //go to the end of file
    size_t filesize = ftell(fin); //get file size
    fseek(fin, 0, SEEK_SET); //go back to the beginning 

    //allocate enough memory
    char* buffer = malloc(filesize * sizeof(char));

    //read one character at a time (or `fread` the whole file)
    size_t i = 0;
    while (1)
    {
        int c = fgetc(fin);
        if (c == EOF) break;

        //save to buffer
        buffer[i++] = (char)c;
    }

    //write one character at a time (or `fwrite` the whole file)
    for (i = 0; i < filesize; i++)
        fputc(buffer[i], fout);

    free(buffer);
    fclose(fin);
    fclose(fout);
    return 0;
}