zoukankan      html  css  js  c++  java
  • 用Count() > 0 来判断集合非空的问题

    Linq 出现之前,我们通常使用下面的方式来判断集合是否非空,即集合包含元素:

    var array = new int[0];
    var b1 = array.Length > 0;
    
    var list = new List<string>();
    var b2 = list.Count > 0;
    
    var collection = new Collection<double>();
    var b3 = collection.Count > 0;

    使用 Length 或 Count 属性,上面的写法没有问题。

    但到了 Linq 时代,Enumerable.Count 扩展方法“统一了“ Length 和 Count 属性,于是就有了下面判断非空的写法:

    public static void SomeAction<T>(IEnumerable<T> source){
        if (source.Count() > 0){
            //...
        }//...
    }

    这种写法可以,运行也正常,但可能会产生非常严重的性能的问题

    注意是可能,并不是一定,上面的方法如果传入的是 Array、List<T>或Collection<T>,不会有问题。

    那么什么时候会出问题呢?我们来看如下方法:

    public static IEnumerable<int> GetNums(int start, int count)
    {
        var end = start + count;
        for (int i = start; i < end; i++)
            yield return i;
    }

    如下调用时:

    var nums = GetNums(0, int.MaxValue);
    SomeAction(nums);

    执行速度会相当慢,我的电脑大约用了 70 秒的时间来执行 source.Count() > 0。

    分析下的话,你会发现 GetNums 第 5 行代码 yield return i 执行了 int.MaxValue 次,有必要吗?

    其实只要返回一个元素我们就可以断定集合非空,完全不需要将所有的元素返回。

    我们可以使用 Enumerable.Any 扩展方法

    将 SomeAction 方法修改如下:

    public static void SomeAction<T>(IEnumerable<T> source){
        if(source.Any()){ // 切勿使用 source.Count() > 0
            //...
        }//...
    }

    再次调用 ,你会发现执行时间可以忽略不计了。

    总结下规律, Count() > 0 遇上 yeild return 必定会出现性能问题

  • 相关阅读:
    23种设计模式(12):策略模式
    23种设计模式(11):责任链模式
    23种设计模式(10):命令模式
    23种设计模式(9):访问者模式
    23种设计模式(8):观察者模式
    23种设计模式(7):中介者模式
    23种设计模式(6):模版方法模式
    创建型模式总结
    23种设计模式(5):原型模式
    leetcode6
  • 原文地址:https://www.cnblogs.com/wd0730/p/3281670.html
Copyright © 2011-2022 走看看