変数が初期化されない可能性がある

Question

import java.io.*;
import java.lang.*;

public class Q2 {
 public static void main(String[] args) {
 
  String str1,str2;
  
  try {
  BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
  System.out.println("1つ目の文字列を入力してください");
  str1 = br.readLine();
  System.out.println("2つ目の文字列を入力してください");
  str2 = br.readLine();
  } catch (IOException ioe) {
  System.out.println("標準入力において例外が発生しました。");
  } catch (NumberFormatException ioe) {
  System.out.println("標準入力において例外が発生しました。");
  }
  if (str1.equals(str2)) {
  System.out.println("Same!");
  } else {
  System.out.println("Different!");
  }
 }
}

以上のコードをコンパイルすると、「変数str1/str2は初期化されていない可能性があります」とのエラーメッセージが出てしまいます。どこが原因なのでしょうか？ご教授願います。

Answer 1

tryの中で例外が発生すると残りの処理は飛ばしてcatchに行きます。

str1、str2は
str1 = br.readLine();
str2 = br.readLine();
で初めて値が入りますが(初期化されますが)、
たとえばtryの次の行で例外が発生すると値が入らないままcatchに飛んで残りの処理に進みます。

つまりtryの前で初期化(初期値を代入)しておきましょうということです。

Answer 2

str1 = br.readLine();やstr2 = br.readLine();を実行するよりも前に例外が発生すると
catchに処理がうつり、その後のif (str1.equals(str2))で未初期化のstr1/str2を使用してしまいます。

Answer 3

str1,str2に最初に値が入る可能性があるのが、tryブロックの中です。tryの中で例外が発生した場合、str1,str2に何も代入されない状態でifに到達することになります。
宣言時点で何かしら代入しておくべきでしょう。ただしstr1にnullを代入するとヌルポで落ちます。例えばstr1に空文字""、str2にnullを代入する、など。

Comments

[PARISSIENNE]

回答いただきありがとうございます。変な質問になってしまっていると思いますが、変数の宣言のすぐあとで初期化を行った場合、tryブロックの中で同じ変数にreadLine()メソッドで代入を行うにはどうしたらよいのでしょうか。

[swordone]

finalでなければ何度でも代入できるのでそのままでも大丈夫です。

[PARISSIENNE]

先ほどのソースコードに
String str1,str2;
str1 = firststring;
str2 = secondstring;
以上のように初期化を行ったのですが、今度は「シンボルが見つかりません」と表示されてしまいました。この方法はどこが誤りですか？

[swordone]

エラーを読んでください。firststringとかsecondstringとかは一体何ですか？

[PARISSIENNE]

変数でない値を代入しようとしていたのでエラーが出てしまっていたのですね。無事解決しました。
丁寧に説明していただきありがとうございます。