如何修复CppCheck报的char*与double*转换的可移植性警告？

CppCheck报这个警告并非小题大做，直接把char*强转成double*存在两个核心的可移植性风险：

• 内存对齐问题：double类型通常要求4或8字节的内存对齐，而USB驱动返回的char*内存块可能不满足这个对齐要求。在ARM、PowerPC这类对内存对齐严格的架构上，直接访问未对齐的double会触发硬件异常。
• 类型别名违规：C++标准明确规定，除非是用char/unsigned char类型访问其他对象，否则用不同类型指针访问对象属于未定义行为。虽然当前平台可能兼容这种操作，但标准不保证所有编译器/架构都支持。

下面是两种规范的C++实现方式，彻底解决这个问题：

方法一：用内存拷贝（最通用，兼容所有C++版本） #

避免直接指针转换，通过std::memcpy将字节数据拷贝到double变量中，完全符合C++标准，没有对齐和类型别名的隐患：

#include <cstring>
#include <iostream>

void printAsDouble( char* myData, size_t size )
{
    if ( size % sizeof(double) == 0 )
    {
        const size_t count = size / sizeof(double);
        for (size_t pos = 0; pos != count; ++pos)
        {
            double value;
            // 从char数组的对应位置拷贝sizeof(double)字节到value
            std::memcpy(&value, myData + pos * sizeof(double), sizeof(double));
            std::cout << "Value #" << (pos+1) << " is " << value << std::endl;
        }
    }
}

方法二：用std::bit_cast（C++20及以上版本可用） #

如果你的项目已经升级到C++20，std::bit_cast是标准提供的按位类型转换工具，会在编译期检查两个类型的大小是否一致，比手动拷贝更简洁：

#include <bit>
#include <iostream>
#include <array>

void printAsDouble( char* myData, size_t size )
{
    if ( size % sizeof(double) == 0 )
    {
        const size_t count = size / sizeof(double);
        for (size_t pos = 0; pos != count; ++pos)
        {
            // 定位到当前double对应的字节块，转换为std::array<char, sizeof(double)>
            auto* byte_block = reinterpret_cast<std::array<char, sizeof(double)>*>(myData + pos * sizeof(double));
            // 按位转换为double
            double value = std::bit_cast<double>(*byte_block);
            std::cout << "Value #" << (pos+1) << " is " << value << std::endl;
        }
    }
}

要不要直接屏蔽警告？ #

绝对不建议。虽然当前所有支持平台运行正常，但一旦未来要移植到对齐严格的架构，或者编译器更新后严格执行C++标准，这段代码很可能会崩溃或出现诡异的未定义行为。上面两种方法都能彻底消除警告，同时保证代码的可移植性和合规性。

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