コンソールアプリケーション内の独自コマンド作成について

Question

現在DiscordのBOTをC#で開発している者です。
BOTを開発するにあたってBOTの挙動をある程度起動中に変更できるように
Console.ReadLine() で待機させてBOTアプリケーション内の独自コマンドを受け付けるような実装をしているのですが、
デバッグで使用する早期の必要性からコードの見栄えより機能の実現を優先したため、
単純にswitch文で分岐させてcaseには対応コマンドをstringでそのまま実装しているのでコードがかなり酷い状態になっています。（機能自体は正常に動いています）
開発のためにDiscord APIの英語コミュニティにも通っているのですが、案の定「そのswitch文酷すぎワロタ」（意訳）と言われてしまい・・
一応ここに質問する前にググったのですが、意外とアプリケーション独自のCUIコマンドの実装を紹介しているところがないんですね。。
皆さんならどのように独自のCUIコマンドを実装するのでしょうか？

仕様としては
・Console.ReadLine()でコマンドの入力を受け付ける
・何も入力しなかったり誤ったコマンドを入力しEnterすると画面のみ更新される
・対応するコマンドを入力するとそのコマンドに対応した処理を実行する

が満たせれば十分です。
よろしくお願いします。

以下現状の酷いコードです。。。

public static async Task StartConsole()
        {
            try
            {
                while (true)
                {
                    string input;
                    input = Console.ReadLine();
                    string savewindowtimes_id;
                    switch (input)
                    {
                        default:
                            Program.isShowLog = false;
                            Console.Clear();
                            VersionHeader();
                            Console.WriteLine(SystemMessageDefine.CommanderModeGuide_JP);
                            break;

                        case "showlog":
                            Console.Clear();
                            VersionHeader();
                            Program.isShowLog = true;
                            Console.WriteLine(SystemMessageDefine.LogShowMode_JP);
                            Console.WriteLine(SystemMessageDefine.CommandAccepting_JP);
                            break;
                        case "quit":
                            Console.Clear();
                            VersionHeader();
                            Environment.Exit(0);
                       (以下延々と続く）

Answer 1

コマンドごとに関数を用意して、コマンド（文字列）と関数ポインタのマップを作ると見通しが良くなるように思います。

１）コンソールからコマンドを入力する
２）入力された文字列が、マップのKeyに含まれていなければ
defaultの関数を呼び出す。　(if)
マップのkeyに含まれていれば、
マップから関数ポインタを得て実行する。(else)

というのがメインループの中身。

ループの前にマップを作るコードが入り、コードの後半はそれぞれのコマンドで実行する内容を関数として記述したものを列挙します。

こうしておけば、コマンドを追加するのは関数の追加とマップの修正だけで済みますし、１つのコマンドの機能を修正する際に変更する箇所が1つの関数だけになるのでコードの管理が楽です。

Comments

[yushistr]

ありがとうございます。参考にさせていただきます。

Answer 2

using System;
using System.Collections.Generic;

namespace Wandbox
{
    class Program
    {
        static void Main(string[] args)
        {
          var table = new Dictionary<string, Action<string, string>>();
          // コマンドの追加
          table["test"] = (user, msg) => {Console.WriteLine($"{user}: {msg}");};
          // コマンドを呼び出してみる
          table["test"]("Tom", "Hello");
        }
    }
}

こんな感じ

Comments

[yushistr]

Dictionaryでの管理は自分では思いつきませんでした。こちらも参考にさせていただきます。

Answer 3

プログラムを見やすくする、というのは、いかに分割するか、に尽きます

現状ではtryがあってwhileがあってswitchがあって、と理解しようとするとなかなか骨が折れます。
そこで、まずはswitch 文を丸々べつ関数に切り出してしまいます（文字列(input)を引数とする関数となりますか）。
そうすると、StartConsole関数は、文字列入力してなんかさせる関数に渡すだけの関数、とひと目でスッキリ理解できるようになります
なんかさせる関数、の方でも、caseが多くなったりその中の処理がややこしくて長文になったりすると、これまたややこしい関数になってしまうので、その処理もなんかする関数にして切り出していきます。

こうしていくと、処理を考えるにしても、余計な細かいことまでアタマに入れなくて済むので楽になります

当然のことながら、考えなしに関数増やしてしまうと、これまたややこしくなってしまいますがｗ

Comments

[yushistr]

こちらも設計の参考にさせていただきます。どうもです。