c++ - 无法将迭代器的 std::set 中的元素插入到 std::list

Question

在展示给我编译器错误的代码之前，我想简要解释一下为什么我需要一组迭代器到一个列表，只是为了避免像“你真的需要那个吗？”这样的评论。或者也许将它们更改为评论“您的代码可以通过这种方式解决......但是我应该像这样处理原始问题”。您可以跳过阅读并直接转到最后一节(“未编译代码”)。

背后的原因

我构建了一个有向二分加权图。每条弧都存储在一个结构中

typedef struct edge_ {
    int src;
    int des;
    double w; //weight of the arc
}edge;

我使用此类结构的列表存储有关图的信息。

list<edge> all_edges;

之后，我按重量对弧线进行排序，然后将它们从最轻到最重循环循环。因为我想从 all_edges 中删除一些元素在循环中，在我调用的每个循环步骤的开始

list<edge>::iterator smallest = all_edges.begin();

在该循环的每一步中，在对弧的权重做了一些处理后，我想从图中删除所有离开 src 的节点。或以 des 结尾.为此，在图的构造过程中，我创建了两个 vector ，一个用于二分图的每个组件，由它们的元素索引，并且在每个 vector 的每个位置，我将所有迭代器的列表存储到边缘all_edges它偏离与所考虑的 vector 位置相对应的节点。代码如下(“小”和“大”是我识别二分图两个分量的方式)

vector<list<list<edge>::iterator>> edges_from_small(small.size());
vector<list<list<edge>::iterator>> edges_from_big(big.size());

这是我用来填充上述 vector 的代码

//inside a loop...
    edge e;
    e.src = ...
    e.des = ...
    e.w = ...
    all_edges.push_back(e);                
    edges_from_small[e.src].push_back(--(all_edges.end()));
    edges_from_big[e.des].push_back(--(all_edges.end()));

假设我想删除边 e .我很想循环所有元素 edges_from_small[e.src]并为他们每个人打电话 all_edges.erase(iterator) , 但是这样做，因为一条边可以同时列在 edges_from_small 中和 edges_from_big ，我会结束使用取消引用的迭代器删除元素的尝试!

Set 将是一个解决方案!我只需要创建一组 list<edge>::iterator并用 edges_from_small[e.src] 的元素填充它和 edges_from_big[e.des]然后，由于删除了重复项，因此删除列表中的所有元素 all_edges .

但我的代码无法编译，它在以下行之一处给我一个我无法理解的错误:

set<list<edge>::iterator> to_remove;

for (auto it = edges_from_small[smallest->src].begin(); it != edges_from_small[smallest->src].end(); ++it) {
        to_remove.insert(*it); //error here! 
    }
for (auto it = edges_from_big[smallest->des].begin(); it != edges_from_big[smallest->des].end(); ++it) {
        //to_remove.insert(*it); //error here!
    }
for (auto it = to_remove.begin(); it != to_remove.end(); ++it) {
        all_edges.erase(*it);
    } 

编译器给了我一个相当大的输出(全部引用上面的行)，目前我只输入了第一行和最后一行，我认为这是最具指示性的。

g++ -ggdb3 -g -O0 -std=c++14 -Wall -Werror -o compare main.cpp peaks.cpp common.cpp compare.cpp parameters.cpp 
In file included from compare.cpp:1:
In file included from /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/iostream:38:
In file included from /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/ios:216:
In file included from /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/__locale:15:
In file included from /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/string:439:
In file included from /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/algorithm:628:
In file included from /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/memory:606:
In file included from /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/iterator:344:
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/__functional_base:63:21: error: 
      invalid operands to binary expression ('const std::__1::__list_iterator<_edge,
      void *>' and 'const std::__1::__list_iterator<_edge, void *>')
        {return __x < __y;}
                ~~~ ^ ~~~
....................MUCH MORE OUTPUT....................
compare.cpp:226:23: note: in instantiation of member function
      'std::__1::set<std::__1::__list_iterator<_edge, void *>,
      std::__1::less<std::__1::__list_iterator<_edge, void *> >,
      std::__1::allocator<std::__1::__list_iterator<_edge, void *> > >::insert'
      requested here
            to_remove.insert(*it);
                      ^
1 error generated.
make: *** [compare] Error 1

Compilation exited abnormally with code 2 at Tue Sep  5 17:34:39

知道那条线有什么问题吗？

未编译代码

typedef struct _edge {
    int src;
    int des;
    double w; //weight of the arch
}edge;

list<edge> all_edges;
vector<list<const list<edge>::iterator>> edges_from_small(small.size());
vector<list<const list<edge>::iterator>> edges_from_big(big.size());

//graph construction    
//for loop ...
    edge e;
    e.src = ...
    e.des = ...
    e.w = ... 
    all_edges.push_back(e);
    edges_from_small[e.src].push_back(--(all_edges.end())); 
    edges_from_big[e.des].push_back(--(all_edges.end()));
//end of loop

list<edge>::iterator smallest = all_edges.begin();
set<list<edge>::iterator> to_remove;
for (auto it = edges_from_small[smallest->src].begin(); 
     it != edges_from_small[smallest->src].end(); ++it) {
    to_remove.insert(*it); //<--- COMPILER ERROR HERE, you can see the error description in the last lines of the previous paragraph
}    

Answer 1

std::list::iterator 无法排序，因为没有函数或运算符来比较它们(wrt less/greater)。

改为使用 std::unordered_set，这不需要对元素进行排序，它使用元素的哈希值将它们放入桶中。您可能需要提供一个哈希函数，只需在 std::unordered_set 的 namespace std 中放置一个 std::hash 的重载即可找到并使用它。
另请注意，std::unordered_set 具有插入的平均恒定时间复杂度，而 std::set

的对数时间复杂度