星座の判別をIF文を大量に使わずに判別する方法

Question

前提・実現したいこと

現在のソースだと、IF文が大量にあるので、見栄えが悪く、ソースの修正があった場合、修正行が多くなることがあると思うので、星座判別の規則性を使ってソースを改変したいのですが、規則がわからず苦戦中
星座の判別について、javaのコマンドライン引数を使って星座を判別します。
コマンドライン引数は、
例）1201
1003

Case文も使用せずにソースの改変を行いたい

発生している問題・エラーメッセージ

エラーは現在ありません。

該当のソースコード

public class Constellation {
    public static void main(String[] args) {
        // 入力値MMDD(想定)
        String myDay = args[0];
        try {
            int month = Integer.parseInt(myDay.substring(0, 2));
            int day = Integer.parseInt(myDay.substring(2, 4));
            // 星座名取得.
            String seizaName = getSeiza(month, day);
            if (seizaName == null) {
                System.out.println("日付が正しくない");
            }
            System.out.println(seizaName);

        } catch (Exception e) {
            System.out.println("日付が正しくない");
        }
    }

    /**
     * 星座名取得.
     *
     * @param month 月
     * @param day   日
     * @return seizaName 星座名
     */
    public static String getSeiza(int month, int day) {
        String seizaName = null;
        if (month < 13) {
            if (month == 3 && day >= 21 || month == 4 && day <= 19) {
                return seizaName = "牡羊座";
            } else if (month == 4 && day >= 20 || month == 5 && day <= 20) {
                return seizaName = "牡牛座";
            } else if (month == 5 && day >= 21 || month == 6 && day <= 21) {
                return seizaName = "双子座";
            } else if (month == 6 && day >= 22 || month == 7 && day <= 22) {
                return seizaName = "蟹座";
            } else if (month == 7 && day >= 23 || month == 8 && day <= 22) {
                return seizaName = "しし座";
            } else if (month == 8 && day >= 23 || month == 9 && day <= 22) {
                return seizaName = "乙女座";
            } else if (month == 9 && day >= 23 || month == 10 && day <= 23) {
                return seizaName = "てんびん座";
            } else if (month == 10 && day >= 24 || month == 11 && day <= 22) {
                return seizaName = "蠍座";
            } else if (month == 11 && day >= 23 || month == 12 && day <= 21) {
                return seizaName = "射手座";
            } else if (month == 12 && day >= 22 || month == 1 && day <= 19) {
                return seizaName = "山羊座";
            } else if (month == 1 && day >= 20 || month == 2 && day <= 18) {
                return seizaName = "水瓶座";
            } else {
                return seizaName = "魚座";
            }
        } else {
            return seizaName;
        }
    }
}

補足情報（FW/ツールのバージョンなど）

Comments

[swordone]

とりあえず、getSeizaで使ってるローカル変数seizaNameは不要。
あと、日付の正当性はチェックしなくてよいのでしょうか(1032の様な入力)？

Answer 1

基本的に配列を使って｢月｣で割り振る。例えば1月は基本的に山羊座と考える。
月ごとに星座が切り替わる日付も配列で管理し、ある月でその日以降の日であれば月を1増やす。1月は20日以降が水瓶座なので、それを｢2月｣として扱う。
12月の次の月になった場合は1月に戻す。

    public static String getSeiza(int month, int day) {
        String[] seizaName = {"山羊座", "水瓶座", "魚座", "牡羊座", "牡牛座", "双子座",
                              "蟹座", "しし座", "乙女座", "てんびん座", "蠍座", "射手座"};
        int[] border = {20, 19, 21, 20, 21, 22, 23, 23, 23, 24, 23, 22};

        int index = month - 1;
        if(day >= border[index]){
            index++;
        }
        if(index == 12){
            index = 0;
        }
        return seizaName[index];
}

また、Java8以降ならMonthDayとDateTimeFormatterを使って、入力された日付の妥当性チェックもまとめて行うこともできます。

public class Constellation {
    public static void main(String[] args) {
        try {
            // 日付変換 不適当な入力であればDateTimeParseException発生
            MonthDay date = MonthDay.parse(args[0], DateTimeFormatter.ofPattern("MMdd"));
            // 星座名取得.
            String seizaName = getSeiza(date);
            System.out.println(seizaName);

        } catch (DateTimeParseException e) {
            System.out.println("日付が正しくない");
        }
    }

