java 変数が値が含まれていない

Question

java
```
コード
```equalsを簡単なゲームを作成しています。

ですが、if文での処理にて変数の値が受け取れません

変数 resultが初期化できていませんと表示されますが
初期化するのではなく、引継ぎをしたいのですが、初歩的な質問にはなりますがよろしくお願い致します。


public static void main(String[] args) {

Scanner sc = new Scanner(System.in);
System.out.println("******");
System.out.println("*High & Low*");
System.out.println("******");

System.out.println();

int leftCard = (int)(Math.random()*9)+1;        
System.out.println(" [問題表示]   ");
System.out.println("* *");
System.out.println("*   * * * *");
System.out.println("* " +leftCard + " * * * *");
System.out.println("*   * * * *");
System.out.println("* *");
System.out.println("High  or  Low ?(h/l) > ");        
String select = sc.nextLine();

if (select.equals("h")) {
System.out.println("→Highを選択しました。");

} else {
System.out.println("→Lowを選択しました。");
};

System.out.println("");

System.out.println("[結果表示]");
System.out.println("* *");
System.out.println("*   * *   *");
int rightCard = (int)(Math.random()*9)+1;        
System.out.println("* +leftCard+  * * +rightCard+ *");
System.out.println("*   * *   *");
System.out.println("* *");

String result ;
if (leftCard < rightCard) {
result =  "h" ;
} else if (leftCard >rightCard ) {
result = "l";
} else if ( leftCard == rightCard) {
result = select ; 
};

if (result.equals(select)) {
System.out.println("→You Win!");
} else if (result != select) {
System.out.println("→you Lose...");
};

System.out.println("");

System.out.println("--ゲーム終了--");

}
}

Comments

[m.ts10806]

コードはマークダウンのcode機能を利用してご提示ください。

[hiro.a]

失礼しました。修正します。

[m.ts10806]

なんでしょう。「引き継ぎをしたい」とは。
初期化されてないというエラーなら初期値を設定すればいいのでは？

[m.ts10806]

質問投稿前にプレビュー確認しましょう。

[m.ts10806]

質問未解決のまま次々と投稿するのはやめてください。
解決する気がないように感じます。
「受付中」のままの質問を放置しすぎです。

Answer 1

String result ;
if (leftCard < rightCard) {
result =  "h" ;
} else if (leftCard >rightCard ) {
result = "l";
} else if ( leftCard == rightCard) {
result = select ; 
};

残念ながら、Javaコンパイラは「ifの3ついずれかの条件に入る」ということを認識できません（仮にleftCardやrightCardがdoubleだった場合、NaNという特殊な値が入れば3つとも成立しないパターンは作成可能です）。

最後のelse ifをelseに変えて、必ず通るようにしておきましょう。

Answer 2

質問者のこだわりが全く分からないのですけど、

ここを

String result ;
if (leftCard < rightCard) {
   result =  "h" ;
} else if (leftCard >rightCard ) {
    result = "l";
} else if ( leftCard == rightCard) {
    result = select ; 
}

↓こう変えるだけで終わる話ではないかな

String result = select;
if (leftCard < rightCard) {
   result =  "h" ;
} else if (leftCard >rightCard ) {
    result = "l";
}

これで最後の同じかどうかって判定は、なくてもよくなると思います。

Comments

[hiro.a]

確かにそうですね！ありがとうございます。

[m.ts10806]

なぜこれで通るかご理解の上での解決済みなら良いのですけど。

Answer 3

まず、「変数が初期化されていない」というのは、宣言だけして1度も代入していない状態です。
初期化されていない変数を(代入以外で)使おうとすると、「変数~~~を初期化してください」と怒られます。

今回のコードでは、

String result ;
if (leftCard < rightCard) {
    result =  "h" ;
} else if (leftCard >rightCard ) {
    result = "l";
} else if ( leftCard == rightCard) {
    result = select ; 
}; 

の部分で、実際にはどんなケースでもresultには何かしらの値が代入されますが、コンパイラは頭が悪いので、
「どの条件も満たさなかった場合はresultが初期化されない」と判断してしまいます。
なので、コンパイラが見ても明らかに初期化されているように書く必要があります。

例えば宣言時に適当な値を入れて初期化するか:

String result == "";
if (leftCard < rightCard) {
    result =  "h" ;
} else if (leftCard >rightCard ) {
    result = "l";
} else if ( leftCard == rightCard) {
    result = select ; 
}; 

一番最後の条件をelseにします :

String result ;
if (leftCard < rightCard) {
    result =  "h" ;
} else if (leftCard >rightCard ) {
    result = "l";
} else {
    result = select ; 
}; 

Comments

[maisumakun]

leftCardやrightCardがintである以上、「どの条件も満たさなかった場合」は実際には発生しないかと思います。

[hiro.a]

初期化してしまうと今までの処理が引き継がれずになると言う事ですよね？

[ozwk]

@maisumakun
...と言われたらコンパイラはそんな賢くないです云々と返そうと思ってたんですけど
よく考えたら予め書いておいたほうがスムーズに事が運びましたね。

[ozwk]

「引き継がれずになる」ってなんですか? 何が聞きたいのかわかりません。
引き継がれずに(なにがどう)なるんですか?

[hiro.a]

わかりましたありがとうございます。

[jimbe]

> 初期化してしまうと今までの処理が引き継がれず
初期化しろと言われているのは, 変数であって, 処理ではありません.

[hiro.a]

leftCardとに対してrightCardの数字を比較するゲームになります。

if文を利用してresult・selectに値を代入してきた中でresultの値を初期化してしますと意味が無いのでは無いかと思っています。

「引き継がれずになる」ってなんですか? 何が聞きたいのかわかりません。
引き継がれずに(なにがどう)なるんですか?→私の誤字と説明不足です。

[maisumakun]

> resultの値を初期化してしますと意味が無いのでは無いかと思っています。

いえ、正反対です。初期化しないローカル変数は、コンパイルエラーになります。

[hiro.a]

初期化すると変数に代入されていた値を新たに代入すると言う認識ですが間違ってますか？

[maisumakun]

> 初期化すると変数に代入されていた値を新たに代入すると言う認識ですが間違ってますか？

はい、違います。この場面での「初期化」は、「変数の最初の値を決める」という意味です。