C++ 从unsigned char数组直接构造GUID后8字节为0问题

• 核心错误是对GUID结构体的成员定义理解错误：Windows平台的GUID结构体定义如下：

typedef struct _GUID {
  unsigned long  Data1;
  unsigned short Data2;
  unsigned short Data3;
  unsigned char  Data4[8];
} GUID;

你初始化时第四项仅取了data+8位置的1个字节赋值给Data4[0]，剩下7个Data4数组元素未初始化，自然全部为0，这就是输出GUID最后12位全0的直接原因。

• 隐性问题1：直接对uint8_t数组执行非对齐的多字节指针强制转换，在ARM等不支持非对齐访问的架构上会触发未定义行为，轻则程序崩溃，重则出现随机数据错误。
• 隐性问题2：你的初始化逻辑默认系统为小端字节序，代码可移植性差，若在大端平台运行会生成完全错误的GUID。

推荐使用手动逐字段赋值的写法，既避免对齐问题，也保证字节序和原有逻辑完全匹配：

GUID CreateGuid(const uint8_t* data)
{
    GUID guid;
    // 处理4字节的Data1，匹配原有逻辑的字节序
    guid.Data1 = ((uint32_t)data[3] << 24) | ((uint32_t)data[2] << 16) | ((uint32_t)data[1] << 8) | data[0];
    // 处理2字节的Data2
    guid.Data2 = ((uint16_t)data[5] << 8) | data[4];
    // 处理2字节的Data3
    guid.Data3 = ((uint16_t)data[7] << 8) | data[6];
    // 处理8字节的Data4，按顺序赋值
    guid.Data4[0] = data[8];
    guid.Data4[1] = data[9];
    guid.Data4[2] = data[10];
    guid.Data4[3] = data[11];
    guid.Data4[4] = data[12];
    guid.Data4[5] = data[13];
    guid.Data4[6] = data[14];
    guid.Data4[7] = data[15];
    return guid;
}

如果你确定代码仅在x86/x64小端平台运行，也可以用memcpy规避非对齐访问问题，性能和直接转换相当但更安全：

GUID CreateGuid(const uint8_t* data)
{
    GUID guid;
    memcpy(&guid.Data1, data, 4);
    memcpy(&guid.Data2, data + 4, 2);
    memcpy(&guid.Data3, data + 6, 2);
    memcpy(guid.Data4, data + 8, 8);
    return guid;
}

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