当我叫了两次时，set_union得到了错误的结果

set_union got wrong result when i called it twice

本文关键字：union 错误 结果 set 我叫 两次      更新时间：2023-10-16

这个问题让我困惑了很多小时， 请帮帮我！ 第一次叫set_union，结果是对的，但第二次叫，结果错了，看代码：

std::vector<int> set1{ 1, 2, 3, 4, 5, 6 };
std::vector<int> set2{ 4, 5, 6, 7, 8 };
std::vector<int> result{};
std::set_union(std::begin(set1), std::end(set1),
std::begin(result), std::end(result), 
std::back_inserter(result));  
// result is 1 2 3 4 5 6
std::back_insert_iterator< std::vector<int>> back2 =
std::set_union(std::begin(set2), std::end(set2), 
std::begin(result), std::end(result), 
std::back_inserter(result));

我调试上面的代码，得到这个结果：

[0] 1   int
[1] 2   int
[2] 3   int
[3] 4   int
[4] 5   int
[5] 6   int
[6] 1   int
[7] - 572662307 int
[8] - 572662307 int
[9] - 572662307 int
[10] - 572662307    int
[11] - 572662307    int
[12]    4   int
[13]    5   int
[14]    6   int
[15]    7   int
[16]    8   int

生成的范围不能与任一输入范围重叠。 https://en.cppreference.com/w/cpp/algorithm/set_union

你为什么不使用它？

std::set_union(std::begin(set1), std::end(set1), 
std::begin(set2), std::end(set2), 
std::back_inserter(result));

您在此处有未定义的行为。当您将迭代器作为输入和输出参数传递给result时std::set_union，写入输出迭代器会使所有输入迭代器失效。这是由于std::vector的内存布局 - 如果容量无法容纳新元素，则写入它可能需要将其所有数据移动到新的内存位置，因此引用初始内存位置的引用/指针/迭代器不能再使用。

您希望将结果存储在新容器中，例如

std::vector<int> out;
std::set_union(std::begin(set2), std::end(set2), 
std::begin(result), std::end(result), 
std::back_inserter(out));

另请注意，对于第一次调用，std::copy是更合适的算法(或赋值result = set1;)。

相关文章：