使用反射而不是长的 switch 语句

Question

嘿，所以我有一个很长的 switch 语句，它调用一个基于这样的字符串的方法

private void callMethod(String name) {
    switch(name) {
        case "blah":
            blah();
            break;
        case "method":
            method()
            break;
    }
}

等等

所以我用反射来缩短代码

try {
    Method method = ClassName.class.getDeclaredMethod(name, args);
    method.invoke(args);
} catch (NoSuchMethodException e) {
    // switch default
}

只是想知道使用这样的反射是否对内存/性能不利。我只是偶尔调用这个函数。

编辑：我使用它的原因是因为用户可以选择根据他们输入的字符串来启动游戏的不同部分。启动事件的每个方法都以他们需要输入的内容命名。

Answer 1

反思通常是最糟糕的做事方式。它很慢，无法在运行时进行优化，编译器也无法标记错误。而且正如 kpie 所指出的，让用户输入直接控制执行哪个方法是一个安全漏洞。

任何长的switch 语句都可以替换为Map。在您的情况下，Map<String, Runnable> 就足够了：

Map<String, Runnable> actions = new HashMap<>();
actions.put("blah", this::blah);
actions.put("method", this::method);

// ...

Runnable action = actions.get(name);
if (action == null) {
    throw new IllegalArgumentException("Invalid name: " + name);
}
action.run();

Answer 2

像这样使用反射是个坏主意。查看here 了解有关缺点的信息。

关于性能：

“由于反射涉及动态解析的类型，某些Java虚拟机优化无法执行。因此，反射操作的性能比非反射操作慢，应避免在经常调用的代码段中性能敏感的应用程序。”

不要将反射视为String 或ArrayList 之类的普通工具，而应尝试实现适合您需求的更好的软件设计。那么你将不得不很少考虑反射。

---

编辑：

性能损失当然值得商榷，但请注意其他缺点。正如其他人已经说过的那样，存在安全问题，例如当用户“调用”一个不打算由用户直接“调用”的方法时。另一个缺点是您的代码中可能存在由于反射而在编译时无法找到的问题。还有更多的缺点...

在我看来，反射应该主要用于解决与生产代码没有直接关系的技术问题，例如在测试您的软件时。