動的C＃メソッドまたは複数のIf-else-ifブロック？

Question

WPFアプリケーションで複数のチェックボックスをチェックし、それに応じてコードブロックを実行するかどうかを確認する必要がある状況があります。コードブロックのすべての反復で変更される唯一のものはファイル名です。サンプルコードは次のようになります

if(checkbox1.IsChecked == true) 
{ 
    Code(fileName1); 
} 
if(checkbox2.IsChecked == true) 
{ 
    Code(fileName2); 
} 
if(checkbox3.IsChecked == true) 
{ 
    Code(fileName3); 
} 

すべてが冗長に見えます。また、メソッドを作成してファイル名とチェックボックス名を渡すことはできません。チェックボックスは文字列として渡され、IsCheckedプロパティは文字列に対して無効になります。これを克服し、コードをきれいにする方法はありますか？

Answer 1

：最後に、このようなもので、あなたの質問からコードを置き換えます辞書にチェックボックスアップ、そしてそれらをループ：

var filesByCheckbox = new Dictionary<CheckBox, string> { 
    { checkbox1, filename1 }, 
    { checkbox2, filename2 }, 
    { checkbox3, filename3 } 
}; 

foreach (var kvp in filesByCheckbox) 
{ 
    if (kvp.Key.IsChecked) 
    { 
     Code(kvp.Value); 
    } 
} 

これは本当に何を改善していないが、それはちょうどそうリストに追加することが一つだけあるファイルに添付されているチェックボックスそれは少し明確になります行の代わりに3行。

それがあなたのために働くなら、タナーの答え。

Answer 2

CheckBoxのサブクラスを使用するようにフォームを変更できますか？もしそうなら、あなたはこれを行うことができます：

class MyCheckBox : CheckBox 
{ 
    public string FileName { get; set; } 

    public void MyMethod() 
    { 
     if (IsChecked) 
     { 
      (do something with FileName); 
     } 
    } 
} 

を次にフォームデザイナでFileNameの値を設定します。あなたは手動であなたが今それを持っているのと同じ方法を文字列にチェックボックスをリンクする必要がある、とだけ短いコードを探しているなら、あなたがラップすることができ

checkbox1.MyMethod(); 
checkbox2.MyMethod(); 
checkbox3.MyMethod(); 

Answer 3

チェックボックスをファイル名にマップする必要があります。簡単な方法は、Tagプロパティを使用することです：

checkbox1.Tag = filename1; 
checkbox2.Tag = filename2; 
checkbox3.Tag = filename3; 

次に配列やプロセスにyoutはチェックボックスを置く：

var checkboxes = new [] { checkbox1, checkbox2, checkbox3 }; 

foreach (var checkbox in checkboxes.Where(cb => cb.IsChecked)) 
{ 
    Code((string)checkbox.Tag); 
}