じゃんけんゲームについて

Question

前提・実現したいこと

じゃんけんゲームを作っているのですがうまくいきません

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

じゃんけん・・・ぽん！！
pochiさんの手 :null
taroさんの手 :null
あいこ！

といった具合にnullになってしまいます。

該当のソースコード

package じゃんけん;

public class Janken {

	public static void main(String[] args) {
		

		
		Player plyr = new Player(args[0]) ;
		Player plyr2 = new Player(args[1]) ;
		
		System.out.println("じゃんけん・・・ぽん！！") ;
		System.out.println(plyr.name + "さんの手 :" + plyr.handstatus) ;
		System.out.println(plyr2.name + "さんの手 :" + plyr2.handstatus ) ;
		
		if(plyr.r == plyr2.r) {
			System.out.println("あいこ！") ;
		}else if(plyr.r < plyr2.r) {
			System.out.println(plyr.name + "さんの勝ち") ;
		}else if (plyr.r > plyr2.r) {
			System.out.println(plyr2.name + "さんの勝ち") ;
		}

	}

}

package じゃんけん;

import java.util.Random ;

public class Player {
	
	String name ;
	String handstatus ;
	int r  ;
	
	Player(String hito){
		
		this.name = hito ;
		
	}

	void makeHandStatus() {
		
		String[] hands = {"グー" , "パー" , "チョキ"} ;
		
		Random r = new Random() ;
		
		String hand = hands[r.nextInt(3)] ;
		
		this.handstatus = hand ;
		
	}
}

試したこと

なぜnullになってしまうのか全く分かりません。ちゃんと代入してるのにどうしてでしょう？

Comments

[dodox86]

PlayerクラスのmakeHandStatus()メソッドは何の為にあるのでしょうか。それを考え直してみてください。

[NEKOTABETAI]

...申し訳ありません

Answer 1

なぜnullになってしまうのか全く分かりません。ちゃんと代入してるのにどうしてでしょう？

確かに、makeHandStatusメソッドで、疑似乱数の値を基にグー・チョキ・パーの手をthis.handstatusに代入していますね。

あなたがやっていないのは、makeHandStatusメソッドを呼び出す(実行する)ことです。
たとえば、コンストラクター内で呼び出せば、オブジェクト生成時にhandに値が設定されます。

Player(String hito){
    this.name = hito ;
    makeHandStatus();
}

追記:
そして、判定に使っているrフィールドの値も設定していませんよね。makeHandStatusの中で設定してください。

void makeHandStatus() {
    String[] hands = { "グー", "パー", "チョキ" };
    Random rnd = new Random();
    r = rnd.nextInt(3);
    String hand = hands[r];
    this.handstatus = hand;
}

Comments

[NEKOTABETAI]

おっしゃる通りでした。コンストラクター内で呼び出すやり方、参考にさせていただきます。

[NEKOTABETAI]

なるほど。いろいろと足りなかったのですね。
しょうもないミスですみませんでした。気を付けます。
回答ありがとうございました。