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

Question

前提・実現したいこと

あいこじゃなくなるまでじゃんけんをさせたいです

該当のソースコード

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 == 2 & plyr2.r == 0) {
			System.out.println(plyr2.name + "さんの勝ち") ;
		}else if (plyr.r == 0 & plyr2.r == 2) {
			System.out.println(plyr.name + "さんの勝ち") ;	
		}else if(plyr.r < plyr2.r) {
			System.out.println(plyr2.name + "さんの勝ち") ;
		}else if (plyr.r > plyr2.r) {
			System.out.println(plyr.name + "さんの勝ち") ;
		}
		
	}

}

package じゃんけん;

import java.util.Random ;

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

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

試したこと

if(plyr.r == plyr2.r){
    System.out.println("あいこ！") ;
    while(plyr.r == plyr2.r){
         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 ) ;
    }

と入れてみたのですが↑の場所で重複変数plyrなどとエラーが出てしまいうまくいきませんでした。
ならそこを入れなければいいのかと思い、入れずにprintlnの文だけ入れると…
といった感じでうまくいきませんでした。

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

eclipseを使っているので文頭にエラーメッセージを出してくれます。
エラーとはそのことです。

Answer 1

折角makeHandStatus()があるんですから、それを呼んだらいいんじゃないですかね？（あとちょっと手直し）

import java.util.Random;

public class Janken {

    public static void main(String[] args) {
        Player p1 = new Player(args[0]);
        Player p2 = new Player(args[1]);

        System.out.println("じゃんけん・・・ぽん！！");
        System.out.println(p1.name + "さんの手 :" + p1.handstatus);
        System.out.println(p2.name + "さんの手 :" + p2.handstatus);

        while (p1.r == p2.r) {
            System.out.println("あいこ！");

            p1.makeHandStatus();
            p2.makeHandStatus();
//            p1 = new Player(args[0]); // まあこうでもいいでしょうけど
//            p2 = new Player(args[1]);

            System.out.println("じゃんけん・・・ぽん！！");
            System.out.println(p1.name + "さんの手 :" + p1.handstatus);
            System.out.println(p2.name + "さんの手 :" + p2.handstatus);
        }

        if (p1.r == 2 & p2.r == 0) {
            System.out.println(p2.name + "さんの勝ち");
        } else if (p1.r == 0 & p2.r == 2) {
            System.out.println(p1.name + "さんの勝ち");
        } else if (p1.r < p2.r) {
            System.out.println(p2.name + "さんの勝ち");
        } else if (p1.r > p2.r) {
            System.out.println(p1.name + "さんの勝ち");
        }
    }
}

class Player {
    private static final Random ran = new Random();
    private static final String[] hands = { "グー", "パー", "チョキ" };

    final String name;
    String handstatus;
    int r;

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

    void makeHandStatus() {
        r = ran.nextInt(3);
        this.handstatus = hands[r];
    }
}

---

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

どんどん増えていくんでしょうか？
エラー内容等、具体的なタイトルにしましょう。

エラーが出てしまいうまくいきませんでした。

エラーが出た場合↓のように、質問にそのままコピペで書きましょう。

エラー:(18, 20) java: 変数 plyrはすでにメソッド main(java.lang.String[])で定義されています

Comments

[NEKOTABETAI]

回答ありがとうございます。ご指摘次から気を付けます。
なぜPlayerの方にfinalをつけないといけないのですか？

[TN8001]

final String nameこれですかね？
変わる可能性があるならつけれませんが、名前は変わらないと思うので付けました。
変わらないとわかっている場合、つける癖をつけておくと間違って変更してしまうミスを防げます^^

[NEKOTABETAI]

なるほど。参考になります。

Answer 2

プレイヤーが新しくなるわけではないので、plyrとplyr2はそのままで、あいこのときはmakeHandStatusを実行して手を新しくしましょう。そうなると、ranはフィールドにして、Randomをコンストラクターで1度だけ実行するように変更したほうがいいでしょう。

そして、あいこのあとであいこになり……という展開もあるので、結局のところ、勝ち負けが決まるまで繰り返すようにすることになるでしょう。

    public static void main(String[] args) {

        Player plyr = new Player(args[0]);
        Player plyr2 = new Player(args[1]);

        while (true) {
            System.out.println("じゃんけん・・・ぽん！！");

            System.out.println(plyr.name + "さんの手 :" + plyr.handstatus);
            System.out.println(plyr2.name + "さんの手 :" + plyr2.handstatus);

            int katimake = (plyr.r - plyr2.r + 3) % 3;

            if (katimake == 1) {
                System.out.println(plyr.name + "さんの勝ち");
                break;
            } else if (katimake == 2) {
                System.out.println(plyr2.name + "さんの勝ち");
                break;
            }

            System.out.println("あいこ！");
            plyr.makeHandStatus();
            plyr2.makeHandStatus();
        }

    }

Comments

[NEKOTABETAI]

回答ありがとうございます。判定システムが短くなってる！…すげぇ
while文の条件にtrueを使うと無限ループになるのを初めて知りました。
とてもスマートで分かりやすかったです。

[Daregada]

Javaはマイナスの値に対する剰余がマイナスになるので +3 しましたが、言語によっては、「(plyr.r - plyr2.r) % 3」だけで行けます(Pythonなど)。

[NEKOTABETAI]

Pythonもそのうち手をつけてみたいです。
いつになるやら…w

Answer 3

同じ変数名を複数回定義することはできません。
名前を変えましょう