初期化されていないエラー

Question

前提

暗算の問題をランダムに出題するプログラムを作っています。

エラー

ローカル変数kotaeが初期化されていない可能性があります。

というエラーが出ます。
同じブロック内（public static void main(String[] args) {}）にあると思うのですが。なぜこのようなエラーが出るのでしょうか。また、解決方法を教えていただきたいです。
（可能でしたらこの箇所以外でもなにかありましたらお願いします。）

該当のソースコード

import java.util.Random;
import java.util.Scanner;

public class Hello{

	public static boolean retry() {
		try(Scanner stdIn = new Scanner(System.in)){
		int retry;
		do {
			System.out.println("もう一度？Yes:1/No:0");
			retry = stdIn.nextInt();
		}while(retry != 1 && retry != 0);
		return retry == 1;
	}
	}

	public static void main(String[] args) {
		try(Scanner stdIn = new Scanner(System.in)){
			Random rand = new Random();

			System.out.println("暗算テスト");

			int kotae;
		do {
			int x = rand.nextInt(100) + 1;
			int y = rand.nextInt(100) + 1;
			int z = rand.nextInt(100) + 1;
			int pattern = rand.nextInt(4);


			switch(pattern) {
			case 0 : kotae = x + y + z; break;
			case 1 : kotae = x + y - z; break;
			case 2 : kotae = x - y + z; break;
			case 3 : kotae = x - y - z; break;
			}

			while(true) {
				System.out.print(x + ((pattern < 2) ? "+" : "-")
					+ y + ((pattern % 2 == 0) ? "+" : "-") + z + "=");
				int answer = stdIn.nextInt();
				if(answer == kotae) break; //ここにエラーが出ます。
				System.out.println("違いますよ");
			}

			}while (retry());

		}
		}

	}

Answer 1

            switch(pattern) {
            case 0 : kotae = x + y + z; break;
            case 1 : kotae = x + y - z; break;
            case 2 : kotae = x - y + z; break;
            case 3 : kotae = x - y - z; break;
            }

さて、ここでpatternが４以上の場合はどうなるかを考えてください。
まさにここを見て、初期化されない可能性がある、と判断されて先のワーニングが出ています

Comments

[Tazusa]

defaultをすっかり忘れていました。ありがとうございます。

[y_waiwai]

defaultでいいんですが、このswitchの前で何らかの初期値を与えておく、のほうが私の好みですねー

[Tazusa]

みなさんおっしゃっている通り初期値なんですね。ありがとうございます。

Answer 2

こんにちは,
エラーが出る原因は
1.変数kotaeを初期化してない
2.switch文のところでcaseが0～3以外だったらkotaeは値を持っていない。（計算されません。）
3.その状態で最後if文で評価しようとするからエラーが出る
と考えられます。

解決策は
1.まずkotaeを初期化する (とりあえず int kotae = 0; としておきましょう!!)
2.通常　switch文はdefaultを付けるのが規則です。

以上を試してみてください。

Comments

[Tazusa]

整理していただきありがとうございます。もっと勉強します。

Answer 3

int kotae;
↓
int kotae = 0;

でやってみて下さい

※補足
”ローカル変数　初期化　Java”　でググると解説サイトが複数ヒットします。

Comments

[Tazusa]

ありがとうございました。検索してみます。

Answer 4

patternのスイッチだけをみると0123以外が入ってる可能性があるので初期化されない可能性があります。

コンパイラは戻り型しかみないのです。

Comments

[Tazusa]

ありがとうございます。