关于映射和迭代器的理论说明

Question

如果我有一个带有地图的类作为私有成员，例如

class MyClass
{
  public:
    MyClass();
    std::map<std::string, std::string> getPlatforms() const;
  private:
    std::map<std::string, std::string> platforms_;
};

MyClass::MyClass()
        :
{
  platforms_["key1"] = "value1";
  // ...
  platforms_["keyN"] = "valueN";
}

std::map<std::string, std::string> getPlatforms() const
{
  return platforms_;
}

在我的主函数中，这两段代码会有区别吗？

代码1：

MyClass myclass();
std::map<std::string, std::string>::iterator definition;
for (definition = myclass.getPlatforms().begin();
     definition != myclass.getPlatforms().end();
     ++definition){
  std::cout << (*definition).first << std::endl;
}

代码2：

MyClass myclass();
std::map<std::string, std::string> platforms = myclass.getPlatforms();
std::map<std::string, std::string>::iterator definition;
for (definition = platforms.begin();
     definition != platforms.end();
     ++definition){
  std::cout << (*definition).first << std::endl;
}

在 Code2 中，我刚刚创建了一个新的地图变量来保存从 getPlatforms() 函数返回的地图。

无论如何，在我的真实代码中（我无法从中发布真实代码，但它直接对应于这个概念）第一种方式（代码 1）会导致运行时错误，无法访问某个位置的内存。

第二种方法有效！

您能告诉我这两段不同代码之间发生了什么的理论基础吗？

Answer 1

getPlatforms() 按值返回地图，而不是按引用返回，这通常是个坏主意。

你已经举了一个例子来说明为什么这是一个坏主意：

getPlatforms().begin() 是地图上的迭代器，该迭代器在使用之前就消失了，getPlatforms().end() 是来自同一原始地图的不同副本上的迭代器。

Answer 2

您能告诉我这两段不同代码之间发生了什么的理论基础吗？

当您按值返回时，您返回的是数据的深层副本。

当您调用myclass.getPlatforms().begin(); 和myclass.getPlatforms().end(); 时，您实际上是在构造数据的两个副本，然后从一个副本获取开始迭代器，从另一个副本获取结束迭代器。然后，比较两个迭代器是否相等； 这是未定义的行为。

导致运行时错误，无法访问某个位置的内存。

这是因为definition被初始化了，然后用来创建它的临时对象被删除，使迭代器指向的数据失效。然后，您尝试通过迭代器使用数据。

Answer 3

您遇到的一个问题是您应该使用const_iterator 而不是iterator。这是因为函数getPlatforms 是const 限定的，而映射中的函数iterator begin() 不是；您必须使用 const 限定 const_iterator begin() const 来明确告诉编译器您不会修改该类的任何成员。

注意：这只是代码 1 的情况，顺便说一句，它应该返回 const&