除了一连串的if语句或switch之外，还有其他更智能的方法吗？

我正在实现一个接收消息的IRC机器人，并且正在检查该消息以确定要调用的函数。有更聪明的方法吗？在我掌握了20条命令之后，它似乎很快就变得一发不可收拾。

也许有一种更好的抽象方法？

 public void onMessage(String channel, String sender, String login, String hostname, String message){

        if (message.equalsIgnoreCase(".np")){
//            TODO: Use Last.fm API to find the now playing
        } else if (message.toLowerCase().startsWith(".register")) {
                cmd.registerLastNick(channel, sender, message);
        } else if (message.toLowerCase().startsWith("give us a countdown")) {
                cmd.countdown(channel, message);
        } else if (message.toLowerCase().startsWith("remember am routine")) {
                cmd.updateAmRoutine(channel, message, sender);
        }
    }

使用调度表。这是一个包含对（“消息部分”，pointer-to-function）的表。然后，调度程序将如下所示（使用伪代码）：

for each (row in dispatchTable)
{
    if(message.toLowerCase().startsWith(row.messagePart))
    {
         row.theFunction(message);
         break;
    }
}

（equalsIgnoreCase可以在以前的某个地方处理特殊情况，或者如果您有很多测试，则可以使用第二个分派表进行处理）。

当然，pointer-to-function外观必须取决于您的编程语言。这是C或C ++中的示例。在Java或C＃中，您可能会为此目的使用lambda表达式，或者通过使用命令模式来模拟“指向函数的指针”。免费的在线书籍“ Higher Order Perl ”有完整的章节，介绍了使用Perl的调度表。

我可能会做这样的事情：

public interface Command {
  boolean matches(String message);

  void execute(String channel, String sender, String login,
               String hostname, String message);
}

然后，您可以让每个命令实现此接口，并在与消息匹配时返回true。

List<Command> activeCommands = new ArrayList<>();
activeCommands.add(new LastFMCommand());
activeCommands.add(new RegisterLastNickCommand());
// etc.

for (Command command : activeCommands) {
    if (command.matches(message)) {
        command.execute(channel, sender, login, hostname, message);
        break; // handle the first matching command only
    }
}

您正在使用Java-使其美观；-)

我可能会使用注释来做到这一点：

1. 创建自定义方法注释

   @IRCCommand( String command, boolean perfectmatch = false )

2. 将注释添加到类中的所有相关方法中，例如

   @IRCCommand( command = ".np", perfectmatch = true )
   doNP( ... )

3. 在构造函数中，使用Reflections从类中所有带注释的方法中创建方法的HashMap：

   ...
   for (Method m : getDeclaredMethods()) {
   if ( isAnnotationPresent... ) {
       commandList.put(m.getAnnotation(...), m);
       ...

4. 在您的onMessage方法中，只需遍历一个循环，以commandList尝试匹配每个字符串，然后调用method.invoke()适合的字符串即可。

   for ( @IRCCommand a : commanMap.keyList() ) {
       if ( cmd.equalsIgnoreCase( a.command )
            || ( cmd.startsWith( a.command ) && !a.perfectMatch ) {
           commandMap.get( a ).invoke( this, cmd );

如果定义一个接口，说说IChatBehaviour有一个调用了一个方法Execute并接受message一个cmd对象的方法，该怎么办？

public Interface IChatBehaviour
{
    public void execute(String message, CMD cmd);
}

然后，在代码中实现此接口并定义所需的行为：

public class RegisterLastNick implements IChatBehaviour
{
    public void execute(String message, CMD cmd)
    {
        if (message.toLowerCase().startsWith(".register"))
        {
            cmd.registerLastNick(channel, sender, message);
        }
    }
}

其余的依次类推。

然后，在您的主类中，您会看到IRC机器人实现的行为（List<IChatBehaviour>）列表。然后，您可以将if语句替换为以下内容：

for(IChatBehaviour behaviour : this.behaviours)
{
    behaviour.execute(message, cmd);
}

以上应该减少您拥有的代码量。通过上述方法，您还可以向bot类提供其他行为，而无需修改bot类本身（按照Strategy Design Pattern）。

如果您一次只希望触发一种行为，则可以将execute方法的签名更改为yield true（该行为已触发）或false（该行为未触发），然后将上述循环替换为以下内容：

for(IChatBehaviour behaviour : this.behaviours)
{
    if(behaviour.execute(message, cmd))
    { 
         break;
    }
}

由于您需要创建和传递所有额外的类，因此上述实现和初始化会更加繁琐，但是，由于所有行为类都将被封装并且希望彼此独立，因此它应该使您的机器人易于扩展和修改。

“智能”可以是（至少）三件事：

更高的性能

调度表（及其等效表）建议是一个很好的建议。这样的表在过去几年中被称为“无法添加；甚至无法尝试”。但是，请考虑使用注释来帮助新手维护者如何管理所述表。

可维护性

“让它变得美丽”绝非空谈。

而且，经常被忽视...

弹性

toLowerCase的使用存在一个陷阱，即某些语言的某些文本在微妙和微妙的转换之间必须经过痛苦的重组。不幸的是，toUpperCase存在相同的陷阱。请注意。

您可以让所有命令实现相同的接口。然后，消息解析器可能会向您返回仅执行的适当命令。

public interface Command {
    public void execute(String channel, String message, String sender) throws Exception;
}

public class MessageParser {
    public Command parseCommandFromMessage(String message) {
        // TODO Put your if/switch or something more clever here
        // e.g. return new CountdownCommand();
    }
}

public class Whatever {
    public void onMessage(String channel, String sender, String login, String hostname, String message) {
        Command c = new MessageParser().parseCommandFromMessage(message);
        c.execute(channel, message, sender);
    }
}

看起来只是更多代码。是的，您仍然需要解析消息以知道要执行哪个命令，但是现在它位于正确定义的位置。它可以在其他地方重用。（您可能想注入MessageParser，但这是另一回事。另外，根据您希望创建的数量，Flyweight模式对于这些命令可能是个好主意。）

我要做的是：

1. 将您拥有的命令分组。（您现在至少有20位）
2. 在第一级，按组进行分类，因此您具有与用户名相关的命令，歌曲命令，计数命令等。
3. 然后进入每个组的方法，这一次您将获得原始命令。

这将使它更易于管理。如果“ else if”的数量增加太多，则会获得更大的收益。

当然，有时候拥有这些“如果不是”不是什么大问题。我认为20没有那么糟糕。