爲什麼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可能被看作是積極的。