ifステートメントまたはスイッチの長いチェーン以外に、これを行うためのよりインテリジェントな方法はありますか？

メッセージを受信する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以前のどこかで特別なケースとして扱うことができます。または、これらのテストが多数ある場合は、2番目のディスパッチテーブルを使用します）。

もちろん、どのpointer-to-functionように見えるかはプログラミング言語に依存します。ここで CまたはC ++の例です。JavaまたはC＃では、おそらくそのためにラムダ式を使用するか、コマンドパターンを使用して「関数へのポインター」をシミュレートします。無料のオンラインブック「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各1上の文字列に一致するようにしようと呼び出してmethod.invoke()、それが収まるところ。

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

インターフェースを定義すると、IChatBehaviour1つのメソッドが呼び出さExecuteれ、1つmessageのcmdオブジェクトと1つのオブジェクトが取り込まれます。

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);
}

上記はあなたが持っているコードの量を減らすはずです。上記のアプローチでは、ボットクラス自体を変更することなく、ボットクラスに追加の動作を提供することもできます（をStrategy Design Pattern参照）。

一度に1つの動作のみを起動する場合、executeメソッドのシグネチャを変更してtrue（動作が起動した）またはfalse（動作が起動しなかった）を生成し、上記のループを次のように置き換えることができます。

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

余分なクラスをすべて作成して渡す必要があるため、上記の実装と初期化はより面倒ですが、すべての動作クラスがカプセル化され、できれば相互に独立しているため、ボットを簡単に拡張および変更できるようにする必要があります。

「インテリジェント」には、（少なくとも）3つのことがあります。

より高い性能

ディスパッチテーブル（およびそれに相当するもの）の提案は良いものです。このようなテーブルは、「追加できません;試してさえいません」という意味で、過去何年も「CADET」と呼ばれていました。ただし、このテーブルを管理する方法について初心者のメンテナーを支援するコメントを検討してください。

保守性

「美しくする」というのは、怠慢ではありません。

そして、しばしば見落とされます...

弾力性

toLowerCaseの使用には落とし穴があります。一部の言語の一部のテキストは、magisculeとminisculeを切り替える際に苦痛を伴う再構築を行う必要があります。残念ながら、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はそれほど悪いとは思いません。