Есть ли более разумный способ сделать это, кроме длинной цепочки операторов 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 # вы, вероятно, будете использовать лямбда-выражения для этой цели, или вы будете имитировать «указатель на функции», используя шаблон команды. В бесплатной онлайн-книге « 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);
        }
    }
}

И так далее для всего остального.

В вашем основном классе у вас есть список поведений ( List<IChatBehaviour>), которые реализует ваш IRC-бот. Затем вы можете заменить свои ifзаявления чем-то вроде этого:

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

Выше должно уменьшить количество кода, который у вас есть. Вышеуказанный подход также позволит вам предоставить дополнительные варианты поведения вашему классу ботов без изменения самого класса ботов (согласно Strategy Design Pattern).

Если вы хотите, чтобы одновременно запускалось только одно поведение, вы можете изменить сигнатуру executeметода на yield true( falseповедение сработало ) или (поведение не сработало) и заменить приведенный выше цикл следующим образом:

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

Вышеприведенное было бы более утомительным для реализации и инициализации, поскольку вам нужно создавать и передавать все дополнительные классы, однако это должно сделать вашего бота легко расширяемым и модифицируемым, поскольку все ваши классы поведения будут инкапсулированы и, будем надеяться, независимы друг от друга.

«Интеллигентным» может быть (как минимум) три вещи:

Более высокая производительность

Таблица рассылки (и ее эквиваленты) является хорошим предложением. Такая таблица называлась "CADET" в прошлые годы для "Не могу добавить; даже не пытается". Однако, рассмотрите комментарий, чтобы помочь начинающему сопровождающему только о том, как управлять упомянутой таблицей.

Ремонтопригодность

«Сделай это красиво» - не пустое наставление.

и часто упускают из виду ...

упругость

Использование 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. Затем вы идете в метод каждой группы, на этот раз вы получите оригинальную команду.

Это сделает это более управляемым. Больше пользы, когда число «еще, если» растет слишком сильно.

Конечно, иногда иметь это «если бы еще» не было большой проблемой. Я не думаю, что 20 это так плохо.