C++值传递std::map时返回的迭代器在主函数中取值异常问题

你写的get_population函数形参接收的map是值传递形式，调用时会把主函数里的populationMap完整深拷贝一份，作为函数栈帧上的局部临时对象。

• 函数内通过find得到的迭代器，指向的是这个栈上临时拷贝map的内部节点，和主函数里的原map没有关联。
• 当get_population执行完返回时，这个栈上的临时map会被立即析构，对应的栈内存会被回收复用。这时候把这个迭代器带回主函数使用，本质是读取已经释放的内存块，属于标准的未定义行为。

为什么只有Maharashtra的查询值异常 #

悬空迭代器的行为是不确定的，不会每次都直接崩溃或者返回全错的结果：

• 栈内存被释放后不会立刻清零，原来存储的人口数据会在内存里残留一段时间。如果后续程序还没覆写这部分内存，读到的值就和临时map里存的一致，看起来就像“程序运行正常”——你测试时查Bihar、West Bengal的结果正确，就是刚好这几个条目对应的内存还没被覆写。
• 栈内存的覆写是随机的：程序执行输出操作时，新的函数栈帧会占掉刚才临时map所在的内存区域，刚好把Maharashtra条目对应的内存位置覆盖成了其他临时变量的值，这时候读出来就是1769234796这种无意义的乱码值。
• 本质上不管查哪个键，这段代码的逻辑都是有问题的，“其他键正常”只是内存状态碰巧匹配，不是逻辑真的无bug。

修正方法 #

• 把map的传参从值传递改成const引用传递，从根源上避免生成临时拷贝。这样函数内find得到的迭代器直接指向主函数里一直存活的原map，对象不会被提前析构，迭代器自然不会失效。
• 补充查找结果的合法性判断：如果find返回的是end()迭代器（用户输入的行政区不存在），不要直接解引用，避免触发崩溃。

修正后的参考代码：

std::map<std::string, int>::const_iterator get_population(
    const std::string& state,
    const std::map<std::string, int>& populationMap
) {
    auto iter = populationMap.find(state);
    if (iter == populationMap.end()) {
        std::cout << "未找到对应行政区的人口数据\n";
        return populationMap.end();
    }
    std::cout << "函数内读取人口值：" << iter->second << "\n";
    return iter;
}

• 额外优化：这个查找逻辑本身非常简单，完全没必要单独封装返回迭代器。要么直接在主函数里执行查找操作，要么让函数直接返回匹配到的人口数值，从根源上规避迭代器生命周期的问题。

内容的提问来源于stack exchange，提问作者Harsheen Rajpal