任务:有一个以不同方式实现某些东西的类。类的用户应该只看到代表可用选项的public enum
,同时隐藏不同行为的所有实现。
为了避免在每次调用方法时检查提供的“样式”,构造函数在提供的枚举值上使用switch
来为字段分配适当的内部私有类。
这是SSCCE:
public class Greeter {
public enum GreetingStyle { HEY, HELLO }
private String name;
private GreetingChooser greetingChooser;
public Greeter(String name, GreetingStyle style) {
this.name = name;
switch(style) {
case HEY:
greetingChooser = new Hey();
break;
case HELLO:
greetingChooser = new Hello();
break;
default :
throw new UnsupportedOperationException("GreetingStyle value not handled : " + style.toString());
}
}
public void greet() {
// need to avoid switch(style) here
System.out.println(greetingChooser.greeting() + ", " + name + "!");
}
// this interface can't be public
private interface GreetingChooser {
String greeting();
}
private class Hey implements GreetingChooser {
public String greeting() {
return "Hey";
}
}
private class Hello implements GreetingChooser {
public String greeting() {
return "Hello";
}
}
public static void main(String args[]) {
new Greeter("John Doe", Greeter.GreetingStyle.HEY).greet();
new Greeter("John Doe", Greeter.GreetingStyle.HELLO).greet();
}
}
问题:这是一种很好的方法来实现这样的功能,使其在未来可维护(例如,我们需要添加GreetingStyle ALOHA
)?我的另一个想法是使用静态地图
private static final Map<GreetingStyle, GreetingChooser> greetingMap;
static {
greetingMap = new HashMap<>();
greetingMap.put(Greeter.GreetingStyle.HEY, new Hey());
greetingMap.put(Greeter.GreetingStyle.HELLO, new Hello());
}
并在构造函数中使用greetingMap.get(style)
。
注意:在Java 8中,最好用lambdas实现(如果接口只有一个函数),但我受限于Java 7.
答案 0 :(得分:2)
不是仅使用枚举作为工厂开关的常量,而是通过将枚举常量配置为任一配置来避免切换
public enum GreetingStyle
{
HEY("Hey"),
HELLO("Hello");
GreetingStyle(String text) {
this.text = text;
}
public final String text;
}
或行为:
public enum GreetingStyle
{
HEY {
public void greet() { /* performs hey style greeting*/ }
},
HELLO{
public void greet() { /* performs hello style greeting*/ }
};
public abstract void greet();
}
答案 1 :(得分:0)
为什么要绑定枚举?您使用多态性可以轻松实现您所描述的内容。如果您担心将来扩展它,可以使用Factory或Decorator等设计模式。