为继承的类寻求一种优雅的方式来拥有“StateName”属性

Question

请参阅下面的解决方案 - 搜索 UPDATE。

我有一个广泛的状态机架构，我通过为每个状态创建一个类来实现它（有多个机器，但所有的状态都继承自同一个“MachineState”类）。每个状态都有一个静态属性“StateName”：

public class SomeState: MachineState
{

  // THIS BLOCK SHOULD BE COPIED TO ALL STATE CLASSES!!
  private static string _StateName;
  public static string StateName
  { 
     get {
           if (_StateName == null)
           { 
               _StateName = MethodBase
                  .GetCurrentMethod()
                  .DeclaringType
                  .ToString()
                  .Split(new char[] { '.' })
                  .ToList()
                  .Last();
           }
           return _StateName; 
          }  
  }
  // END OF BLOCK


  public SomeState(Queue<string> messages)  // 
        : base(messages)
  {
       ...
  }
...
}

呃。

至少我会调用处理器密集型的东西，以便每个类只获得一次名称——就我的目的而言，这是可以接受的成本。但我真的很想为他们找到某种方式来“继承”这段代码——或者至少是某种方式让他们将它包含在宏之类的东西中。我有一个抽象属性，所以如果它没有实现，我会在编译时捕获它；但是，仍然必须有一种方法可以避免将这种混乱复制到每个班级中——然后如果需要，必须在每个班级中更改它。

有什么想法吗？

谢谢。

------------ 更新 ---------------------- -----

生活充满了妥协；这个我可以忍受。 @Tallek 在基类中提出了这个建议：

 public static string GetStateName<T>() where T : MachineState
    {
        return typeof(T).Name;
    }

我将它与我的静态属性集成，如下所示（对于“SomeState”类）：

    // THIS BLOCK SHOULD BE COPIED TO ALL STATE CLASSES!!
    public static string StateName { get { return GetStateName<SomeState>(); } }

它并不完美；我必须确保在每个类的 GetStateName 调用中获得正确的州名。但它做了两件我急于做的事情：将逻辑移动到一个位置，并且更易于阅读。保持 StateName 抽象将帮助我捕获任何尚未实现 StateName 的状态。

再次感谢大家。

Answer 1

• 你有一个状态
• 你的状态是类
• 您想将该状态与另一个状态进行比较
• 您不想为了比较而实例化状态

我不认为你这样做比：

if(state.GetType() == typeof(SomeState))

Answer 2

它不是很好，但也许值得考虑......

public class MachineState
{
   public static string GetStateName<T>() where T : MachineState
   {
       return typeof(T).Name;
   }

}

这样使用：

if("MyState" == MachineState.GetStateName<MyState>()) { ... }

Answer 3

您可以使用扩展方法完成此操作：

public static string GetStateName(this MachineState state) {

  return state.GetType().Name;

}

像这样使用它：

if(state.GetStateName() == "SomeState") { /* Do something */ }

如果您愿意，可以使用自己的缓存：您可以在此处访问所需的任何静态结构。

Answer 4

我可能同意 CodeCaster。对于枚举而言，20 或 30 个州并没有那么大。

根据您对接收消息并识别该消息的处理程序的描述，并结合查看您的示例：

if(stateName == SomeState.StateName) { }

这意味着您将stateName 作为参数。所以你对每个状态都有一个 if 块，这样你就可以确定消息适用于哪个状态？

if(stateName == SomeState.StateName) { 

}
if(stateName == OtherState.StateName) { 

}

如果是这样的话...... （如果给出有限的用例信息，那么这个答案的其余部分是基于该前提的，所以如果其余部分不适用，请不要激怒我）

您希望所有类都自动具有此 StateName 属性。这看起来很干，但是我们看到您仍然必须为每个状态设置一个 if 块，因为该 IMO 有更多代码，所以干的更少。您已将一个 DRY 换成另一个 DRY。

我会有枚举，每个枚举都有一个

public enum States {
...
[Handler(typeof(SomeState))]
SomeState = 5,
...

结合工厂模式，现在你扔掉所有的 if 块，只需要调用你的工厂：

MachineState newState = StateFactory.Create(stateName);

工厂使用Enum.Parse将stateName转换为枚举，从中访问属性获取需要实例化的类型。 不需要 switch/case/if/else。

这意味着每次你实现一个新的状态类，你只需要接触一个地方，那就是枚举，并且代码重复最少。

如果每个 if 块中都有针对该特定状态的特定逻辑 将该代码移动到在 MachineState 或 IMachineState 接口中定义的 HandleMessage 方法中，该方法为每个 SomeState 执行特定于该状态的操作。假设您的消息指示 stateName，并且消息中可能有一些“内容”或“数据”需要处理：

MachineState newState = StateFactory.Create(stateName);
newState.HandleMessage(messageContent);

我意识到它可能比这更复杂。您可能需要将状态与状态处理分离到单独的类中以使其正常工作。很难说。如果我站在你的立场上，我肯定会认真考虑这一点。