C++98旧代码GCC编译出现non-const lvalue reference绑定错误求助

C++98标准明确规定：非常量左值引用无法绑定到临时对象（右值），你的代码触发该规则的逻辑如下：

1. std::map::insert调用后返回的是std::pair<iterator, bool>类型的临时对象，取其first成员得到的迭代器仍然属于临时值（右值）
2. UnpackCommunityData的形参声明为std::map<long, CommunityData>::iterator&，是非常量左值引用，无法接收右值入参，因此编译器抛出匹配失败错误。

早年GCC（2.x/3.x早期版本）存在非标准扩展，默许非常量左值引用绑定临时对象，因此90年代的旧代码当时可以编译通过，新版GCC严格遵循C++标准后就会触发该报错。

以下两种方案均符合C++98标准，可按需选择：

方案1（改动最小，无兼容性风险，推荐） #

将临时迭代器保存为局部左值变量后再传参，不需要修改原函数的签名：

// 先将insert返回的迭代器存入局部变量（属于左值，可以绑定到非常量左值引用）
std::map<long, CommunityData>::iterator it = data.insert(std::map<long, CommunityData>::value_type(ReadLong(), CommunityData())).first;
UnpackCommunityData(it);

方案2（调用代码无需改动，适合迭代器本身不会被修改的场景） #

观察你提供的UnpackCommunityData实现，函数内仅修改迭代器指向的CommunityData对象，从未修改迭代器本身的指向，因此可以直接修改函数的参数类型为值传递：

// 修改函数声明和定义的参数，去掉引用符号
void UnpackCommunityData(std::map<long, CommunityData>::iterator itInfo);

迭代器本身是轻量对象，值传递的开销可忽略，修改后原调用代码不需要做任何调整即可正常编译。

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