在Java中使用Enum工厂是最佳实践吗?

时间:2013-07-10 21:36:29

标签: java enums factory

Java允许我们在Enum上嵌入数据和行为。 我不想直接在Enum上实现工厂,因为我认为这不是它的作用。

但是我可以在枚举上放置类引用,并在外部工厂上构造对象。与传统工厂模式相比,您的最佳实施方案是什么?在哪种情况下最好使用哪种解决方案?

现在,代码。

两种解决方案中用于构造对象的函数。如果需要,可以使用Map实现fly-weight模式。

private Action getAction(Class<? extends Action> actionClazz) {
    // logger + error handling
    return actionClazz.newInstance();
}

1)有一家传统工厂:

public enum ActionEnum {
    LOAD_DATA,
    LOAD_CONFIG;
}

public Action getAction(ActionEnum action) {
    switch (action) {
    case LOAD_CONFIG:
        return getAction(ActionLoadConfig.class);
    case LOAD_DATA:
        return getAction(ActionLoadData.class);
    }
}

2)使用Enum风格的工厂:

public enum ActionEnum {
    LOAD_DATA(ActionLoadConfig.class),
    LOAD_CONFIG(ActionLoadData.class);

    public ActionEnum(Class<? extends Action> clazz){...}
    public Class<? extends Action> getClazz() {return this.clazz}
}

public Action getAction(ActionEnum action) {
    return getAction(action.getClazz());
}

4 个答案:

答案 0 :(得分:15)

第二个更干净:它不需要任何长开关块,并且有忘记其中一个枚举值的风险,就像第一个那样。

但并不总是可以使用它,因为枚举可能是一些通用枚举(例如Month),不应该与动作工厂耦合。

答案 1 :(得分:11)

如果可能的话,应该避免IMO调用newInstance(),因为它公然违反了java给出的一些编译时保护(读取它的javadoc)并引入了新的Exception来处理。

这是一个类似于what Sergey provided的解决方案,由于功能接口和方法引用,更简洁一点。

public enum ActionEnum {
  LOAD_DATA(ActionLoadData::new),
  LOAD_CONFIG(ActionLoadConfig::new)

  private Supplier<Action> instantiator;

  public Action getInstance() {
    return instantiator.get();
  }

  ActionEnum(Supplier<Action> instantiator) {
    this.instantiator = instantiator;
  }
}

public Action getAction(ActionEnum action) {
  return action.getInstance();
}

答案 2 :(得分:9)

这对我有用:

 enum ActionEnum
    {
      LOAD_DATA {

        @Override
        public ActionLoadData getInstance() {
            return new ActionLoadData ();
        }

    },
    LOAD_CONFIG {

        @Override
        public ActionLoadConfig getInstance() {
            return new ActionLoadConfig();
        }

    };

    public abstract ILightBulb getInstance();
}

class ActionFactory
{
    public  Action getAction(ActionEnum action)
    {
       return action.getInstance();
    }
}

答案 3 :(得分:7)

要解耦更多:

static final EnumMap<ActionEnum, Class<? extends Action>> enumToClass = new EnumMap<>();
static
{  
    enumToClass.put(ActionEnum.LOAD_DATA, ActionLoadData.class);
    etc...
}


public Action getAction(ActionEnum action) 
{
    return getAction(enumToClass.get(action));
}

EnumMap非常快,所以不用担心。