根据找到的字符串使用循环调用不同的方法

Question

我正在使用 Pircbotx (https://github.com/pircbotx/pircbotx/wiki/Documentation) 提供的侦听器来检测何时在 Twitch 聊天中找到命令，并且我正在尝试根据调用的命令使用不同的方法（格式为 !command）。使用的类：监听器、命令。

命令存储在一个命令对象数组中，由一个字符串（名称）组成。每个 Command 对象最终都将使用其自己的方法，该方法将在 Command 类中定义。 Listeners 对象在实例化时会立即将数组的每个元素放入哈希表（命令）中。

当侦听器检测到消息时，它会使用本地字符串变量 (msg) 进行存储。发生这种情况时，循环遍历 Command 对象数组，然后......应该调用对应于该特定对象的方法，在本例中为 Command.addDeath()。这就是我卡住的地方。

我以前为我的听众使用了一堆 if 语句，但是当有一堆命令时，事情会变得非常非常混乱。如果我的代码块中的格式很奇怪，请提前道歉，我对使用 Stackverflow 还是很陌生，而且我也是一名 Java 新手，我正在学习。再次查看代码后，似乎我并不真的需要哈希表 - 但我将它留在里面以防万一你们对如何处理它们有更好的想法。

public class Listeners {

String name;
String message;    
private static MessageEvent event;
Command [] commandNames = {new Command("!clearchat", new Command("!addDeath")}; 
Hashtable<String, Command> commands = new Hashtable<String, Command>();

 public Listeners() {
    for (int i = 0; i < commandNames.length; i++) {
        commands.put(commandNames[i].name, new Command(commandNames[i].name));
    }   
    
    if (event.getMessage() != null) {
        String msg = event.getMessage();
        for (int x = 0; x < commandNames.length; x++ ) {
            if (msg.startsWith(commandNames[x].name)) {
                // call Command method here 
            }
        }
    }   
}

这里是 Command 类：

public class Command {

String name;

public Command(String name) {
        this.name = name;
}

public static void addDeath() {
    DeathCounter.addDeath();
    Listeners.sendMessage("Death Counter: " + DeathCounter.getDeaths());
} 
}

Answer 1

您可以使用接口来执行命令：

public interface Command {
    public abstract void execute();
}

然后让你的命令实现接口：

public class DeathCommand implements Command {
    @Override
    public void execute() {
        DeathCounter.addDeath();
        Listeners.sendMessage("Death Counter: " + DeathCounter.getDeaths());
    }
}

在您的侦听器类中，将命令字符串映射到相应命令的实例：

public class Listeners {

    String name;
    String message;
    private static MessageEvent event;
    static Map<String, Command> commands = new HashMap<>();

    static {
        commands.put("!addDeath", new DeathCommand());
    }

    public Listeners() {
        if (event.getMessage() != null) {
            String msg = event.getMessage();
            Optional<String> key = commands.keySet().stream().filter(k -> msg.startsWith(k)).findFirst();
            key.ifPresent(s -> commands.get(s).execute());
        }
    }
}

我用 HashMap 替换了您的 Hashtable，这在大多数方面都更好（但使用方式相同）。

我对将地图作为静态成员持怀疑态度，但由于我不熟悉您的用例，所以我保留原样。

编辑

所有 Java 类都有一个默认构造函数（没有参数），除非您编写自己的构造函数（如果您仍然需要，则必须自己编写默认构造函数）。接口没有构造函数，因为它们不能直接实例化。他们只是指定实现类必须具有的方法。这允许您拥有接口类型的引用（命名字段/变量），并且能够在不知道或不必知道它是哪个实现的情况下调用方法。

例子：

Command com = new DeathCommand();
com.execute(); // <- this will run the execute() code in the DeathCommand class
Command com2 = new SomeOtherCommand();
com2.execute(); // <- this will run the execute() code in the SomeOtherCommand class

上面的Command代码就完成了。没有什么了。至于 DeathCommand 和其他实现，您需要添加所需的代码。

每个类和接口都放在自己的文件中，命名为类型：

Command.java
DeathCommand.java

关于 HashTable 与 HashMap。我应该说最好使用 Map 及其实现。如果您需要线程安全，请按照 agilob 指出的那样使用 ConcurrentHashMap，因为常规 HashMap 不是线程安全的。