コンストラクタによるクラスフィールドの自動初期化

Question

私のデザインには影響しませんが、コードとパフォーマンスの将来の変更に影響するケースを見つけました。

私のクラスは以下の通りです：

public sealed class Commands { 
public interface ICommand { 
    string commandName { get; set; } 
} 

public ICommand Maintain; 
public ICommand Dispatch; 
public ICommand Memorize; 
//............// 

private class ReckonCommand : ICommand { 
    public string commandName { get; set; } 

    public ReckonCommand (string name) { 
     this.commandName = name; 
    } 
} 

public Commands() { 
    Maintain = new ReckonCommand ("Maintain") as ICommand; 
    Dispatch = new ReckonCommand ("Dispatch") as ICommand; 
    Memorize = new ReckonCommand ("Memorize") as ICommand; 
} 

このアプローチは、コマンドクラスで各ICommandの変数を初期化し、手動で必要になります。したがって、フィールドがより多くなると、維持するのが難しくなります。

代わり、動的なアプローチは以下の通りです：

public ICommand Maintain; 
public ICommand Dispatch; 
public ICommand Memorize; 
//............// 

private class ReckonCommand : ICommand { 
    public float commandName { get; set; } 
    //Stuff 

} 

public Commands() { 
    System.Reflection.FieldInfo[] _fields = GetType().GetFields(); 
    for (int i = 0; i < _fields.Length; i++) { 
     if (_fields [i].FieldType == typeof(ICommand)) { 
      _fields [i].SetValue (this, new ReckonCommand() as ICommand); 
     } 
    } 
} 

ただし、コマンドクラスが何回も（これはアイデアを得ることからあなたを妨げている場合、名前を無視）、そのパフォーマンスを初期化することが期待されています損失は​​ある時点で影響を受ける可能性があります。

私が使用しているこの特定のデザインの代替品はありますか？私の懸念は、Commandクラスのコンストラクタ内のすべてのICommandフィールドを自動的に初期化することです。手作業ではエラーが発生する可能性があります。

Answer 1

動的アプローチは完全に受け入れられます（大規模アプリケーションでは、説明したように問題を解決するためにすべてを使用します）。

ただし、クラスの静的コンストラクタで検出を実装し、その結果を静的コレクションに格納して、クラスのインスタンスが作成されるたびにリフレクションを実行する必要はありません。線に沿って

何か：

private static System.Collections.Generic.List<System.Reflection.FieldInfo> _fields; 

    static Commands() 
    { 
     _fields = new System.Collections.Generic.List<System.Reflection.FieldInfo>(); 

     var fields = typeof(Commands).GetFields(); 
     var commandType = typeof(ICommand); 

     foreach (var field in fields) 
     { 
      if (field.FieldType == commandType) 
      { 
       _fields.Add(field); 
      } 
     } 
    } 

    public Commands() 
    { 
     foreach (var field in _fields) 
     { 
      field.SetValue(this, new ReckonCommand() as ICommand); 
     } 
    }