关于在List中遍历删除元素的问题的自省

2 篇文章 0 订阅

关于在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的没更新,导致逻辑进行下去了,就出现了越界的错误
          //逻辑又怎么会对呢??? 
          //以后写逻辑一定要严谨细心!!!谨以此警示自己!!!
        }
    }
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值