std::size_t最大值转std::streampos的问题及解决方案咨询

问题场景 #

你的代码尝试通过tellg()获取文件大小（返回std::streampos），并判断其是否超过size_t的最大值，但直接将std::numeric_limits<size_t>::max()赋值给std::streamoff时，会因为有符号类型溢出变成-1，导致错误触发unsupportedFileSize判断。代码示例：

std::ifstream file(filePath, std::ios::in | std::ios::binary | std::ios::ate);
if (!file.is_open() || file.bad())
    return StatusCode::fileOpenError;

std::streampos fileSize = file.tellg();
constexpr std::streamoff maxSupportedFileSize = std::numeric_limits<std::size_t>::max();
if (fileSize > maxSupportedFileSize)
    return StatusCode::unsupportedFileSize;

解决方案 #

核心思路是避免直接将无符号的size_t最大值转换为有符号的streamoff，而是先验证文件位置的有效性，再分场景安全判断：

std::ifstream file(filePath, std::ios::in | std::ios::binary | std::ios::ate);
if (!file.is_open() || file.bad())
    return StatusCode::fileOpenError;

std::streampos fileSizePos = file.tellg();
// 先检查tellg()调用是否失败
if (fileSizePos == std::streampos(-1))
    return StatusCode::fileSizeError;

std::streamoff fileSizeOff = fileSizePos;
// 确保文件位置为非负
if (fileSizeOff < 0)
    return StatusCode::fileSizeError;

constexpr std::size_t maxSupportedSize = std::numeric_limits<std::size_t>::max();
// 分情况判断是否超过size_t最大值
if (sizeof(std::streamoff) > sizeof(std::size_t)) {
    // 当streamoff位数大于size_t时，直接比较数值
    if (fileSizeOff > static_cast<std::streamoff>(maxSupportedSize)) {
        return StatusCode::unsupportedFileSize;
    }
}
// 当streamoff位数小于等于size_t时，其正数最大值必然小于size_t最大值，无需额外判断

// 安全转换为size_t
std::size_t fileSize = static_cast<std::size_t>(fileSizeOff);

如果项目支持C++20，可使用更简洁的std::cmp_greater替代手动位数判断，它会自动处理类型转换的安全边界：

if (std::cmp_greater(fileSizeOff, maxSupportedSize)) {
    return StatusCode::unsupportedFileSize;
}

是否属于设计缺陷？ #

不算设计缺陷。std::streampos和std::streamoff的设计目标是表示文件位置，需要支持错误状态（用负数表示），因此采用有符号类型是合理的；而size_t是专为表示内存对象大小设计的无符号类型，两者应用场景不同。标准库没有强制它们的类型兼容，是因为开发者需要根据具体场景处理类型转换的安全边界。

内容的提问来源于stack exchange，提问作者no one special