乱数を用いてstringで変数を定義

Question

前提・実現したいこと

javaで簡単な占いゲームを作ってみようとしましたが、エラーが出てしまいます…
乱数を発生させた後にswitch文で場合分けをして変数colorをそれぞれ別のに定義しようとしたのですが、すでに定義されている、というエラーメッセージが出てきてしまいます。何が間違っているのか教えてもらいたいです。

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

public class part03

{
	public static void main(String[]args){
	
		System.out.println("今からあなたを占いましょう");
		System.out.println("氏名を漢字で入力してください。");
		String name = new java.util.Scanner(System.in).nextLine();
		System.out.print(name + "さんですね。あなたの運勢は…");
		int r = new java.util.Random().nextInt(4);
		int p = new java.util.Random().nextInt(4);
		r++;
		p++;
		switch (r){
		case 1:
			System.out.print("大吉！");
			break;
		case 2:System.out.print("中吉！");
			break;
		case 3:System.out.print("凶！");
			break;
		case 4:System.out.print("大凶！");
			break;
		}
		
		switch(p){
		case 1:
			String color = "青";
			break;
		case 2:
			String color = "赤";
			break;
		case 3:
			String color = "黄色";
			break;
		case 4:
			String color = "緑";
			break;
		}
		System.out.print("ラッキーカラーは"+ color +"です！");
		
	}
}

part03.java:30: エラー: 変数 colorはすでにメソッド main(String[])で定義されています
                        String color = "赤";
                               ^
part03.java:33: エラー: 変数 colorはすでにメソッド main(String[])で定義されています
                        String color = "黄色";
                               ^
part03.java:36: エラー: 変数 colorはすでにメソッド main(String[])で定義されています
                        String color = "緑";
                               ^
part03.java:39: エラー: シンボルを見つけられません
                System.out.print("ラッキーカラーは"+ color +"です！");
                                             ^
  シンボル:   変数 color
  場所: クラス part03
エラー4個

該当のソースコード

switch(p){
		case 1:
			String color = "青";
			break;
		case 2:
			String color = "赤";
			break;
		case 3:
			String color = "黄色";
			break;
		case 4:
			String color = "緑";
			break;
		}
		System.out.print("ラッキーカラーは"+ color +"です！");

Answer 1

switch の各 case 内は変数のスコープとしては分かれておりません.

switch の {} の中で 1 つのスコープとなっていますので, 複数の case 内で同じ変数の定義が記述されれば "すでに定義されている" となります.

同じ原因で, case 内で定義された変数は switch の {} の外では効きませんので, print の個所で "シンボルを見つけられません" となります.

#追加
どちらかの入門書のサンプルのようにも見えますが, このような場合は配列を用いられると良いかと思います.

import java.util.*;
public class part03 {
    private static final String[] FORTUNE = {"大吉！", "中吉！", "凶！", "大凶！"};
    private static final String[] COLOR = {"青", "赤", "黄色", "緑"};
    public static void main(String[]args){
        System.out.println("今からあなたを占いましょう");
        System.out.println("氏名を漢字で入力してください。");
        String name = "";
        try(Scanner scanner = new Scanner(System.in);) {
            name = scanner.nextLine();
        }
        System.out.println(name + "さんですね。");
        Random random = new Random();
        int r = random.nextInt(FORTUNE.length); //0-3
        System.out.println("あなたの運勢は…"+FORTUNE[r]);
        int p = random.nextInt(COLOR.length); //0-3
        System.out.println("ラッキーカラーは"+COLOR[p]+"です！");
    }
}

Answer 2

Switch...case内で複数のString color が定義されています。

なので「重複しているのが原因」です。

        String color;
        
        switch(p){
        case 1:
            color = "青";
            break;
        case 2:
            color = "赤";
            break;
        case 3:
            color = "黄色";
            break;
        case 4:
            color = "緑";
            break;
        default: // これがないといけない…( 初期化していないし )
            color = "";
            break;
        }

というふうに、変数宣言をswitchの外にくくりだせばいいのです。

Comments

[momon-ga]

String color; だと、colorが初期化されない可能性があるので、コンパイルエラーになりませんか？
明示的に初期化するか、defaultが必要かと。

[BeatStar]

あ、確かにそうですね!
修正しておきます。

[mookeni]

できました！回答ありがとうございました。

Answer 3

質問への回答ではありませんが、nextInt(4) が返す値は 0～3 です。

また、
Random r = new Random();
r.nextInt(4);
r.nextInt(4);
のように一つの乱数発生器 r を用意して、そこから順番に値を取り出すようにしましょう。
そうではなく、毎回 new Random() を実行して値を取り出すと、正常な乱数とはならないでしょう。

Comments

[swordone]

だからその後にr++とかしてますね。

[kazuma-s]

すみません。見落としていました。