不要使用有副作用的std.map.[]操作符

map的[]操作符包含隐性操作:当key不存在的时候，会默认执行insert操作

这种隐性操作在大多数情况下是有害的。

比如下面的代码：

// 判断是否有效技能ID inline BOOL KSkillManager::IsValidSkillID(int nSkillID) { BOOL bRet = m_mapSkillID2SettingIndex[nSkillID] > 0;//当nSkillID不存在时，会执insert(nSkillID, 0) return (bRet); }

判断一个nSkillID是否有效，却偷偷的对无效的技能id做了插入操作，导致有大量非技能ID用作查询的时候，map变得非常大，严重影响系统效率。

这种情况的正确用法如下：

// 判断是否有效技能ID inline BOOL KSkillManager::IsValidSkillID(int nSkillID) const { std::map<int, int>::const_iterator it = m_mapSkillID2SettingIndex.find(nSkillID); BOOL bRet = (it != m_mapSkillID2SettingIndex.end() && it->second > 0); return (bRet); }

修改要点:

1.只用作查询的成员函数，申明为const成员，这样可以让编译器帮我们确认是否修改到成员。

2.使用map.find查询key是否存在，而不要使用带有副作用的[]操作符。