为何两个看似相同的范围for循环一个编译通过一个报错？

咱们一步一步拆解为什么两个循环表现完全不同：

第一个循环：为什么能正常编译？ #

for (auto & c : solver) c = tolower(c);

solver是std::string类型，它的底层存储是可修改的char序列。这里的auto &c会被编译器推导为char&——直接引用字符串里的每个可修改字符，所以你可以放心地给c赋值，完全没问题。

第二个循环：为什么报错？ #

问题出在这两行：

for (auto p : opts)
    for (auto & c : p.first) c = tolower(c);

首先，opts是std::map<std::string, option_t>，而std::map的每个元素是std::pair<const std::string, option_t>——重点看这里：pair的第一个成员（也就是map的key）是const限定的！这是map的设计规则：为了维护内部有序的结构，不允许直接修改已存入的key。

当你写auto p : opts时，是把map里的每个pair<const string, option_t>做了一次值拷贝，得到的p.first自然也是const std::string。接下来遍历p.first的字符时，auto &c就被推导成了const char&——这是一个只读的引用，你当然不能给它赋值，Clang的报错完全在情理之中。

怎么解决？ #

要看你的实际需求：

• 如果只是需要获取key的小写版本来做后续操作（不需要修改原map）：

  for (auto &p : opts) { // 用引用避免不必要的拷贝
      std::string lower_key = p.first; // 拷贝key到可修改的string
      for (auto &c : lower_key) {
          c = tolower(static_cast<unsigned char>(c)); // 加个static_cast避免tolower处理负数值的问题
      }
      // 这里用lower_key做你需要的逻辑
  }
  

• 如果是想把原map的所有key都替换成小写版本：
  因为map的key不能直接修改，你需要创建一个新map，或者先删除旧元素再插入新的：

  std::map<std::string, option_t> new_opts;
  for (const auto &p : opts) {
      std::string lower_key = p.first;
      std::transform(lower_key.begin(), lower_key.end(), lower_key.begin(), 
                     [](unsigned char c) { return std::tolower(c); });
      new_opts.emplace(std::move(lower_key), p.second);
  }
  opts = std::move(new_opts); // 转移所有权，避免拷贝
  

额外提醒 #

调用tolower时最好把字符转成unsigned char，避免处理某些带符号字符（比如扩展ASCII）时出现未定义行为。

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