このコードを短くする方法はありますか？

Question

//list the controls from the main form 
foreach (Control c in Controls) 
{ 
    if (c is ComboBox) 
    { 
     ((ComboBox)c).SelectedIndex = -1; 
    } 
    else if (c is TextBox) 
    { 
     ((TextBox)c).Text = ""; 
    } 
    else if (c is CheckBox) 
    { 
     ((CheckBox)c).Checked = false; 
    } 
    //etc. with FIFTY different types to check against 
} 

Answer 1

このメソッドを使用：

public void Set(object obj, string property, object value) 
{ 
    //use reflection to get the PropertyInfo of the property you want to set 
    //if the property is not found, GetProperty() returns null 
    var propertyInfo = obj.GetType().GetProperty(property); 
    //use the C# 6 ?. operator to only execute SetValue() if propertyInfo is not null 
    propertyInfo?.SetValue(obj, value); 
} 

はこのようにそれを呼び出します。

foreach (Control c in Controls) 
{ 
    Set(c, "SelectedIndex", -1); 
    Set(c, "Text", ""); 
    Set(c, "Checked", false); 
} 

Answer 2

一つの方法は、dynamicにキャスト特定のタイプのための3つのオーバーロードされたメソッドを追加し、このような呼び出しを行うことです。

foreach (dynamic c in Controls) { 
    ClearOut(c); 
} 
... 
private static void ClearOut(ComboBox c) { 
    c.SelectedIndex = -1; 
} 
private static void ClearOut(TextBox t) { 
    t.Text = string.Empty; 
} 
private static void ClearOut(CheckBox c) { 
    c.Checked = false; 
} 

cがdynamicあるので、C＃はランタイムするClearOut方法の結合を延期します、きれいなコードを提供します。このアプローチの欠点は、オーバーロードの1つが欠けていると、C＃がコンパイル時に通知できないことです。あなたのコントロールのプロパティを設定するには

Answer 3

あなたがのコントロールをクリアアクションにサポートされている各タイプから、ルックアップを作成することができますそのタイプに対応するタイプごとにその参照にハンドラを追加することができます：

public class ControlClearer 
{ 
    private static Dictionary<Type, Action<Control>> lookup = new Dictionary<Type, Action<Control>>(); 

    static ControlClearer() 
    { 
     AddMapping((TextBox control) => control.Text = ""); 
     AddMapping((ComboBox control) => control.SelectedIndex = -1); 
     AddMapping((CheckBox control) => control.Checked = false); 
    } 

    private static void AddMapping<T>(Action<T> clearAction) 
     where T : Control 
    { 
     lookup[typeof(T)] = control => clearAction((T)(object)control); 
    } 

    public static void Clear<T>(T control) 
     where T : Control 
    { 
     //todo support case where T isn't in the dictionary 
     lookup[typeof(T)](control); 
    } 

    public static void Clear(Control control) 
    { 
     //todo support case where the type isn't in the dictionary 
     lookup[control.GetType()](control); 
    } 
}