无法通过 C 中的 fwrite 和 fread 复制文本文件 [关闭]

Question

代码如下：

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

int main() {
    FILE *f, *fp;
    char buff[512];
    int buff_size;
    int bytes;

    fp = fopen("File.txt", "rb");

    if (fp == NULL) {
        printf("Cannot Open Source File!\n");
        exit(1);
    }

    f = fopen("append.txt", "ab+");

    if (f == NULL) {
        printf("Cannot Open Target File!\n");
        fclose(fp);
        exit(1);
    }

    buff_size = sizeof(buff);

    while (bytes = fread(&buff, buff_size, 1, fp) > 0) {
        if (bytes > 0)
            fwrite(&buff, buff_size, 1, f);
        else
            break;

        printf("Appending...\n\n");
    }

    rewind(f);

    while (bytes = fread(&buff, buff_size, 1, f) > 0)
        if (bytes > 0)
            printf("%s", buff);


    fclose(fp);
    fclose(f);
}

所以，它碰巧没有输出任何东西，当我检查文件“append.txt”时，它也不包含任何东西。 请注意，源文件“File.txt”不为空。 谁能告诉我它有什么问题？

编辑：

我通过将buff_size 替换为strlen(buff) 解决了这个问题，如下所示： bytes = fread(&buff, strlen(buff), 1, f) > 0 和 fwrite() 和第二个 fread() 相同。

有人能解释一下为什么会这样吗？

Answer 1

char buff[512];
int buff_size;

// [...]

bytes = fread(&buff, buff_size, 1, fp)

这将尝试读取一个 512 字节的块。返回值是读取的块数，因此不会是字节。但撇开这一点不谈，如果您的文件短于 512 字节，这将不会读取 任何内容。

你想要的是读取 512 次 1 个字节，然后你会得到字节数，所以交换 buff_size 和 1 的位置。

---

旁注：

• 如果您正确地检查循环条件，例如：

  while ((bytes = fread(buff, 1, buff_size, fp)) > 0 )
  

  对if (bytes > 0) 的额外检查是多余的。

• 写入时，你只想写入你实际读取的字节数：

  fwrite(buff, 1, bytes, f);
  

• 对于尺寸，请始终使用size_t -- int 很可能是错误的：

  size_t buff_size;
  size_t bytes;
  

• 用printf("%s", ) 打印buff 是未定义的行为，因为您没有在fread() 读取的数据之后添加'\0' 字节。 C 字符串必须以'\0' 结尾。当fread() 读取的数据意外不包含'\0' 时，printf() 将读取和使用未初始化的数据，甚至可能读取超出buff 的范围。

Answer 2

Felix Palmen 在您的代码中列出了一些问题，您的修复完全错误，因为 buff 甚至没有空终止符。

这是一个更好的版本：

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

int main(void) {
    FILE *f, *fp;
    char buff[512];
    size_t bytes;

    fp = fopen("File.txt", "rb");
    if (fp == NULL) {
        fprintf(stderr, "Cannot open source file!\n");
        exit(1);
    }

    f = fopen("append.txt", "ab+");
    if (f == NULL) {
        fprintf(stderr, "Cannot open target file!\n");
        fclose(fp);
        exit(1);
    }

    while ((bytes = fread(buff, 1, sizeof buff, fp)) != 0) {
        if (fwrite(buff, 1, bytes, 1, f) != bytes) {
            fprintf(stderr, "Error writing to the target file\n");
            break;
        }
        printf("Appending...\n\n");
    }

    rewind(f);

    while ((bytes = fread(buff, 1, sizeof buff, f)) != 0) {
        printf("%.*s", (int)bytes, buff);
    }
    fclose(fp);
    fclose(f);
    return 0;
}

Answer 3

bytes = fread(&buff, buff_size, 1, fp) > 0

看看C Precedence Chart。运算符> 高于=，因此fread 的返回值将与零进行比较，然后将比较结果存储在bytes 中。你打算写这个

(bytes = fread(&buff, buff_size, 1, fp)) > 0

Answer 4

在给定固定文件名的情况下，这是我对您的问题的解决方案。如果是我自己的代码，它会将文件名作为参数。

#include <stdio.h>

int main(void)
{
    const char src_file[] = "File.txt";
    FILE *fp = fopen(src_file, "rb");

    if (fp == NULL)
    {
        fprintf(stderr, "Cannot Open Source File '%s'!\n", src_file);
        return(1);
    }

    const char tgt_file[] = "append.txt";
    FILE *f = fopen(tgt_file, "ab+");

    if (f == NULL)
    {
        fprintf(stderr, "Cannot Open Target File '%s'!\n", tgt_file);
        fclose(fp);
        return(1);
    }

    char buff[512];
    int bytes;
    while ((bytes = fread(buff, sizeof(char), sizeof(buff), fp)) > 0)
    {
        fwrite(buff, sizeof(char), bytes, f);
        printf("Appending...\n\n");
    }

    rewind(f);

    while ((bytes = fread(buff, sizeof(char), sizeof(buff), f)) > 0)
        printf("%.*s", bytes, buff);

    fclose(fp);
    fclose(f);
    return 0;
}

有各种各样的修复，其中大部分都在沿线某处的 cmets 中阐明。

• 我将文件名放入数组中，以便在fopen() 和错误消息中都可以使用该名称，这些消息将打印到stderr，而不是stdout。这对通用代码中的其他用户很有帮助。如果文件名来自命令行参数，这很简单。

• 对fread() 的调用已修复，以便报告字节数，而不是 512 字节块的数量（将是 0 或 1）。这涉及将大小/计数参数的顺序反转为fread()。传递的是缓冲区而不是缓冲区的地址。

• 正确捕获读取的字节数。

• 读取的字节数用于控制fwrite()的大小。

• 读取的字节数用于控制printf()打印的字节数。

• 我不喜欢 C99 及更高版本中允许 main()（但只允许 main()）默认返回 0 的特殊规则。 AFAIAC，它是一个定义为返回int 的函数；它应该返回一个int。但是，也有人不同意。