映射/设置迭代器不兼容 [c++] [关闭]

Question

我正在尝试遍历地图，如下所示：

data = &server->update();

    for (auto const iter : *data)
    {
        do things
    }

但一旦 for 行运行，我就会收到一条错误消息，指出“映射/设置迭代器不兼容”。

数据变量是一个

std::map<unsigned int, Packet> *data;

它是一个指针，因为它位于：

public ref class ServerForm : public System::Windows::Forms::Form

这似乎不像任何不是指针的东西。

我尝试了多种方法来遍历此地图，但每次尝试都完全失败。非常感谢您了解为什么会发生这种情况以及如何解决它，谢谢。

Answer 1

data = &server->update();
// same as
data = &(server->update());

由于 update 函数按值返回，因此您将指针绑定到具有自动存储持续时间的实例，其生命周期以该行结束（“临时”）。

事实上，在你的循环后面

for(auto const it : *data)

您正在取消引用无效指针，这会产生未定义的行为。

真正让我吃惊的是，代码竟然还在编译。我无法让这样的测试用例在 ideone/GCC 中工作。

---

要使用更新函数返回的临时值，您有以下可能性：

1. 将地图存储在局部变量中。

2. 通过将临时对象绑定到引用来延长其生命周期：

   auto & map = server->update ();
   

   这在调用堆栈中“向上”不起作用，因此您无法返回该引用。

3. 将循环带入限制临时生命周期的完整表达式：

   for (auto const it : server->update()) {
     // ....