使用jsoncpp删除Json::Value键值对时迭代器报错的解决方法

你遇到的迭代器错误，本质是遍历JSON对象键的过程中直接删除元素导致迭代器失效。jsoncpp的Value对象（对应JSON对象）底层基于类似std::map的结构实现，当你调用removeMember删除某个键时，指向该键的迭代器会直接失效。如果在删除后继续对这个失效的迭代器执行++操作，就会触发"cannot increment value-initialized map/set iterator"这类非法迭代的错误。

要解决这个问题，需要避免在遍历迭代器的同时直接修改容器结构，以下两种方式都能可靠解决：

方式1：预存下一个迭代器再删除 #

遍历过程中先保存下一个迭代器的位置，再执行删除操作，不依赖失效的迭代器：

#include <json/json.h>
#include <unordered_set>

using namespace std;

int main() {
    Json::Value jsonArray; // 假设这是你的目标JSON数组
    unordered_set<string> keepKeys = {"name", "job", "salary"};

    for (auto& obj : jsonArray) {
        if (!obj.isObject()) continue;

        auto it = obj.begin();
        while (it != obj.end()) {
            string key = it.key().asString();
            // 先保存下一个迭代器，再移动当前迭代器
            auto nextIt = it++;
            if (keepKeys.find(key) == keepKeys.end()) {
                obj.removeMember(key);
            }
        }
    }
    return 0;
}

方式2：先收集待删除键，再批量删除 #

先遍历一遍对象，把不需要保留的键存到临时集合中，再循环删除这些键，完全规避迭代器失效问题：

#include <json/json.h>
#include <unordered_set>
#include <vector>

using namespace std;

int main() {
    Json::Value jsonArray; // 你的原始JSON数组
    unordered_set<string> keepKeys = {"name", "job", "salary"};

    for (auto& obj : jsonArray) {
        if (!obj.isObject()) continue;

        vector<string> keysToRemove;
        // 先收集所有要删除的键
        for (const auto& member : obj.getMemberNames()) {
            if (keepKeys.find(member) == keepKeys.end()) {
                keysToRemove.push_back(member);
            }
        }
        // 批量删除键值对
        for (const auto& key : keysToRemove) {
            obj.removeMember(key);
        }
    }
    return 0;
}

如果要保留的键数量不多，直接创建新的JSON对象，只复制需要的键值对会更简洁安全，完全不会涉及迭代器失效问题，代码可读性也更高：

#include <json/json.h>
#include <unordered_set>

using namespace std;

int main() {
    Json::Value jsonArray; // 原始JSON数组
    Json::Value newJsonArray; // 处理后的新数组
    unordered_set<string> keepKeys = {"name", "job", "salary"};

    for (const auto& obj : jsonArray) {
        if (!obj.isObject()) {
            newJsonArray.append(obj); // 非对象元素直接保留
            continue;
        }

        Json::Value newObj;
        for (const auto& key : keepKeys) {
            if (obj.isMember(key)) {
                newObj[key] = obj[key];
            }
        }
        newJsonArray.append(newObj);
    }

    // 后续直接使用newJsonArray即可
    return 0;
}

这种方式逻辑清晰，还能保留原始数据（如果有需要的话），完全规避了容器修改带来的风险。

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