[英]How to simplify huge switch-case expression?
我在处理下一个代码时遇到了一些麻烦。 我有简单的界面,如:
public interface Game {
int start();
}
许多实现此接口的类,例如:
public class FirstGame implements Game {
public static final int ID = 1;
@Override
int start() {
// Do something and return result
}
}
和 GameManager 类有一个这样的方法:
public Game getGameById(int id) {
switch(id) {
case FirstGame.ID:
return new FirstGame();
case SecondGame.ID:
return new SecondGame();
// ..... and many other cases....
}
return null;
}
我试图使用像这样的反射来简化这个 switch-case 结构:
@Target(ElementType.TYPE)
@Retention(RetentionPolicy.RUNTIME)
public @interface GameId {
long value() default 0;
}
FirstGame类:
@GameId(value = 1)
public class FirstGame implements Game {
public static final int ID = 1;
@Override
int start() {
// Do something and return result
}
}
和 GameManager 方法是这样的:
public Game getGameById(int id) {
Game game = null;
try {
Reflections reflections = new Reflections();
for (Class<?> clazz : reflections.getTypesAnnotatedWith(GameId.class)) {
if (clazz.getAnnotation(GameId.class).value() == id) {
Constructor constructor = clazz.getConstructor();
game = (Game) constructor.newInstance();
break;
}
}
} catch (Exception ex) { ex.printStackTrace();}
return game;
}
但它的工作速度太慢了。 那么,如何以其他方式简化 switch-case 表达式? 感谢和抱歉我的英语不好。
这个怎么样?
static final List<Supplier<Game>> GAMES = List.of(
FirstGame::new,
SecondGame::new
// ...
);
public Game getGameById(int id) {
return GAMES.get(id).get();
}
或者
static final Map<Integer, Supplier<Game>> GAMES = Map.of(
1, FirstGame::new,
2, SecondGame::new
);
public Game getGameById(int id) {
return GAMES.get(id).get();
}
感谢 Holger 的回答。 我的反思方法非常接近成功。 它只需要将所有构造函数添加到 HashMap 一次,其中 ID 用作键。 所以现在我的 GameManager 代码如下所示:
Map<Integer, Constructor> constructorMap = new HashMap<>();
private void fillConstructorMap() {
try {
ConfigurationBuilder configurationBuilder = new ConfigurationBuilder();
configurationBuilder.addUrls(ClasspathHelper.forPackage("com.my.package.name"));
Reflections reflections = new Reflections(configurationBuilder);
for (Class<?> clazz : reflections.getTypesAnnotatedWith(GameId.class)) {
Integer id = clazz.getAnnotation(GameId.class).value();
Constructor constructor = clazz.getConstructor();
constructorMap.put(id, constructor);
}
} catch (NoSuchMethodException ex) {
ex.printStackTrace();
}
}
public Game getGameById(int id) {
Game game = null;
try {
game = (Game) constructorMap.get(id).newInstance();
} catch (Exception ex) {ex.printStackTrace();}
return game;
}
这种方法与使用 switch-case 表达式的方法一样快。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.