我有一个“微类”我写的是为了做一些快速的数据操作。
这是项目:
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 。
右?
我在这里做错了什么?
答案 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