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

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

ただいまの
回答率

90.53%

  • C#

    7066questions

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

循環的複雑度を下げる

解決済

回答 1

投稿 編集

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

junkan

score 3

 質問

循環的複雑度を下げようとしているのですが、IFELSE文、SWITCH CASE文を連ねていると簡単に上がってしまいます。
条件分岐を分割するわけにもいかずどうすればいいか困っています。どうすればよいのでしょうか。

 サンプルソース

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace ConsoleApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            string[] Programmers =
            {
                "Java",
                "CSharp",
                "VBnet",
                "CSharp"
            };

            foreach (var p in Programmers)
                Coding(p);

            Console.ReadKey();
        }

        private static void Coding(string Language)
        {
            switch (Language)
            {
                case "Java":
                    Console.ForegroundColor = ConsoleColor.Red;
                    Console.WriteLine("Java Coding");
                    break;
                case "CSharp":
                    Console.ForegroundColor = ConsoleColor.Blue;
                    Console.WriteLine("C# Coding");
                    break;
                case "VBnet":
                    Console.ForegroundColor = ConsoleColor.Green;
                    Console.WriteLine("VB.NET Coding");
                    break;
            }
        }
    }
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • SurferOnWww

    2018/07/08 10:33

    具体例を示された方がレスしやすいと思うのですが。どういうコードで、質問者さんとしては何が問題と思っていて、どう改善したいかなど。

    キャンセル

  • junkan

    2018/07/08 14:03

    すみません。サンプルのソースコード載せました。

    キャンセル

  • SurferOnWww

    2018/07/08 15:16

    > サンプルのソースコード載せました。 ←そのサンプルは Chironian さんの回答の 2. からリンクが張られている記事のもので、継承と多態性を利用した対応策もその記事に書かれているのですけど。それは質問者さんの求める答えになってないということですか? Chironian さんの回答にレスしていただくのがよさそうです。

    キャンセル

回答 1

checkベストアンサー

+4

こんにちは。

循環的複雑度は初めてみました。要するにプログラム中の実行経路の組み合わせの数が増えると複雑化するという概念のようですね。であれば、条件分岐を減らす以外に循環的複雑度を下げることはできないように感じます。

条件分岐を減らすのは難しいですが、減らせる場合があるのも確かですので、それらを地道に適用するのが良いと思います。

パッと思いつく比較的汎用なテクニックとしては以下があります。

  1. 例外を使うことで、エラー条件判定用if文を大幅に減らせます。
  2. 種類判定のswitch-caseが2セット以上ある場合は、動的ポリモーフィズムのテクニックで1~2セットまで減らせます。
  3. 複数の種類の関数を登録するタイプの関数テーブルも有用な筈です。(2.と本質的には同じものですが、見た目がかなり異なりますので分けました。)
  4. ジェネリクスも効果的です。(型の違いだけで同様なコードを2つ以上書くなら、ジェネリクスでまとめれば、コード中の条件分岐をまとめることができます。)

例外は結構使い方が難しい上に結構面倒です。ですが、使えるようになると効果は絶大です。
2.、3.は、switch-caseが1セットしか無い時に適用してしまうと、コードが却って複雑化することが多いのでよく検討が必要です。(本当に複数セットへ拡張しますか?)

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/07/08 15:04

    回答ありがとうございます。
    2番を採用しました。

    キャンセル

  • 2018/07/08 15:29 編集

    補足
    私が持っているソースは長文のため、
    サンプルのソースはChironianさんの紹介した記事を利用しました。
    元の記事はEnumでしたが、私のソースはString型をSwitchの条件分岐に使用していたため
    合わせて加工しております。

    キャンセル

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

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

関連した質問

  • 受付中

    計算機のプログラム

    初めて質問投稿します。質問に対して不足している情報などあるかもしれませんが、ご容赦ください。 Javaで簡単な計算機の仕組みを作ろうとしています。 しかし、いざコンパイルしようと

  • 受付中

    10進数から16進数に変換

    10進数を入力すれば16進数に変換されるというプログラムを作っているのですがいまいちうまくいきませんどこがまちがっているのでしょうか? import java.io.IOExce

  • 受付中

    社員情報のプログラム

    社員情報のプログラム (JAVA) プログラの機能 (1)社員情報の追加 入力項目としては、社員番号、氏名(性、名)、生年月日(年、月、日) (3)で読み込んだ情報を追加する仕

  • 解決済

    メソッドを使ったプログラムを作りたいです

    前提・実現したいこと Javaのメソッドを使う問題を解いています。 問題は以下の通りです。 ↓ 好物を元に動物の名前と特徴を出力するプログラム  ・作成するクラス 

  • 解決済

    【JAVA】switch文を使って0~9までの値だけ出力させたいです

    package java6; public class J_6_2 { public static void main(String args[]) {

  • 受付中

    java で入力待ちのプログラムの実行を保存して自動で実行できる方法はありますか

    •eclipse上で標準入力からユーザー入力を受け付けるあるプログラムAがある  •Aを起動して例えば数行の入力をした段階でいったんAを終了させる  •Aを再度起動したときに前

  • 解決済

    javaの配列に文字を格納して処理する方法

     疑問、質問 javaについての質問です。 キーボードから文字を一字ずつ入力し配列に格納する。 その後配列に格納されていた文字によってそれぞれ順番に処理していくというプログラ

  • 受付中

    おみくじのプログラムについて

    while文を使って、好みのメッセージが出るまで(「吉」が出るまで)おみくじを引き続けるプログラムを作りたいのですが、何度も実行しても1回の実行につきメッセージは1つしか出てきませ

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

  • C#

    7066questions

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