C++11中按16位字读取二进制文件的正确方法是什么？

你的这段代码大部分是符合C++11规范的，不过有个小细节可以调整，同时我们可以借助C11引入的新特性，让代码更安全、更贴合现代C的风格。

先看现有代码的合规性 #

• 头文件里的<stdint.h>是C标准头文件，C11更推荐使用对应的C版本头文件<cstdint>——虽然<stdint.h>在C11中也能兼容，但<cstdint>严格遵循C的命名空间规范（所有相关类型都在std命名空间下，实际使用中两者差异不大，但<cstdint>更“正统”）。
• 剩下的部分：std::ifstream的初始化方式、read()方法的调用、用static_cast<uint8_t>避免char作为有符号类型时的符号扩展问题，这些都是完全符合C++11标准的，尤其是符号扩展的处理非常到位，避免了移位时出现意想不到的错误。

更优的C++11实现方式 #

这里给你几个实用的优化方向：

1. 用std::array替代原始字符数组 #

原始数组char c[2]缺乏边界检查等安全特性，C++11引入的std::array既保留了数组的性能，又提供了更安全的接口（比如at()方法会做边界检查），代码修改后如下：

#include <iostream>
#include <fstream>
#include <cstdint>
#include <array>

int main() {
    std::ifstream file("./tetris.rom", std::ios::in | std::ios::binary);
    std::array<char, 2> c;
    while (file.read(c.data(), c.size())) {
        uint16_t word = (static_cast<uint8_t>(c[0]) << 8) | static_cast<uint8_t>(c[1]);
        std::cout << "word\t" << std::hex << word << std::endl;
    }
}

2. 显式处理字节序（让代码更健壮） #

如果你的ROM文件是固定字节序（比如常见的大端格式），可以用C++11的constexpr封装一个字节序转换函数，让转换在编译期完成，既高效又清晰：

#include <iostream>
#include <fstream>
#include <cstdint>
#include <array>

constexpr uint16_t big_endian_to_host(uint16_t big_endian_val) {
    #if __BYTE_ORDER__ == __ORDER_LITTLE_ENDIAN__
        return ((big_endian_val >> 8) & 0xFF) | ((big_endian_val & 0xFF) << 8);
    #else
        return big_endian_val;
    #endif
}

int main() {
    std::ifstream file("./tetris.rom", std::ios::in | std::ios::binary);
    std::array<char, 2> c;
    while (file.read(c.data(), c.size())) {
        uint16_t raw_word = *reinterpret_cast<uint16_t*>(c.data());
        uint16_t word = big_endian_to_host(raw_word);
        std::cout << "word\t" << std::hex << word << std::endl;
    }
}

注：这里用reinterpret_cast是合法的，因为C++标准允许char*与其他类型的指针互相转换；如果担心严格别名问题，保留你原来的移位拼接方式也是完全没问题的。

3. 用auto简化变量声明 #

C++11的auto关键字可以让编译器自动推导变量类型，让代码更简洁，比如把uint16_t word = ...改成：

auto word = (static_cast<uint8_t>(c[0]) << 8) | static_cast<uint8_t>(c[1]);

编译器会自动推导word的类型为uint16_t，不影响代码的可读性，还能减少手动写类型的麻烦。

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