读取文件时遇到EOF不正确

我不知道出了什么问题

编码不是防御性的，因为它不会寻找意外事件。

要了解出了什么问题，请改进代码的错误检测。

换句话说，不要相信用户输入——这是邪恶的。

---

为什么只读取 3 个字节？

下面从文件中读取最多 3 个字节，将一行的其余部分留给以后读取。

char line[4]={0x0};
fgets(line,sizeof(line),plaintext);

它不会只从文件中读取 3 个字节并抛弃该行的其余部分。

检查 fgets（） 的返回值

// fgets(line,sizeof(line),plaintext);
if (fgets(line,sizeof(line),plaintext) == NULL) {
  TBD_alert();
}

使用 int

int fgetc()可以返回 257 个不同的值。储蓄会失去一些东西。char

// char ch;
int ch;

// char sh;
int sh;

缺乏错误检查

如果返回 0（无转换）怎么办？sscanf(str,"%02x ",&x1)

如果 4 太小怎么办？

如果存在额外的空格 （space， ， ...），则 4 太小了。'\r'

char line[4]={0x0};
fgets(line,sizeof(line),plaintext);

相反，读取一整行，允许至少 2 倍大小的缓冲区超过预期大小，并容忍额外的空格或最后一行的缺失。给定一行 ，使用缓冲区大小 （3*3 + 1）*2。fgets()'\n'"c1 c1 c1\n"

我怀疑有很多潜伏在周围。'\r'

避免UB。使用匹配说明符

"%x"匹配 ，而不是 。unsignedint

// int x1=0x0;
unsigned x1 = 0x0;  
...
 // '0' is useless, '2' is not needed.  Trailing space is useless
// sscanf(str,"%02x ",&x1);  
sscanf(str,"%x", &x1);    // Check return value - not shown

或直接保存

sscanf(str, "%hhx", &pt[i]);

每行 3 个值之外还有什么？

读取一行 3 个十六进制值并检测大量错误的示例代码。

#define EXPECTED_LINE_SIZE (3*3 + 1 /* for the \0 */)

char line[EXPECTED_LINE_SIZE * 2];
if (fgets(line, sizeof line, plaintext) == NULL) {
  return failure;
}
int n = 0;
unsigned val[3];
sscanf(line "%2x %2x %2x %n", &val[0], &val[1], &val[2], &n);
if (n == 0 || line[n] != '\0') {
  return failure;
}

检查 fopen（） 是否失败

FILE *fp=fopen("rgb_values.txt","r");
FILE *fo=fopen("out.txt","w");

if (fp == NULL || fo == NULL) {
  TBD_Error_out();  // Add your code here
}

如果您只想从一个文件复制到另一个文件，请以二进制模式打开，并在循环中打开一个 4KiB 的缓冲区，然后将其复制到另一个文件。方便快捷：freadfwrite

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

int main(void)
{
    FILE *in  = fopen(..., "rb");
    FILE *out = fopen(..., "wb");

    if (!in || !out)
        return EXIT_FAILURE;

    char buffer[4096];
    size_t nread;
    while ((nread = fread(buffer, 1, sizeof buffer, in)) > 0)
        fwrite(buffer, 1, nread, out);
}

否则，如果您需要将其作为字符串读取，请在循环中使用：fgetsfputs

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

int main(void)
{
    FILE *in  = fopen(..., "r");
    FILE *out = fopen(..., "w");

    if (!in || !out)
        return EXIT_FAILURE;

    char line[256];
    while (fgets(line, sizeof line, in) != NULL)
        fputs(line, out);
}

或者正如我所提到的，要读取一整行，解析出所有三个值，传回调用函数（连同文件读取成功状态），并同时读取所有三个值：fgetssscanffprintf

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

int read_values(FILE *in, int *values)
{
    char line[256];
    if (fgets(line, sizeof line, in) == NULL)
        return 0;  // Failure to read

    if (sscanf(line("%x %x %x", &values[0], &values[1], &values[2]) != 3)
        return 0;  // Failure to parse

    return 1;  // Success
}

void write_values(FILE *out, int *values)
{
    fprintf(out, "%02x %02x %02x\n", values[0], values[1], values[2]);
}

int main(void)
{
    FILE *in  = fopen(..., "r");
    FILE *out = fopen(..., "w");

    if (!in || !out)
        return EXIT_FAILURE;

    int values[3];
    while (read_values(in, values))
        write_values(out, values);
}

请注意，我不会关闭文件。如果你的程序在此之后做了更多的事情，那么它们真的应该被关闭。如果程序刚刚退出，那么系统将为我们关闭它们。