做个测试,随便搞个类,头文件加入
#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++。