C++序列化PNG结构体读写文件异常排查求助

问题根源 #

你遇到的ifs.good()返回false的情况，并不是真正的读取失败，而是文件读取到末尾时eofbit被置位——std::ifstream::good()要求eofbit、failbit、badbit全部为0才会返回true。此时数据已经完整读取，所以能正常重建图片，只是状态检查逻辑误判了正常的文件结束状态。

另外你的最小复现示例中还有一个潜在bug：stbi_write_png的 stride 参数错误，当前用的是dstImage.imageWidth * sizeof(int)，正确值应该是dstImage.imageWidth * dstImage.colorChannels，这会导致图片行字节数计算错误，可能引发数据错位。

修复方案 #

1. 修正文件读取的状态检查逻辑 #

把读取后的ifs.good()判断替换为检查是否发生非EOF的读取失败——只有failbit置位且不是因为到达文件末尾时，才判定为读取错误：

批量读取函数的修改

// 原错误代码
if (!ifs.good()) 
{
    std::cerr << "Something went wrong when reading " << key << " from the file!" << std::endl;
    break;
}

// 替换为
if (ifs.fail() && !ifs.eof())
{
    std::cerr << "Something went wrong when reading " << key << " from the file!" << std::endl;
    break;
}

最小复现示例的修改

// 原错误代码
if (!ifs.good())
{
    std::cerr << "Something went wrong when reading " << srcImageName << " from the file!" << std::endl;
    ifs.close();
    return 1; 
}

// 替换为
if (ifs.fail() && !ifs.eof())
{
    std::cerr << "Something went wrong when reading " << srcImageName << " from the file!" << std::endl;
    ifs.close();
    return 1; 
}

2. 修正stbi_write_png的stride参数 #

在最小复现示例的图片重建部分，修正行字节数的计算：

// 原错误代码
stbi_write_png(dstImageName.c_str(), dstImage.imageWidth, dstImage.imageHeight, dstImage.colorChannels, dstImage.pixels.c_str(), dstImage.imageWidth * sizeof(int));

// 替换为
stbi_write_png(dstImageName.c_str(), dstImage.imageWidth, dstImage.imageHeight, dstImage.colorChannels, dstImage.pixels.c_str(), dstImage.imageWidth * dstImage.colorChannels);

补充说明 #

• std::ifstream::good()的设计是判断流是否处于可继续读取的状态，当文件读取到末尾时，流无法继续读取，所以good()返回false，但此时之前的读取操作已经成功完成，数据是完整的。
• 如果需要更精准的校验，可以额外检查ifs.gcount()是否等于预期的读取长度，比如读取entryCount字节后，判断ifs.gcount() == entryCount，确认是否读取到了完整的数据。

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