リストをループし、一致するものがない場合にのみ "返す"

Question

文字列のリストをループし、それらを1つの文字列に対してチェックしようとしています。一致するものが見つからない場合は、コードを終了する必要があります。設定で1つだけの文字列があるかどう

// loadedObj.Settings contains the list of strings, can be any number of strings  
foreach (var currentCheckBox in loadedObj.Settings.Where(currentCheckBox => currentCheckBox != null)) 
    { 
     // docTypeAlias is a single string that needs to be matched 
     var docTypeAlias = sender.ContentType.Alias; 
     // This is the current value of currentCheckBox 
     var requiredTypeAlias = currentCheckBox; 
     if (!requiredTypeAlias.Equals(docTypeAlias)) return; 
    } 

コードが正常に動作しますが、最初のものは、明らかに一致しない場合は、すぐにあなたが複数の文字列を持っているように、コードがあまりにも早く終了します。

Answer 1

Anyを使用すると、シーケンス内の要素が条件に一致するかどうかを確認できます。存在しない場合、結果はfalseになります。

var docTypeAlias = sender.ContentType.Alias; 
bool hasMatch = loadedObj.Settings.Any(current => docTypeAlias.Equals(current)); 
if (hasMatch) 
{ 
    // can work 
} 
else 
{ 
    // can't work 
} 

Answer 2

あなたの代わりに、関数から返す「リターン」の（foreachの中で、コードの残りの部分をスキップし、次の項目をチェックするために）「継続」したいように見えます。

コレクションの述語を確認する必要がある場合は、FirstOrDefaultの使用を検討してください。

Answer 3

答えを記録するにはブール値を追加します。

bool found = false; 
foreach (var currentCheckBox in loadedObj.Settings.Where(currentCheckBox => currentCheckBox != null)) 
{  
    // docTypeAlias is a single string that needs to be matched 
    var docTypeAlias = sender.ContentType.Alias; 
    // This is the current value of currentCheckBox 
    var requiredTypeAlias = currentCheckBox; 
    if (requiredTypeAlias.Equals(docTypeAlias)) { 
     found = true; 
     break; 
    } 
} 
if (!found) return; 

また、この別の関数にする：私はそれをよく理解場合は、代わりにこれを使用することができると思い

private void eventhandler(object sender, EventArgs e) 
{ 
    if (!ControlIsListed(sender, loadedObj)) return; 
    // ... 
} 

Answer 4

bool doAnyMatch = loadedObj 
        .Settings 
        .Where(x => x != null) 
        .Any(x => x.Equals(docTypeAlias)); 

if(!doAnyMatch) return; 

Answer 5

：

bool ControlIsListed(object sender, MySettingsClass loadedObj) 
{ 
    foreach (var currentCheckBox in loadedObj.Settings.Where(currentCheckBox => currentCheckBox != null)) 
    {  
     // docTypeAlias is a single string that needs to be matched 
     var docTypeAlias = sender.ContentType.Alias; 
     // This is the current value of currentCheckBox 
     var requiredTypeAlias = currentCheckBox; 
     if (requiredTypeAlias.Equals(docTypeAlias)) return true; 
    } 
    return false; 
} 

し、それを呼び出します

loadedObj.Settings.Find(delegate(String currentCheckBox) 
{ 
    return sender.ContentType.Alias == currentCheckBox 
}); 

何かが見つかった場合はnullでない場合はstringを返します。