質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.61%

  • C#

    6859questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

case文の場合分けが多過ぎる問題

解決済

回答 5

投稿

  • 評価
  • クリップ 0
  • VIEW 496

tools.i

score 3

命令文が1行ごとに連なったテキストファイル(下記のPlain text)を読み込んで、
その行の命令を実行するプログラム(下記C#)を作成しています。

下記のようなプログラムだと、命令の種類が多数あるため、case文が長くなる上、対応する関数を同じファイルに書き込んでいるので、そのプログラムファイルが無関係な命令文が入り混じった冗長なものになっています。

すっきり書くような常套手段のようなものはありますか?

なお、私はC言語とc++は数年前ですが、そこそこ経験がありますが、c#でのプログラミングはこれが初めてです。

よろしくお願いします。

Set_X_Parameter, 1, 2, 0.005
Set_Y_Thickness, 10, 2.5
....
switch (cmd)
{
case "Set_X_Parameter":
    Set_X_Parameter(cmd_args);
    break;
case "Set_Y_Thickness":
    Set_Y_Thickness(cmd_args);
    break;
//
//
// たくさん続く
//
//
default:
    break;
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • sazi

    2018/05/10 10:51

    その固定値の内容が条件により収斂されるもので無い限り、固定値の分岐が有るという情報だけでは、分岐を減らすことなどできないと思います。 固定値の分岐によって何を行っているかの情報があれば、別でしょうけど。

    キャンセル

  • sazi

    2018/05/10 11:34

    固定値と呼び出す関数名が一致しているのですね。失礼しました。

    キャンセル

  • sazi

    2018/05/10 12:57

    回答の内容と被っていますが、参考までに。http://www.atmarkit.co.jp/ait/articles/0512/16/news110.html

    キャンセル

  • tools.i

    2018/05/10 14:11

    正直、さっぱり分かりません。

    キャンセル

回答 5

+5

メソッド名がわかるならリフレクションが使えます。
typeof演算子やGetTypeメソッドでTypeを取得し、GetMethodメソッドでMethodInfoを取得すれば、Invokeメソッドで呼び出すことができます。

ただし、パフォーマンスは良くないので注意してください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/10 11:41

    呼び出す度に MethodInfo を取得するならパフォーマンスは良くありませんが、キャッシュしておけば良いだけなので、私もこの方法を取ると思います。
    そうでなければ DynamicObject で動的オブジェクトを作るかもしれません。

    キャンセル

+3

他の回答者さん達の回答をまとめてコードにしました。

各関数は private static void **(string[]) で統一されていることが前提です。

static Dictionary<string, Action<string[]>> cache;

static void Init()
{
  cache = typeof(クラス名)
    .GetMethods(BindingFlags.Static | BindingFlags.NonPublic)
    .Where(mi => mi.ReturnType == typeof(Void))
    .Where(mi => mi.GetParameters().SingleOrDefault()?.ParameterType == typeof(string[]))
    .ToDictionary(
      mi => mi.Name,
      mi => mi.CreateDelegate(typeof(Action<string[]>), null) as Action<string[]>);
}

public static void InvokeCommand(string name, string[] args)
{
  if (cache.TryGetValue(name, out var cmd))
  {
     cmd(args);
  }
}

何やってるかわからないと思うので、解説を追記します。

.NET にはリフレクションという機能があり、プログラム実行時にプロパティやメソッドを検索できます。

上のコードではリフレクションを使い、 
まず static かつ public では無いメソッドを片っ端から取得します。
さらに、戻り値無し、引数が一つでその型が string[] という情報を元にコマンドだけを絞り込んでいきます。

また、この処理中に何度か配列が返ってくるのですが、これをスマートに処理する Linq という機能を使っています。
Where, SingleOrDefault, ToDictionary が Linq です。

=> の部分はラムダ式といって、デリゲート(後述)を簡単に定義する記法です。

リフレクションは負荷のかかる処理なので、デリゲート(関数ポインタのようなもの)に変換して負荷を軽くします。
これによって、各コマンドのデリゲート(関数ポインタ)が作成されます。
こんな感じです。
delegate void Action<string[]>(string[]);

switch文の処理はまるまる Dictionary.TryGetValue() で置き換えました。

難しいとは思いますが、 C# のエッセンスが詰まっているのでぜひ勉強してみてください。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/11 09:47

    勉強します。どうもありがとうございました。

    キャンセル

checkベストアンサー

+2

よくやるのは、コマンド実行のルーチンを複数に分ける仕組みを作ってやりますね
ある程度コマンド群をグループに分けて、

bool commandA(string cmd)
{
    switch(cmd){
    case "uniuni":
        うにうに
        break;
    ...
    ...
    default:
        return false;
    }
    return true;
}

bool commandB(string cmd){ ... }
bool commandC(string cmd){ ... }
bool commandD(string cmd){ ... }


// メインのコマンド実行ルーチン
void commandMain(string cmd)
{
    if(commandA(cmd)) return;
    if(commandB(cmd)) return;
    if(commandC(cmd)) return;
    if(commandD(cmd)) return;
}

これでグループごとにファイル変えたり、クラスを変えたりできるし、デバッグもしやすくなります

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/11 09:46

    勉強します。

    キャンセル

  • 2018/05/11 09:48

    とりあえず、すぐ理解できたので、僭越ながらベストアンサーということで。

    キャンセル

+2

ちょっとすっきりするかも?

public void CommandInvoke(string cmd, string[] cmd_args)
{
    Dictionary<string, Action<string[]>> commandDic = new Dictionary<string, Action<string[]>>();
    commandDic.Add("Set_X_Parameter", a => Set_X_Parameter(a));
    commandDic.Add("Set_Y_Thickness", a => Set_Y_Thickness(a));

    commandDic[cmd].Invoke(cmd_args);
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

デザイン時テンプレートとしてT4テキストテンプレートを使い、
コードを自動生成するという方法があります。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • C#

    6859questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。