2017-03-25 12 views
0

は現在、私はこれは、アイテムがきゅうアイテムプレゼント検証

var wifiSSID = userLocation.LocationWiFiSSIDs.Where(x => x.WiFiSSID == attendanceDto.WiFiSSID).FirstOrDefault(); 
if (IsWiFiSSIDEnabled && wifiSSID == null) 
{ 
    throw new ApplicationException(ErrorMessages.InvalidWiFiSSID); 
} 

は私がより良い方法でこれを書くことができ、リストに存在しているかどうかを検証するのですか?任意または含まれていますか?

if (IsWiFiSSIDEnabled && !userLocation.LocationWiFiSSIDs.Any(x => x.WiFiSSID == attendanceDto.WiFiSSID) 
{ 
    throw new ApplicationException(ErrorMessages.InvalidWiFiSSID); 
} 

この方法userLocationIsWiFiSSIDEnabledtrue場合にのみチェックされます:

答えて

2

次のように、あなたがAny()を使用することができ、wifiSSIDだけで、あなたのコード内での検証のために使用されていると仮定すると。

+0

代わりにすべてのどれを使用するための具体的な理由はありますか? – jitender

+0

実際には..... – har07

+0

'All()'はここで 'Any()'よりも少し読みやすくなっています。 – dotNET

2

使用するすべての

if (IsWiFiSSIDEnabled && userLocation.LocationWiFiSSIDs.All(x => x.WiFiSSID != attendanceDto.WiFiSSID) 
{ 
    throw new ApplicationException(ErrorMessages.InvalidWiFiSSID); 
} 
関連する問題