javaの配列型メソッドの引数(int型)をString型に変換し表示させたいのですがうまくいきません。

Question

前提・実現したいこと

profileメソッド(引数　int型)内で配列を3つ作成し
引数をString型に変換、それぞれ配列の0番目の値とString型に変換した引数を比較し
一致した場合に配列変数を戻り値として返す。

mainメソッド内にてprofileメソッドを呼び出し
returnされた配列の要素をfor文を使い0番目の要素以外を表示したい。

エラーメッセージが出てうまく表示ができない→引数がString型にうまく型変換できていないのか？

■■な機能を実装中に以下のエラーメッセージが発生しました。

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

エラーメッセージ

Exception in thread "main" java.lang.NullPointerException
	at practice.Practice.main(Practice.java:31)

該当のソースコード

java

public class Practice {
　　　　//メソッドprofile(引数int型)
	static String[] profile(int brother) {

		//静的配列を3つ作成
		String[] data1= {"1","長男",};
		String[] data2=  {"2","次男",};
		String[] data3=	 {"3","三男",};

		//引数int型をString型に変換
		String strnum=String.valueOf(brother);

		if(data1[0]==strnum) {
			return data1;
		}
		if(data2[0]==strnum) {
			return data2;
		}
		if(data3[0]==strnum) {
			return data3;
		}
//どの配列とも一致しない場合nullを戻り値に
		return null;
	}

	public static void main(String args[]) {

		String[] data=profile(1);

		for(int i=1;i<data.length;i++) {
			System.out.println(data[i]);
		}
	}
}

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

eclipse javaアプリケーションで起動

Answer 1

文字列を==で比較すると、等しくてもfalseとなることがありますので、（よほどのことがない限り）やってはいけません。data1[0].equals(strnum)のように比較してください。

Comments

[pirapira]

ありがとうございます。勉強になり、表示もうまくできました。

Answer 2

Javaで文字列を比較する際は、必ずObject#equalsを使って下さい。

- data1[0]==strnum
+ data1[0].equals(strnum)

---

なお、profileメソッドは連想配列を使った方が組みやすいです。

Comments

[maisumakun]

（今回は固定の配列なので大丈夫ですが）左辺にnullが来る可能性があるときは、Objects.equalsも便利ですね。

[LouiS0616]

そうですね。
参照型の変数の実体を比較する際は常にObjects.equalsを使った方が良いのかもしれません。

String.valueOf(int)はnullを返さないので、今回のケースではstrnum.equals(data1[n])でも充分安心できそうですが。

[pirapira]

詳しくありがとうございます。勉強になります。

[LouiS0616]

@pirapira さん
解決されたようで何よりです。

@低評価を押された方
低評価の理由が分かりません。
・ Stringを==で比較できるケースを無視しているから？
・ 『連想配列』というタームがJavaの仕様に存在しないから？

もちろん評価は自由にしていただいて構わないのですが、回答の質を向上させるためにも問題点のご指摘のほどよろしくお願いします。