STL 迭代器 erase 的一个错误用法

今天检查出代码的一个异常，也怪自己太大意，习惯了ctrl+c/v。。

描述如下：

map<UINT, CModule*>  m_ModulePoints;
for (map<UINT, CModule*>::iterator it = m_ModulePoints.begin();  it != m_ModulePoints.end();  it++)
{
		CModule *pModule= (CModule*)(*it).second;

		if (.......)
		{
			delete pModule;
			m_ModulePoints.erase(it);
		}	
			
}

那么问题来了，iterator it 被erase后，for循环仍旧对齐进行++操作，导致异常。在没有使用erase的情况下，我们确实可以使用这样的for循环结构遍历map，一旦需要erase，则不允许这样做。

正确的做法之一:

for (map<UINT, CModule*>::iterator it = m_ModulePoints.begin();  it != m_ModulePoints.end(); )
{
		CModule *pModule= (CModule*)(*it).second;

		if (.......)
		{
			delete pModule;
			it = m_ModulePoints.erase(it);
		}	
		else

<span style="white-space:pre">		</span>{

<span style="white-space:pre">			</span>it++;

<span style="white-space:pre">		</span>}	
}

利用erase函数的返回值可以避免这一问题。

另一种做法是：

for (map<UINT, CModule*>::iterator it = m_ModulePoints.begin();  it != m_ModulePoints.end(); )
{
		CModule *pModule= (CModule*)(*it).second;

		if (.......)
		{
			delete pModule;
			it = m_ModulePoints.erase(it++);
		}	
	
}