为什么Resharper建议我简化“不等于”“不等于”？

Question

我需要检查是否一个项目不会在C＃项目列表存在，所以我有这样一行：

if (!myList.Any(c => c.id == myID))) 

ReSharper的是在暗示我应该改变，要：

if (myList.All(c => c.id != myID))) 

我可以看出它们是相同的，但它为什么会提示更改？由于某种原因，第一种执行速度较慢？

Answer 1

在第一个例子中，在表达式的开头部分很容易错过!。你因此使表达难以阅读。另外，第一个例子是“不等于”，而第二个例子是“不等于”。并不是巧合，易于阅读的代码可以表达为更容易阅读英文。

更容易阅读的代码很可能会少一些错误，因为它更容易理解它在改变它之前的作用。这是因为第二个例子更清楚ReSharper建议更改您的代码。

Answer 2

表达的可读性对我个人的意见。

为“我不是在集合项目”我会读这个

if (!myList.Any(c => c.id == myID))) 

。这里

if (myList.All(c => c.id != myID))) 

读为“集合中的所有项目都不同于我的项目”。

如果我想问的'问题' - 通过我的linq查询 - '是我的项目不在列表中'，那么第一个查询更适合我想问的问题。第一个查询前面的!不是问题。

Answer 3

一般来说，提出一个积极的问题是更直观的。如果你问用户“你真的不想删除这个记录？”，猜猜他多久会碰到一个错误的按钮。

我个人喜欢把构建这样的周围：

// Not optimal 
if (!x) { 
    A(); 
} else } 
    B(); 
} 

// Better 
if (x) { 
    B(); 
} else } 
    A(); 
} 

例外可能是测试not null其中a != null可能被看作是积极的。