ジャンケンプログラム while利用の繰り返し処理（java）

Question

現在、javaの練習としてジャンケンのプログラムを作っています。

「あいこ」になった際、再度ジャンケンをする、ところが無限ループしてしまいます。
どなたかJavaに詳しい方がいらっしゃいましたらアドバイスいただけませんでしょうか。
よろしくどうぞお願いいたします。

▽条件
ジャンケンをする2人を生成する
その二人がジャンケンをする
勝った方の名前を出力する
あいこになった場合、二人が再度ジャンケンをする

import java.util.Random;
class jyanken{
    public static void main(String[] args){
        Person person1 =new Person("tanaka");
        Person person2 =new Person("suzuki");
        
        System.out.println("それではジャンケンをはじめます");       
        person1.printData();
        person2.printData();

        while (true) {


          if(person1.getStr()=="グー" && person2.getStr()=="グー"||person1.getStr()=="チョキ" && person2.getStr()=="チョキ"||person1.getStr()=="パー" && person2.getStr()=="パー"){
              System.out.println("あいこ");
              person1.printData();
              person2.printData();

          }else if(person1.str=="グー"&&person2.str=="チョキ"||person1.str=="チョキ"&&person2.str=="パー"||person1.str=="パー" && person2.str=="グー") {
              System.out.println(person1.name+"の勝ちです");
              break;
          }else{
              System.out.println(person2.name+"の勝ちです");    
              break;
          }
          
        }


    }  
}

class Person{
    public String name;
    String[] strs = {"グー","チョキ","パー"};
    Random r = new Random();
    String str = strs[r.nextInt(3)]; 
 
    Person(String name){
        this.name = name;
    }

    public String getStr(){
        return str;
    }


    public void printData(){
        System.out.println(this.name);
        Random r = new Random();
        String str = strs[r.nextInt(3)]; 
        System.out.println(this.str);

        }
    }

▽エラーメッセージ
エラーメッセージは出ないが、「あいこ」になった場合、ループしてしまう

▽利用環境
windows10
Visual Studio code
コマンドプロント

Comments

[swordone]

質問編集画面にて、コード部分を反転選択し、<code>というボタンをクリックし、「言語をここに入力」を「java」に書き換えて投稿してください。

[swordone]

「自動ループする」というのは、無限ループするということですか？

[yunaga]

ご指摘ありがとうございます。課題を修正いたしました！「自動ループ」も「無限ループ」へ変更いたしました。

Answer 1

Personクラスに問題があります。
じゃんけんの動作をprintData()メソッドで行っているので、printData()メソッドの中身を考えます。
0以上3未満の乱数を発生させて、それを配列のインデックスとして、対応する手を文字列として出力する。
という動きですが、

    public void printData(){
        System.out.println(this.name);
        Random r = new Random();
        String str = strs[r.nextInt(3)]; 
        System.out.println(this.str);

    }

これだと、じゃんけんの結果の手はローカル変数のstrに入って、メンバ変数のthis.strには入っていません。this.strを初期化の際に

class Person{
    public String name;
    String[] strs = {"グー","チョキ","パー"};
    Random r = new Random();
    String str = strs[r.nextInt(3)]; 
 :
 :

と初期化しているために、何か手が入っている状態のため、問題に気が付きにくい状態になっています。
ですので直すとしたら、下記のようになります。

class Person{
    public String name;
    String[] strs = {"グー","チョキ","パー"};
    String str = ""; 

    Person(String name){
        this.name = name;
    }

    public String getStr(){
        return this.str;
    }

    public void printData(){
        System.out.println(this.name);
        Random r = new Random();
        this.str = this.strs[r.nextInt(3)]; 
        System.out.println(this.str);

    }
}

メンバ変数のRandom rは使っていないので削除しました。

あと、メンバ変数を記述する際にthisをつけたりつけなかったりする書き方をされていますが、初学者だとのことですので、明確に区別できるようにメンバ変数を使っているところは全てthisをつけておきました。

もっと細かいことを言うと、呼び出し側でgetStr()を使ったり、直接メンバ変数のstrを使ったりしていますが、本来でしたら、メンバ変数はprivate宣言しておいて、getStr()を使うように統一したほうが良いです(同様にnameもgetName()を作ってこちらを使用する)。

Comments

[yunaga]

ご丁寧に指示をくださいましてありがとうございます！！勉強になることが本当に多いです！