关于在List中遍历删除元素的问题的自省
程序员文章站
2024-02-09 18:05:42
...
关于在List中遍历删除元素的问题,在网上搜的大多数的都是 通过倒序的方式,当然也可以通过临时建一个list,然后把存储 但是如果不用倒序的方法 又该怎么写呢?
下面是两种通常的写法:
//倒序的方法
private readonly List<TimerTask> m_TimerTaskList = new List<TimerTask>();
private readonly List<TimerTask> m_DeleteTimerTaskList = new List<TimerTask>();
for (int i = m_TimerTaskList.Count - 1; i >= 0; --i)
{
TimerTask timerTask = m_TimerTaskList[i];
if (null == timerTask)
continue;
if (xx条件)
{
m_TimerTaskList.Remove(timerTask);
}
}
// 通过两个list实现
for (int i = m_TimerTaskList.Count - 1; i >= 0; --i)
{
TimerTask timerTask = m_TimerTaskList[i];
if (null == timerTask)
continue;
if (xx条件)
{
m_DeleteTimerTaskList.Add(timerTask);
}
}
for (int i = 0, iCnt = m_DeleteTimerTaskList.Count; i < iCnt; ++i)
{
m_TimerTaskList.Remove(m_DeleteTimerTaskList[i]);
}
但是上面两种方法都有各自的缺点 倒序的话 如果 判断条件内的逻辑是 有序的话就不适合了。
两个list 效率很差。那么如何在正序的情况下正确的遍历并删除list中的元素 ,我就直接贴代码了。
private readonly List<TimerTask> m_TimerTaskList = new List<TimerTask>();
for (int i = 0, iCnt = m_TimerTaskList.Count; i < iCnt; ++i)
{
TimerTask timerTask = m_TimerTaskList[i];
if (null == timerTask)
continue;
if (xx条件)
{
m_TimerTaskList.RemoveAt(i);
--i;
--iCnt; // 这里一定要注意!!!! 网上很多博客到上面那行就结束了!!! 这种就是错的!!
//为什么呢? 因为当List执行了Remove操作之后, 这时List的Count还没更新,
//到了下次循环的判断的时候,正常应该进不来的逻辑,因为icnt的没更新,导致逻辑进行下去了,就出现了越界的错误
//逻辑又怎么会对呢???
//以后写逻辑一定要严谨细心!!!谨以此警示自己!!!
}
}
下一篇: 关于在html中换行引起的空格问题