C++两种for循环遍历容器结果异常差异，请求技术解析

核心问题：悬空引用导致的未定义行为 #

两种循环结果差异的根本原因是**objects_op->objects()的返回类型以及由此产生的悬空引用**：

1. 下标循环的致命错误 #

看下标循环中的关键代码：

EmObject& at = objects_op->objects().at(i);

如果objects()方法返回的是值类型（比如std::vector<EmObject>，而非引用类型），那么objects()会创建一个临时容器对象。当这行语句执行完毕后，临时容器会被立即销毁，而at引用的是临时容器中的元素——此时at就变成了悬空引用。后续访问at.object.polygon.pts时，实际访问的是已经被释放的内存，自然会出现垃圾值。

2. 范围for循环的有效逻辑 #

你贴出的范围for循环代码存在编译错误（未定义的i、重复定义的at），推测实际运行的是正确版本，类似：

for (auto &at : objects_op->objects()) {
  if (at.sub_type != ObjectType::VISION_PLD) {
     continue;
  }
  // 后续打印逻辑
}

C++标准规定：当范围for循环遍历临时对象时，临时对象的生命周期会被延长至整个循环周期。因此at引用的是临时容器中有效的元素，访问成员时不会出现悬空引用，输出结果正常。

额外问题：阴影变量 #

下标循环中内层for重新定义了size_t i：

for (size_t i = 0; i < 4; i++) { ... }

这会覆盖外层循环的i变量，属于不良代码风格，但不是导致垃圾值的直接原因，建议改为其他变量名（比如j）避免混淆。

修复方案 #

方案1：修改objects()方法的返回类型 #

如果objects_op管理的是持久化容器，将objects()的返回类型改为左值引用：

std::vector<EmObject>& objects();
// 只读场景使用const版本
const std::vector<EmObject>& objects() const;

这样无论使用范围for还是下标循环，at都会绑定到持久化容器中的有效元素，不会出现悬空引用。

方案2：提前存储容器的副本或引用 #

如果无法修改objects()的返回类型，在循环前先获取容器的副本或延长其生命周期：

// 存储副本（适合元素体积较小的场景）
auto objects = objects_op->objects();
for (size_t i=0; i< objects.size(); i++) {
   EmObject& at = objects.at(i);
   // 后续逻辑
}

// 只读场景用const引用延长临时对象生命周期
const auto& objects = objects_op->objects();
for (size_t i=0; i< objects.size(); i++) {
   const EmObject& at = objects.at(i);
   // 后续逻辑
}

方案3：修正范围for循环代码 #

确保范围for循环直接使用遍历得到的引用，避免多余错误代码：

for (auto &at : objects_op->objects()) {
  if (at.sub_type != ObjectType::VISION_PLD) {
     continue;
  }
  for (size_t j = 0; j < 4; j++) {  // 改用j避免阴影变量
    std::cout << " c: " << at.object.polygon.pts.at(j).transpose() << std::endl;
  }
  std::cout << BLUE << "after:" << &(at.object.polygon.pts) << RESET << std::endl;
}

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