状态模式执行困难

Question

我在我的应用程序中使用state pattern，因为我的state-entities 具有API，其行为会因当前状态而改变。

在我的情况下，依赖于状态的 API 仅由一种方法组成 - changeState。这个方法改变状态，也执行一些状态相关的逻辑。

但是有一种情况让我提出了这个问题：状态依赖逻辑不仅依赖于当前状态，还依赖于 newState（见下面的代码）。所以我必须使用instanceof。

问题： 真正摆脱instanceof的方法是什么？

public interface IStateEntity {
  void changeState(IState newState);
}

public interface IState {
  void doStateDependentLogic(IState newState);
}

public class StateEntityImpl {

  private IState state;

  @Override
  public void changeState(IState newState) {
    state.doStateDependentLogic(newState);
    state = newState;  
  }
}

public class StateImplA { //each state implementation is a singleton

  @Override
  public void doStateDependentLogic(IState newState) {
      // HARDSHIP #1: state dependent logic depends not only on current state, but also it depends on newState. So I have to use `instanceof`
     if (newState instanceof StateImplA) {
       //do one logic
     } else if (newState instanceof StateImplB) {
       //do another logic
     }
  }
}

编辑（为什么不枚举）：

如果枚举替换 getClass().getSimpleName(): 因为它是函数式编程范式。当我以依赖类名的方式创建架构时，这是出现问题的第一个迹象。在我看来，将 instanceof 更改为 enum 并不意味着要更改架构，而只是为了更改一个坏的更糟。

如果将每个 IState 实现设为枚举：每个 IState 实现都是 Hibernate 实体。

Answer 1

Java 中公认的创建单例的方式是使用enum。这提供了线程安全初始化以及 JVM 保证的单例状态。有很多方法可以创建不是 enum 的 class 的第二个实例。

使用enum 的额外好处是您可以在逻辑中打开enum 的大小写：

public enum State {

    STATE_A {
        @Override
        public void doStateDependentLogic(final State newState) {
            switch (newState) {
                case STATE_B:
                    //do stuff
                //case SOME_OTHER_STATE
            }
        }
    },
    STATE_B {
        @Override
        public void doStateDependentLogic(final State newState) {
        }
    };

    public abstract void doStateDependentLogic(State newState);
}

我不知道你从哪里得到enum 是一种函数式编程范例的想法。不是这种情况。 Java 中的enum 旨在满足您的需求。

Answer 2

您的问题实际上并不是状态模式。您的问题出在状态模式完成其工作之后，即将方法调用路由到表示状态的特定实现。

instanceof 子类问题是一个独立问题。这里的解决方案是避免instanceof 就像在其他代码中一样，最好通过方法在状态和下一个状态之间进行交互。 switch over enum types 看起来已经更好了，但并没有好多少，因为您仍然依赖于另一个对象的具体类型。可以认为没问题，因为它们都定义在同一个类中，所以依赖关系只在你自己身上。

在最好的情况下你应该得到的结果看起来像

  @Override
  public void doStateDependentLogic(IState newState) {
     if (newState.isBeautiful()) {
         System.out.println(newState.getYourPartOfTheInterAction(this));
     } else {
         somethingElse();
     }
  }

即方法调用抽象而不关心实现细节。