c++ - 再次获取 std::map 会更改前一个迭代器

Question

我找不到任何类似的问题。 当我调用 getMap 时，之前的迭代器似乎发生了变化:

//IF I COMMENT THE EVIL PRINT, THE PROBLEM DOES NOT OCCUR
std::cout << "EVIL PRINT" << std::endl;    
Something something;
auto mapElement = something.getTheMap().find("A");
std::cout << "Before: " << mapElement->first << std::endl;
something.getTheMap();
std::cout << "After: " << mapElement->first << std::endl << std::endl;

/****************************************************************************************/

//WITH SHARED POINTERS, THE EVIL PRINT IS NOT NECCESARY TO MAKE THE PROBLEM OCCUR
std::shared_ptr<Something> somePtr;
auto mapElement2 = something.getTheMap().find("A");
std::cout << "Before: " << mapElement2->first << std::endl;
something.getTheMap();
std::cout << "After: " << mapElement2->first << std::endl << std::endl;

输出:

EVIL PRINT
Before: A
After: B

Before: A
After: B

完整的代码可以在这里运行https://coliru.stacked-crooked.com/a/66b48636a476ddb7

这是一种想要的行为吗？发生了什么事？

Answer 1

您没有在问题中包含最重要的部分，即

std::map <std::string, int> getTheMap() {
        return theMap;
}

getTheMap 返回一个拷贝，因此 getTheMap().find("A"); 返回一个指向临时对象的迭代器(该对象在调用完成后停止存在) .
因此，该迭代器引用了一个不再存在的对象，它是一个悬空迭代器。取消引用它(就像使用 mapElement->first 一样)会调用未定义的行为

最惯用的修复方法是让 getTheMap 返回一个引用，例如:

std::map <std::string, int>& getTheMap() {
        return theMap;
}