C# - Автоматическая инициализация полей класса через конструктор (ручной и динамический)

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); 
     } 
    } 
} 

Однако класс команд, как ожидается, будет инициализирован много раз (игнорировать имена, если это мешает вам получить идею), поэтому производительность потеря в какой-то момент может быть эффективной.

Есть ли альтернатива этому конкретному дизайну, который я использую? Моя задача - автоматически инициализировать каждое поле ICommand внутри конструктора класса Commands. Ручной подход подвержен ошибкам, которые могут быть слишком запоздалыми при обнаружении.

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); 
     } 
    }