使用RapidJSON的AddMember生成JSON时出现垃圾值问题求助

核心问题 #

你的代码出现垃圾值的原因主要有两点：

• 重复键触发未定义行为：外层循环5次，每次都用同一个str作为键添加到json对象中。RapidJSON的Object类型不允许重复键，重复添加会导致内存结构损坏，之前的内容被覆盖或变成垃圾值。
• 分配器不匹配：根对象json初始化时没有绑定分配器，后续添加成员时使用d.GetAllocator()，导致内存管理混乱。RapidJSON要求所有关联的Value必须使用同一个分配器，否则会出现内存访问错误。

另外，代码中存在冗余的Move操作，虽然不直接导致问题，但会增加不必要的复杂度。

修复步骤 #

1. 为根对象绑定分配器：初始化json时指定分配器，确保所有Value操作使用同一个分配器。
2. 避免重复键：外层循环中每次迭代使用不同的键（比如把循环变量i拼接到str中），确保每个成员键唯一。
3. 简化Move操作：直接对局部Value调用Move()，无需重新构造Value再Move。

修改后的代码 #

std::string result;
// 初始化根对象时绑定分配器
RapidJSON::Value json(d.GetAllocator());
json.SetObject();

for(int i = 0; i < 5; ++i)
{
    // 生成唯一键，避免重复
    std::string uniqueStr = str + std::to_string(i);
    RapidJSON::Value data(d.GetAllocator());
    data.SetObject();

    for(auto it = HashMap.begin(); it != HashMap.end(); ++it)
    {
        RapidJSON::Value arrObj(d.GetAllocator());
        arrObj.SetObject();

        for(auto it2 = it->second.begin(); it2 != it->second.end(); ++it2)
        {
            arrObj.AddMember(
                RapidJSON::StringRef(it2->first.c_str()),
                RapidJSON::StringRef(it2->second.c_str()),
                d.GetAllocator()
            );
        }
        // 直接Move arrObj，无需重新构造Value
        data.AddMember(
            RapidJSON::StringRef(it->first.c_str()),
            arrObj.Move(),
            d.GetAllocator()
        );
    }
    // 使用唯一键添加成员，直接Move data
    json.AddMember(
        RapidJSON::StringRef(uniqueStr.c_str()),
        data.Move(),
        d.GetAllocator()
    );
}

cleanup:
    retJson.AddToJson(RapidJSON::StringRef("STATUS"), json);
    result = retJson.ToString();

额外注意事项 #

• 确保str、HashMap中的字符串在json序列化完成前始终有效，因为StringRef是对外部字符串的引用，不会拷贝内容。如果字符串生命周期短于json，需要改用RapidJSON::String来拷贝内容。
• 检查d（分配器实例）的生命周期，确保在所有Value操作完成前d不会被销毁。

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