STL迭代器不起作用？ 无法弄清楚为什么

Question

我有一个有一个私人成员的班级：

std::set<unsigned long> Sset;

我有这个功能的问题：

原型：

Set& Union (Set&, Set&);

码：

    Set& Set::Union (Set& s1, Set& s2)
{
    set<unsigned long>::iterator a;
    set<unsigned long>::iterator j;

    for (a = s1.Sset.begin(); a!=s1.Sset.end(); ++a)
        for (j = s2.Sset.begin(); j!=s2.Sset.end(); ++j)
            if (*a = *j)
            {
                Sset.insert(*a);
                break;
            }

    return *this;
}

我得到编译器错误表达式必须是*a=*j的可修改左值

使用迭代器j一切正常，但它不接受*a

有任何帮助或解释吗？ 谢谢

Answer 1

你应该在if语句中使用==吗？

Answer 2

经典错误:-) - 比较需要两个等号

if (*a == *j)

Answer 3

if (*a = *j)

是一项任务

if (*a == *j)

Answer 4

您可以一次将一组插入另一组：

std::set<int> s1, s2;

s1.insert(s2.begin(), s2.end());

集合成员是独一无二的，所以没有什么可担心的。

Answer 5

您不能分配set::iterator指向的值。

为什么？

因为集合保持其元素的任何顺序 （ set ，特别是，已排序 ）以允许快速检索，因此您无法手动告诉它放置元素的位置。

而是insert元素insert正确的位置。

话虽如此 - 你的意思是使用==而不是= ？

Answer 6

#include <algorithm>

...
std::set_union(s1.begin(), s1.end(), s2.begin(), s2.end() ... )

http://en.cppreference.com/w/cpp/algorithm/set_union