enumを利用して、入力した値ごとに処理を分けたい

Question

java初心者です。
enumを使って星座判定をさせるプログラムを作ってみているのですが、
enumの使い方が上手く応用できません。
言語の本を買ったりネットで調べてみているのですが、
どうも詰まってしまったのでアドバイスをお願いしたいです。

前提・実現したいこと

メインで入力した年、月、日によって
星座を出力させたい

該当のソースコード

public class Main {

	public static void main(String[] args) {
		Constellation constellation = Constellation.Aquarius;

		System.out.println(constellation.toString());

		System.out.println("**********星座判定**********");
		System.out.println("誕生年を入力してください　(例：1998-＞1998)");
		int year = new java.util.Scanner(System.in).nextInt();
		System.out.println("誕生月を入力してください　(例：11月-＞11)");
		int month = new java.util.Scanner(System.in).nextInt();
		System.out.println("誕生日を入力してください　(例：22日-＞22)");
		int day = new java.util.Scanner(System.in).nextInt();

		System.out.println(year+"年"+month+"月"+day+"日生まれのあなたの星座は"+constellation+"です");

		switch (constellation) {
		case Aries:
			System.out.println("牡羊座");
			break;

		case Taurus:
			System.out.println("牡牛座");
			break;

		case Gemini:
			System.out.println("双子座");
			break;

		case Cancer:
			System.out.println("牡牛座");
			break;

		case Leo:
			System.out.println("獅子座");
			break;

		case Virgo:
			System.out.println("乙女座");
			break;

		case Libra:
			System.out.println("天秤座");
			break;

		case Scorpio:
			System.out.println("蠍座");
			break;

		case Sagittarius:
			System.out.println("射手座");
			break;

		case Capricorn:
			System.out.println("山羊座");
			break;

		case Aquarius:
			System.out.println("水瓶座");
			break;

		case Pisces:
			System.out.println("魚座");
			break;

		}

	}
}

該当のソースコード

public enum Constellation {


	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),
	Scorpio("蠍座", 10/24, 11/22),
	Sagittarius("射手座", 11/23, 12/21),
	Capricorn("山羊座", 12/22, 1/19),
	Aquarius("水瓶座", 1/20, 2/18),
	Pisces("魚座", 2/19, 3/20);

	private String name;
	private int startDay;
	private int endDay;

	public String getName() {
		return name;
	}

	public void setName(String name) {
		this.name = name;
	}

	public int getStartDay() {
		return startDay;
	}

	public void setStartDay(int startDay) {
		this.startDay = startDay;
	}

	public int getEndDay() {
		return endDay;
	}

	public void setEndDay(int endDay) {
		this.endDay = endDay;
	}

	Constellation(String string, int i, int j) {
		
	}

	public void getType(int year,int month,int day) {

	}
}

試したこと

教本を買って読んだり、ネットでの検索を行いました。

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

ここにより詳細な情報を記載してください。

Comments

[m.ts10806]

>教本を買って読んだり、ネットでの検索を行いました。
もっと詳しく記載してください。
間違ってる情報を元にしたり、正しく理解できていなければやりたいことはできません。あと現状何が起きてるんでしょうか。

[dodox86]

> enumの使い方が上手く応用できません。

具体的にどうしたいのかを示しましょう。enumを使い、どういう操作をしてどういう結果を得たいのか質問文とコードからは分かりづらいように思います。

[dodox86]

入力された月日で星座を判定し、該当の星座の名前を出力することが目的ならば、enumは無理に使わず、単に文字列(String)で出力してしまっても良いはずです。それを押してご自身の課題としてenumを使いたいのであれば、どういう操作をしたいのか詳細を示しましょう。「エスパー回答」と称して回答者が勝手に想像、補間して回答することはできますが、それが質問者さんのご要望に沿うとは限りません。そしてそのような場合、大抵、質問回答が長引きます。

Answer 1

enumの使い方の応用に着目するのは、とても素晴らしいと思います。
特に何も考えずにifをダラダラと並べる人がとても多いからです。

今、提示されているコードも、このダラダラに該当していますので、
あまりenumを使っている意味がない感じになっています。switchを消し去りたいです。

星座は12種類固定であり、今後も増減する可能性は極めて低いため、
このようにダラダラと書いていてもさほど影響することはないですが、
将来的に増えたり減ったりする可能性があるものをifやswitchで条件分岐しちゃうのは、
大変イケてないコードですので、ダラダラ条件分岐をしたくなったら、一旦立ち止まって考えましょう。
以下を参考にしてみてください。

ただし、このままでは、日付として存在しない数字が入力された場合にエラーが起きますので、
その辺りのチェックを入れると、より実用性のあるコードになりますね。

import java.util.Scanner;

public class Main {
	public static void main(String[] args) throws Exception {
		try (Scanner scanner = new Scanner(System.in)) {
			System.out.println("**********星座判定**********");
			System.out.println("誕生年を入力してください　(例：1998-＞1998)");
			int year = scanner.nextInt();
			System.out.println("誕生月を入力してください　(例：11月-＞11)");
			int month = scanner.nextInt();
			System.out.println("誕生日を入力してください　(例：22日-＞22)");
			int day = scanner.nextInt();

			System.out.println(year + "年" + month + "月" + day + "日生まれのあなたの星座は" + Constellation.of(month, day).getName() + "です");
		}
	}
}

import java.time.MonthDay;
import java.util.Arrays;

public enum Constellation {

	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),
	Scorpio("蠍座", 10, 24, 11, 22),
	Sagittarius("射手座", 11, 23, 12, 21),
	Capricorn("山羊座", 12, 22, 1, 19),
	Aquarius("水瓶座", 1, 20, 2, 18),
	Pisces("魚座", 2, 19, 3, 20);

	private String name;
	private MonthDay startDay;
	private MonthDay endDay;

	Constellation(String name, int startMonth, int startDay, int endMonth, int endDay) {
		this.name = name;
		this.startDay = MonthDay.of(startMonth, startDay);
		this.endDay = MonthDay.of(endMonth, endDay);
	}

	/**
	 * 指定された月と日から、該当するConstellationを返します。
	 *
	 * @param month 月
	 * @param dayOfMonth 日
	 * @return Constellationのいずれか
	 */
	public static Constellation of(int month, int dayOfMonth) {
		MonthDay targetDay = MonthDay.of(month, dayOfMonth);
		return Arrays.stream(Constellation.values())
				.filter(constellation -> !(constellation.startDay.isAfter(targetDay) || constellation.endDay.isBefore(targetDay)))
				.findFirst()
				.orElseThrow(IllegalArgumentException::new);
	}

	public String getName() {
		return name;
	}
}

Comments

[koutarou55478]

なるほど・・・コードを書くのに精一杯で、
将来的なことはあまり考えておりませんでした。

有効的に使用できるよう頑張ります。
回答していただき、誠にありがとうございました。

Answer 2

列挙型Constellationのコンストラクタはこう定義されている。

    Constellation(String string, int i, int j) {

    }

コンストラクタでは何もしていないので、列挙型Constellationのインスタンス変数は、初期値のまま。それぞれ、null, 0, 0。

    private String name;
    private int startDay;
    private int endDay;

また、コンストラクタ呼び出しの第二第三引数にも問題があるのではないかと。

 Aries("牡羊座",3/21,4/19)

3/21や4/19は、日付、月日のつもりなのでしょうが、3/21は、3を21で割った結果なので、ゼロ。同様に4/19も、ゼロ。
コンストラクタに何かを書いたとしても、動きはしないでしょうね。