C++读取空文件返回255而非预期0的问题排查与解决

• ifstream::get() 读取到文件末尾（EOF）时，会返回**EOF**宏，这个宏的标准值是 -1。
• 你用uint8_t（无符号8位整数）接收该返回值，-1被强制转换为无符号类型后，会变成二进制全1的8位值，也就是十进制的255，这就是出现该问题的核心原因。
• 代码还存在额外问题：每次调用read()都重复打开、关闭文件，效率极低；且错误使用了ifstream::app模式——这个模式是用于写入追加的，用它打开文件读取完全没必要，甚至可能引发意外行为。

1. 先判断EOF再处理返回值 #

EOF是int类型，必须先用int变量接收get()的返回值，这样才能同时容纳所有ASCII字符值和EOF标记。判断不是EOF后，再将其转换为uint8_t；如果是EOF则返回你预期的0。

2. 优化文件打开逻辑 #

如果是持续操作同一个文件，建议将ifstream作为类的成员变量，在构造函数中打开、析构函数中关闭，避免反复打开关闭文件的开销；如果必须每次调用read()时打开文件，也要使用默认的读取模式（ios::in，可省略），不要用app模式。

修复后的代码示例 #

单次打开版本（适合临时读取）

uint8_t FileDevice::read () {
    ifstream ifile(this->fileName);
    if (!ifile) {
        // 处理文件打开失败，直接返回0
        printf("0\n");
        return 0;
    }
    
    ifile.seekg(this->position);
    int ch = ifile.get(); // 用int接收，兼容EOF和所有字符
    this->position++;
    ifile.close();
    
    if (ch == EOF) {
        printf("0\n");
        return 0;
    } else {
        uint8_t character = static_cast<uint8_t>(ch);
        printf("%d\n", character);
        return character;
    }
}

持久化文件句柄版本（更高效）

// 类定义调整
class FileDevice {
private:
    std::string fileName;
    std::streampos position;
    std::ifstream ifile; // 作为成员变量保存文件句柄
public:
    // 构造函数中打开文件
    FileDevice(const std::string& fn) : fileName(fn), position(0) {
        ifile.open(fileName);
        if (!ifile) {
            // 打开失败可抛出异常或自行处理
            throw std::runtime_error("Failed to open file: " + fileName);
        }
    }
    
    // 析构函数中关闭文件
    ~FileDevice() {
        if (ifile.is_open()) {
            ifile.close();
        }
    }
    
    uint8_t read() {
        if (!ifile.is_open()) {
            printf("0\n");
            return 0;
        }
        
        ifile.seekg(position);
        int ch = ifile.get();
        position++;
        
        if (ch == EOF) {
            printf("0\n");
            return 0;
        } else {
            uint8_t character = static_cast<uint8_t>(ch);
            printf("%d\n", character);
            return character;
        }
    }
};

内容的提问来源于stack exchange，提问作者GabrijelOkorn