C++20中使用范围for循环删除对象内嵌套集合元素的问题

你的判断完全正确——问题出在范围for循环的遍历方式上：当你写for(auto s : nestedSet)时，循环变量s是原内层std::set<int>的副本，所有删除操作都只作用在这个临时副本上，函数执行完副本销毁，原容器this->nestedSet自然不会有任何变化。

要修改原容器中的元素，必须用引用绑定到原集合的内层元素上，只需要把auto s改成auto& s即可，以下是修正后的两种实现：

修正后的第一种实现（使用std::erase_if） #

void MyObject::removeFromNestedSet(int criteria) {
    for(auto& s : nestedSet){  // 用auto&引用原内层集合
        std::erase_if(s, [&criteria](int i){return i == criteria;});
    }
}

修正后的第二种实现（使用find+erase） #

void MyObject::removeFromNestedSet(int criteria) {
    for(auto& s : nestedSet){  // 同样改为引用
        auto it = s.find(criteria);
        if(it != s.end()){
            s.erase(it);  // 若仅需删除单个匹配元素，用erase(it)即可；若要删除从it到末尾的所有元素，保留erase(it, s.end())
        }
    }
}

补充说明：std::set是有序且元素唯一的容器，每个内层集合中最多存在一个等于criteria的元素，因此第二种写法中s.erase(it)就能精准删除目标元素；如果你的实际需求是删除从criteria开始到集合末尾的所有元素（比如利用set的有序性删除大于等于criteria的元素），则保留s.erase(it, s.end())即可。

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