    // 星座名取得.
    public static String getSeiza(MonthDay date) {
        String[] seizaName = {"山羊座", "水瓶座", "魚座", "牡羊座", "牡牛座", "双子座",
                              "蟹座", "しし座", "乙女座", "てんびん座", "蠍座", "射手座"};
        // 各星座の年内の最後の月日
        MonthDay[] lastDays = {MonthDay.of(1, 19), MonthDay.of(2, 18), MonthDay.of(3, 20),
                               MonthDay.of(4, 19), MonthDay.of(5, 20), MonthDay.of(6, 21),
                               MonthDay.of(7, 22), MonthDay.of(8, 22), MonthDay.of(9, 22),
                               MonthDay.of(10, 23), MonthDay.of(11, 22), MonthDay.of(12, 21)};
        for (int i = 0; i < lastDays.length; i++) {
            if (date.compareTo(lastDays[i]) <= 0) {
                return seizaName[i];
            }
        }
        // 12月22日以降だった場合
        return seizaName[0];
    }
}

また、TreeMapを使えばこんな書き方もできます(Map生成の書き方はJava9以降)。
ただここまで長くなると、static importを使わずに書くのはつらいし見づらい…

    // 星座名取得.
    public static String getSeiza(MonthDay date) {
        TreeMap<MonthDay, String> lastDayOfSeiza = new TreeMap<>(
            Map.ofEntries(Map.entry(MonthDay.of(1, 19), "山羊座"), Map.entry(MonthDay.of(2, 18), "水瓶座"),
                          Map.entry(MonthDay.of(3, 20), "魚座"), Map.entry(MonthDay.of(4, 19), "牡羊座"),
                          Map.entry(MonthDay.of(5, 20), "牡牛座"), Map.entry(MonthDay.of(6, 21), "双子座"),
                          Map.entry(MonthDay.of(7, 22), "蟹座"), Map.entry(MonthDay.of(8, 22), "しし座"),
                          Map.entry(MonthDay.of(9, 22), "乙女座"), Map.entry(MonthDay.of(10, 23), "てんびん座"),
                          Map.entry(MonthDay.of(11, 22), "蠍座"), Map.entry(MonthDay.of(12, 21), "射手座"),
                          Map.entry(MonthDay.of(12, 31), "山羊座")
                          ));

        return lastDayOfSeiza.ceilingEntry(date).getValue();
    }

Comments

[jimbe]

> Java8以降ならMonthDayと...
そんなクラスがあったんですね, 知りませんでしたありがとうございます.

[swordone]

あったか自信がなかったので、なかったら適当に2020年などのうるう年指定して…とか考えてましたw

[ryuxx]

>Java8以降ならMonthDayと
そんなクラスがあったの初めて知りました。
よりシンプルにかけるみたいで勉強になりました。
ベストアンサーにさせていただきます

Answer 2

0119 山羊座
0218 水瓶座
0320 魚座
0419 牡羊座
0520 牡牛座
0621 双子座
0722 蟹座
0822 獅子座
0922 乙女座
1023 天秤座
1122 蠍座
1221 射手座
1300 山羊座

このデータから配列を二つ作ります。
一つは Integer 型で、119, 218, 320, ... が入ります。
もう一つは String 型で、山羊座, 水瓶座, 魚座, ... が入ります。
文字列をコードで分割しなくても、最初から配列としてコーディングしてもいいです。

次に調べたい日付の月を 100 倍して日と足します。
たとえば 10 月 14 日であれば 1014 となります。

数値の配列の頭から調べて行き、1014 が初めてその数値以下になった時点で対応する文字列を返します。
1014 の場合、1023 以下になるので、天秤座を返すことになります。

Answer 3

enum にします.

public class Main {
  public static void main(String args[]) {
    String myDay = args[0];
    try {
        int month = Integer.parseInt(myDay.substring(0, 2));
        int day = Integer.parseInt(myDay.substring(2, 4));
        MonthDay monthDay = MonthDay.of(month, day);
        // 星座取得.
        Zodiac constellation = Zodiac.get(monthDay);
        System.out.println(constellation.getJName());
    } catch (DateTimeException e) {
        System.out.println("日付が正しくない");
    }
  }

