C++调用push_back报错：如何修改std::set中对象的vector成员？

你的问题核心在于std::set的迭代器返回的是常量引用——这是因为set是有序容器，元素的排序依赖于你定义的operator<（这里是id），如果允许随意修改元素，可能会破坏容器内部的有序结构（比如不小心改动了id）。不过你要修改的是y，它不影响排序逻辑，我们可以用几种方案解决：

方法1：将y声明为mutable #

mutable关键字允许在const对象中修改该成员，刚好匹配你的需求——你只修改不影响排序的y，不会破坏set的有序性。修改你的结构体：

struct mystruct {
    int id;
    mutable vector<int> y; // 添加mutable修饰
    mystruct(const int id):id(id) {}
    bool operator<(const mystruct& x) const {
        return id < x.id;
    }
    bool operator==(const mystruct& x) const {
        return id == x.id;
    }
};

这样你的代码里i->y.push_back(4)就能正常运行了，甚至x2.y.push_back(3)也可以生效（因为y是mutable，即使x2是const对象）。

⚠️ 注意：一定要确保mutable成员不会影响元素的排序逻辑，否则会导致set的行为异常。

方法2：删除元素→修改→重新插入 #

如果你不想使用mutable，可以遵循set的设计规则：先找到元素，复制一份，删除原元素，修改后再插入回set。这种方法更“安全”，完全不会破坏容器的有序性：

void test() {
    std::set<mystruct> sx;
    mystruct x(1);
    x.y.push_back(1);
    x.y.push_back(2);
    sx.insert(x);

    auto it = sx.find(mystruct(1)); // 构造临时对象查找目标元素
    if (it != sx.end()) {
        // 复制元素到临时对象
        mystruct temp = *it;
        // 删除原元素
        sx.erase(it);
        // 修改临时对象的成员
        temp.y.push_back(4);
        // 将修改后的元素重新插入set
        sx.insert(temp);
    }
}

缺点是如果元素体积很大，复制会有性能开销，但逻辑上完全合规。

方法3：改用std::map替代std::set #

既然你的元素是用id作为唯一标识和排序依据，改用std::map<int, vector<int>>（或者保留结构体的std::map<int, mystruct>）会更合理——map天然将键（id）和值（你的数据）分离，值是可以自由修改的（只要键不变）：

void test() {
    std::map<int, vector<int>> sx; // 直接用id做键，存储vector即可
    sx[1].push_back(1);
    sx[1].push_back(2);

    auto it = sx.find(1);
    if (it != sx.end()) {
        it->second.push_back(4); // 直接修改vector，毫无限制
    }
}

这种方式逻辑更清晰，从根源上避免了set的const迭代器限制，也是最推荐的方案之一，尤其是当你的核心标识就是id的时候。

为什么原来的代码报错？ #

• x2是const mystruct对象，它的y成员默认也是const的，而vector::push_back不是const成员函数，所以无法调用。
• std::set的迭代器返回的是const mystruct&，所以i->y同样是const vector，自然也不能调用push_back。

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