C++范围基于for循环修改类对象成员失效的原因与正确写法

你编写的范围for循环失效，核心是auto m : mList 属于值传递式的遍历逻辑：每次循环迭代时，编译器会把mList中当前位置的元素完整拷贝出一个临时副本存入变量m，你后续对m.watched做的所有修改，全部作用在这个临时副本上，完全不会触碰原vector存储的真实对象，自然无法实现预期的成员修改效果。
而你能用的索引循环之所以生效，本质是mList.at(i)返回的是vector内部存储元素的原生引用，所有操作直接作用在原对象上。

只需要把范围for的遍历变量调整为引用类型，让循环变量直接绑定原vector内的元素，不再生成拷贝副本即可：

• 遍历过程无需修改元素时，使用const auto& m : mList 可以规避不必要的对象拷贝，提升运行效率
• 遍历过程需要修改元素成员时，必须使用非const左值引用作为遍历变量类型，也就是将循环头改写为for(auto& m : mList)

修改后可正常运行的代码示例：

// 可正确修改原对象成员的范围for实现
for(auto& m : mList)
{
   if(m.name == name_val)
   {
      m.watched += 1;
      return;
   }
}

你此前尝试的++m.watched、++(m.watched) 这类成员修改写法本身语法完全正确，失效的根源不在成员操作的写法，而在循环变量本身是元素的拷贝——哪怕对拷贝对象的成员做任何合法修改，都不会反向影响原vector中的存储内容。

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