Чи є більш розумний спосіб зробити це, крім довгого ланцюжка, якщо заяви або перемикання?

Я реалізую бот 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намагаючись узгодити String на кожному з них і зателефонуйте, 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який називається, який приймає в a 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методу, щоб отримати 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 - це так погано.