为什么这个Linq函数在不是时返回true?

时间:2013-08-06 14:17:24

标签: c# visual-studio-2010 lambda

我有一个“微类”我写的是为了做一些快速的数据操作。

这是项目:

class AEmp {
  public AEmp() {
    Parts = new AParts();
  }
  public string Badge { get; set; }
  public AParts Parts { get; set; }
  public int Days { get; set; }
}

以下是清单:

class AEmps {
  private List<AEmp> list;
  public AEmps() {
    list = new List<AEmp>();
  }
  public AEmp this[string badge] {
    get { return list.SingleOrDefault(e => e.Badge == badge); }
  }
  public void Add(AEmp item) {
    if (!Contains(item.Badge)) {
      list.Add(item);
    }
  }
  public bool Contains(string badge) {
    return (list.Where(o => o.Badge == badge) != null);
  }
  public int Count { get { return list.Count; } }
  public AEmp Owner(string serialNo, DateTime startDay, DateTime nextDay) {
    return (list.SingleOrDefault(e => e.Parts.Contains(serialNo, nextDay)));
  }
}

我面临的问题是当我去列表中添加新项目时。如果项目已经存在,我不想添加项目,因此我使用Contains方法进行检查。

列表为空(Count = 0),因此list.Where(o => o.Badge == badge) != null)应返回 FALSE

右?

我在这里做错了什么?

3 个答案:

答案 0 :(得分:10)

  

右?

错误。空序列表示为......空序列。这与null引用不同。相反,它是一个你可以正常呼叫成员的序列 - 你只是不会得到任何结果。哎呀,因为LINQ是懒惰的,序列甚至不会知道是否有任何元素,直到你试图要求它们。 Where无法返回null并仍然懒惰地执行。

我认为你想要Any

return list.Any(o => o.Badge == badge);

请注意,通常情况下,使用空序列(空列表,空数组等)比使用空引用更简洁 - 它允许调用者有一条路径可以以任何方式工作,而不必检查无效每一步。

答案 1 :(得分:3)

你回答了自己的问题。 列表为空(Count == 0),而不是NULL。

所以你的代码应该是

list.Where(o => o.Badge == badge).Count() == 0

答案 2 :(得分:2)

因为您已在构造函数中实例化list,导致它不会变为NULL

也许你的意思是,

list.Where(o => o.Badge == badge).Count() != 0