  enum Zodiac {
    Aries("牡羊座",  3,21,  4,19),
    Taurus("牡牛座",  4,20,  5,20),
    Gemini("双子座",  5,21,  6,21),
    Cancer("蟹座",  6,22,  7,22),
    Leo("獅子座",  7,23,  8,22),
    Virgo("乙女座",  8,23,  9,22),
    Libra("天秤座",  9,23, 10,23),
    Scorpius("蠍座", 10,24, 11,22),
    Sagittarius("射手座", 11,23, 12,21),
    Capricornus("山羊座", 12,22,  1,19) {
      boolean contains(MonthDay target) { //年をまたがるのでこれだけ式が違う
        return start.compareTo(target) <= 0 || target.compareTo(end) <= 0;
      }
    },
    Aquarius("水瓶座",  1,20,  2,18),
    Pisces("魚座",  2,19,  3,20);

    protected String jname;
    protected MonthDay start, end;
    Zodiac(String jname, int startMonth, int startDay, int endMonth, int endDay) {
      this.jname = jname;
      this.start = MonthDay.of(startMonth, startDay);
      this.end = MonthDay.of(endMonth, endDay);
    }
    String getJName() { return jname; }
    boolean contains(MonthDay target) {
      return start.compareTo(target) <= 0 && target.compareTo(end) <= 0;
    }

    static Zodiac get(MonthDay target) {
      for(Zodiac z : values()) if(z.contains(target)) return z;
      throw new DateTimeException("定義に洩れがあるようです. target="+target);
    }
  }
}

Answer 4

入力の形式がどうで、最終的にどういう結果を得たいのか、イマイチ分かりませんが…

配列を3種類用意します。
1つめ：要素が12個、月から日にちの配列を選ぶためのインデックスを得る。
2つ目：12個の配列、それぞれの月ごとに日数分の要素を持つ。
それぞれの日にちから星座のインデックスを取り出します。
3つ目：星座のインデックスから星座名の文字列を取り出す。

月、日の範囲をチェックしない、うるう年を判定しないなら、 if 文を書かなくてもできると思います。
星座名の文字列が不要なら、3つ目は要らない。

Comments

[ryuxx]

入力形式は:1201
みたいな感じでargs[0]に入る

出力結果として
入力された月日を算出して、正座名が出力される

Answer 5

星座を配列に入れて、日付からそのインデックスを算出すればいけますね

Comments

[ryuxx]

日付からインデックスを算出するということは、月の部分は一切判別条件に入れないということでしょうか？

[y_waiwai]

もちろん月も入りますよ

[ryuxx]

ピンと来ないので、もう少しヒントをご教示お願いします。。

Answer 6

こういった問題は配列だと思います。
まずはデータ構造を規定します。
javaに明るくないので、
データ構造をjsonで表します。

[{
  "seiza": "山羊座",
  "startMonth": 12,
  "startDay": 22,
  "endMonth": 1,
  "endDay": 21,
},{
  "seiza": "水瓶座",
  "startMonth": 1,
  "startDay": 22,
  "endMonth": 2,
  "endDay": 21,
},...全ての星座データ]

その後、この配列の中から条件に合致するデータを探す関数を定義します。
例えばjavascriptであれば配列にfind関数があるので、
comparatorを定義するだけです。
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/find

この方法の良いところは応用がきくところと宣言的であることだと思います。
まず、星座以外の場合はこのデータが増える場合がありますが、
ロジックは変更せずに配列のデータを追加するのみになります。

その他判断ロジックが増える場合等は配列に別のデータを追加し、
findに渡す関数のロジックを少し変更するのみです。

Answer 7

swordoneさんの改良してみました。if文なしでやってみました
C#でテストしてないんで正しいかどうか分からないですけど…。

        public static string getSeiza(int month, int day)
        {
            string[,] seizaName =
            {
                { "山羊座", "水瓶座", },
                { "水瓶座", "魚座", },
                { "魚座", "牡羊座", },
                { "牡羊座", "牡牛座", },
                { "牡牛座", "双子座", },
                { "双子座", "蟹座", },
                { "蟹座", "しし座", },
                { "しし座", "乙女座", },
                { "乙女座", "てんびん座", },
                { "てんびん座", "蠍座", },
                { "蠍座", "射手座", },
                { "射手座", "山羊座", },
            };
            int[] border = { 20, 19, 21, 20, 21, 22, 23, 23, 23, 24, 23, 22 };

            int index = month - 1;
            return seizaName[index, (int)(day / border[index])];
        }

Comments

[Zuishin]

申し訳ないけど改悪にしか見えません。
それとここで言うのもなんですが、過去の質問が解決したなら解決済みにしてください。してないのなら質問を編集して既存の回答でなぜ解決しないのかを書いてください。