自定义map键的operator<报错C2679：请求解释错误原因

你尝试创建一个以std::map<std::string, std::string>作为key的外层map，因此封装了TagList类，但在重载<运算符时遇到了编译错误。我们来一步步拆解问题：

错误原因详解 #

先看你代码里触发错误的核心行：

std::string myVal2 = b._map.find(it->first);

std::map::find()方法的返回值是迭代器（具体是std::map<std::string, std::string>::const_iterator，因为b是const引用），而不是std::string类型。你直接把迭代器赋值给std::string变量，编译器找不到能完成这种转换的赋值运算符，所以抛出了C2679: binary '=' : no operator found which takes a right-hand operand of type 'std::_Tree_const_iterator<_Mytree>'错误——直白点说就是：你给字符串变量塞了一个迭代器类型的值，两者完全不兼容，编译器不知道该怎么处理。

另外，你的<运算符逻辑本身也存在漏洞：

• 只遍历了a的元素，完全没考虑b包含a没有的key的情况
• 用strcmp比较std::string属于多余操作，string本身原生支持<、>等比较运算符
• 当b没有a的某个key时直接返回false，这不符合std::map对key要求的严格弱序规则

修复后的operator<实现 #

其实你完全不用自己写遍历逻辑，因为std::map本身已经重载了<运算符，它会按key的字典序逐个比较元素，完全满足严格弱序的要求。所以你的TagList的<运算符可以直接复用底层map的比较逻辑：

inline bool operator< (const TagList &a, const TagList &b) {
    return a._map < b._map;
}

如果你因为特殊需求必须自己实现遍历逻辑，那需要修正迭代器的处理，同时补全完整的比较逻辑：

inline bool operator< (const TagList &a, const TagList &b) {
    auto it_a = a._map.begin();
    auto it_b = b._map.begin();
    
    while (it_a != a._map.end() && it_b != b._map.end()) {
        if (it_a->first != it_b->first) {
            // 先比较key的字典序
            return it_a->first < it_b->first;
        }
        // key相同的情况下，比较value
        if (it_a->second != it_b->second) {
            return it_a->second < it_b->second;
        }
        // 当前元素相等，继续遍历下一组
        ++it_a;
        ++it_b;
    }
    // 如果a的元素已遍历完，而b还有剩余元素，则a < b
    return it_a == a._map.end() && it_b != b._map.end();
}

额外优化建议 #

• 你的getValue方法可以简化，同时加上const修饰（因为方法不修改类成员）：

  std::string getValue(const std::string& tag) const {
      auto it = _map.find(tag);
      return it != _map.end() ? it->second : "";
  }
  

• 成员变量_map建议设置为private，只通过公共接口访问，更符合面向对象的封装原则。

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