关于STL的list，vector等用迭代器iterator，用erase删除元素出现的问题。

做个测试，随便搞个类，头文件加入

#include <vector>
#include <list>
using namespace std;

typedef std::vector<CString> CStringVector;

 

 

然后在一个可执行到的方法中加入如下代码：

CStringVector vctCString;
vctCString.push_back(_T("1111"));
vctCString.push_back(_T("2222"));
vctCString.push_back(_T("3333"));
vctCString.push_back(_T("4444"));
vctCString.push_back(_T("5555"));

CStringVector::iterator it = vctCString.begin();
for (; it != vctCString.end(); it++)
{
    CString strTemp = *it;
    if (strTemp == _T("1111"))
    {
        vctCString.erase(it);
    }
}

很明显，当erase那个it后，再执行it++就会失败，这是最常见的问题，erase之后会导致it失效，怎么解决呢？

erase的返回值是下一个元素的位置，让这个值重新付给it就行了。但是粗心的人会这么写：

CStringVector vctCString;
vctCString.push_back(_T("1111"));
vctCString.push_back(_T("1111"));
vctCString.push_back(_T("3333"));
vctCString.push_back(_T("4444"));
vctCString.push_back(_T("5555"));

CStringVector::iterator it = vctCString.begin();
for (; it != vctCString.end(); it++)
{
    CString strTemp = *it;
    if (strTemp == _T("1111"))
    {
        CStringVector::iterator itTemp = vctCString.erase(it);
        it = itTemp;
    }
}
这么写对吗？也不对，但不会崩溃，也就是解决了前面的问题，但是带来了新的问题，这样写的话会导致这个for循环少循环一次，也就少判断了一次，哪一次呢？就是if成立后的下一个元素，没有判断到，原因就不用说了吧。

最正确的写法是：

CStringVector vctCString;
vctCString.push_back(_T("1111"));
vctCString.push_back(_T("1111"));
vctCString.push_back(_T("3333"));
vctCString.push_back(_T("4444"));
vctCString.push_back(_T("5555"));

CStringVector::iterator it = vctCString.begin();
for (; it != vctCString.end(); )
{
    CString strTemp = *it;
    if (strTemp == _T("1111"))
    {
        CStringVector::iterator itTemp = vctCString.erase(it);
       it = itTemp;
    }
    else
    {
        it++;
    }
}

什么意思？for小括号里的it++不写了，放到里面来，这样当if有效的时候，其实已经是it++过了，所以这种情况不需要小括号内再++了，再++就加多了，当if无效的时候正常执行else的it++。明白了不？不明白给我发邮件吧。haoxing168@163.com