3

我写了一个“微课”来进行一些快速的数据操作。

这是项目：

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。

对？

我在这里做错了什么？

4

3 回答 3

10

对？

错误的。一个空序列被表示为……一个空序列。这与空引用不同。相反，这是一个您可以正常调用成员的序列 - 您不会得到任何结果。哎呀，由于 LINQ 是惰性的，序列甚至不知道是否有任何元素，直到您尝试询问它们。Where不可能返回null并仍然懒惰地执行。

我想你想要Any。

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

请注意，一般来说，使用空序列（空列表、空数组等）比使用空引用更干净——它允许调用者拥有一个可以以任何方式工作的路径，而不必在每一步都检查是否为空。

于 2013-08-06T14:19:17.757 回答
3

你是在自问自答。该列表为空（Count == 0），而不是 NULL。

所以你的代码应该是

list.Where(o => o.Badge == badge).Count() == 0
于 2013-08-06T14:20:26.603 回答
2

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

也许你的意思是，

list.Where(o => o.Badge == badge).Count() != 0
于 2013-08-06T14:19:15.823 回答