同じ処理を延々と繰り返すバグを修正したい

Question

前提・実現したいこと

ジャンケンを使ったHP制のバトルゲームを作っています。
下記のコードを利用すると
「主人公の名前」と「敵の名前」を入力させる処理を延々と繰り返してしまいます。

修正したいのですが、何故そのように繰り返してしまうか原因がわかりません。
イクリプス上で開発していますが、特にエラー表示などは無い状態です。
アドバイスをいただければ幸いです。

該当のソースコード

import java.util.Scanner;

public class Main {

	public static Scanner scanner = null;
	public static final int STONE = 0;//グー
	public static final int SCISSOES = 1;//チョキ
	public static final int PAPER = 2;//パー

    	public static void main(String[] args) {

      Human human =new Human("あなた");
	  Cpu cpu =new Cpu("敵");

      while (0 < human.humanHp && 0 < cpu.cpuHp) {
          human.getHand();

          if (human.hand == 9) {
              System.out.println("コマンド９検出：強制終了しました");
              return; // 勝ち負け判定に入らないように抜ける
          } else if (human.hand >= 3 ) {
        	  System.out.println("数値は0・１・２から選んで入力して下さい");
          }

          cpu.getHand();

          if (human.hand == cpu.com) {
              System.out.println("引き分け");
          }

          // ジャンケンの結果分岐
          else if ((cpu.com == STONE && human.hand == PAPER)
        		  || (cpu.com == SCISSOES && human.hand == STONE) ||
                  (cpu.com == PAPER && human.hand == SCISSOES)) {
        	  cpu.Damag();//【主人公勝利】敵へのダメージ処理
          } else {
        	  human.Damag();//【敵の勝利】主人公へのダメージ処理
          }

		} while(human.humanHp > 0 && cpu.cpuHp >0); //<繰り返し条件>敵HPと主人公のHPが０以上で繰り返し

		//敗北時の処理
		if(human.humanHp <=0 ) {
			System.out.println("✖YOU LOSE✖");//敗北メッセージ
		}

		//勝利時の処理
		if(cpu.cpuHp <= 0 ) {
			System.out.println("☆YOU WIN☆");//勝利した場合のメッセージ

		}

	}
  }

public class Human extends Player{
	int damag =getDamagePoint();
    public int humanHp=getPlayerHp();
    public int hand ;
    public String HumanName=getName();

    Human human =new Human("あなた");
    Cpu cpu =new Cpu("敵");


	Human(String tmpName) {
		super(tmpName);

		// TODO 自動生成されたコンストラクター・スタブ
	}

	void Damag() {
        humanHp-=damag; // enemiHp = enemiHp - damag; 同じ意味
        print(human.getName(),cpu.getName(), damag);

        System.out.println();
        System.out.println(human.getName()+ " HP:" + human.humanHp + "/20");
        System.out.println(cpu.getName()+ " HP:" + cpu.cpuHp + "/20");
    }

	@Override
	public int  getHand() {

		// TODO 自動生成されたメソッド・スタブ
		 System.out.println("-----------------------------------------");
         System.out.println("コマンド? 0:グー　1:チョキ　2:パー");
         this.hand = scanner.nextInt();

		return 0;
	}

}

import java.util.Scanner;

public abstract class Player {


	private int playerHp = 20;
	private String name;

	public int getPlayerHp() {
		return playerHp;
	}

	public void setPlayerHp(int playerHp) {
		this.playerHp = playerHp;
	}

	public String getName() {
		return name;
	}

	public void setName(String name) {
		this.name = name;
	}

	protected static final Scanner scanner = new Scanner(System.in);
	java.util.Random rand = new java.util.Random();

	abstract public int getHand();

Player(String tmpName) {
        System.out.print(tmpName + "の名前を入力してください ▼ ");
        name = scanner.nextLine();
        System.out.println(name + "が入力されました");
    }

	/*ダメージ値とクリティカル率の設定*/
	protected int getDamagePoint() {
		int damag = rand.nextInt(3) + 1;//ダメージ 1～3のランダム設定
		if (rand.nextInt(10) == 9) { // クリティカル時 5倍
			damag *= 5; // damag = damag * 5; 同じ意味
		}
		return damag;
	}

	void print(String winner, String loser, int damage) {
        System.out.println(winner + "の勝ち");
        System.out.println(loser + "は" + damage + "のダメージを受けた");
    }

}

import java.util.Random;

public class Cpu extends Player{
	int damag =getDamagePoint();
    public int cpuHp=getPlayerHp();
   public int com ;


   Human human =new Human("あなた");
   Cpu cpu =new Cpu("敵");

	Cpu(String tmpName) {
		super(tmpName);

		// TODO 自動生成されたコンストラクター・スタブ
	}

	final Random rand = new Random();//Roundomの有効化


	void Damag() {
        cpuHp-=damag; // enemiHp = enemiHp - damag; 同じ意味
        print(human.getName(),cpu.getName(), damag);

        System.out.println();
        System.out.println(human.getName()+ " HP:" + human.humanHp + "/20");
        System.out.println(cpu.getName()+ " HP:" + cpu.cpuHp + "/20");
    }


	@Override
	public int getHand() {
		// TODO 自動生成されたメソッド・スタブ

		this.com = rand.nextInt(3);//敵の手の選択
		return 0;
	}

}

Comments

[Daregada]

まずは、「永遠と」(「延々と」の誤り)というバグを直しましょう。

[D-coder]

失礼しました。誤字の修正を行いました。

Answer 1

何が正しいのかはわかりませんが、下記のコードが原因かと思われます。

public class Human extends Player {

    Human human = new Human("あなた");

Humanクラスをインスタンス化すると、メンバ変数でHumanクラスがインスタンス化されてしまうため、これが繰り返されるのではないでしょうか。

Comments

[D-coder]

今,Human human = new Human("あなた");の部分無しで動くように変えたのですが、それでも繰り返されてしまいます。
HumanクラスのHP情報をCPUに
CpuクラスのHP情報をHumanに等々
いききさせる為にHuman human = new Human("あなた");や
Cpu cpu =new Cpu("敵");を使っていたのですが無しでとなると
どうすればいかわかりません。

[TN8001]

[Java - javaで作成したプログラムコードを短くしたい｜teratail](https://teratail.com/questions/315458

が元ネタなのですが、ここまで台無しになるとは思っていませんでした...（元もイケてないですが^^;

先方をコメント多め＆答え付きに編集しましたorz

Answer 2

Playerクラスの唯一のコンストラクタが次のものになります。

Player(String tmpName) {
        System.out.print(tmpName + "の名前を入力してください ▼ ");
        name = scanner.nextLine();
        System.out.println(name + "が入力されました");
    }

Playerコンストラクタが実行されるたびに名前入力が要求されることになります。
そしてPlayerコンストラクタは、Playerを継承したクラスのコンストラクタが実行されるたびに呼び出されます。
Playerを継承したHumanクラスにもCpuクラスにも、

    Human human =new Human("あなた");
    Cpu cpu =new Cpu("敵");

というメンバ変数が宣言されています。他の方の回答にもありますが、これにより、Humanのコンストラクタを再帰的に呼び出すことになり、そのたびに名前入力を要求され、終わりがないので無限に続きます。
Humanの方を消したところで、Cpuの方で全く同じ問題が起こるので無意味です。
まずはHumanクラスとCpuクラスの両方から、この2行を消してください。

また、コメントで書かれた

HumanクラスのHP情報をCPUに

CpuクラスのHP情報をHumanに

これはHumanやCpuが持つ必要は無く、第三者、例えばmainメソッドが見ればいい情報なので、行き来させる必要がありません。

Comments

[D-coder]

解答いただきありがとうございます。
繰り返しのエラーが解消し無事に行いたい処理が可能になりました。
コードの解説ありがとうございます。

Answer 3

繰り返される「xxの名前を入力させる処理」は、質問の３つ目のプログラムにある以下のメソッドだろうと思われます。
引数に"主人公"か"敵"を入れて使うことを想定しているのでしょう。
残念ながら入力された名前は、変数 name に代入されて、画面表示に使われるだけで、ゲームプレーヤーの名前の設定に使われていません。

Player(String tmpName) {
        System.out.print(tmpName + "の名前を入力してください ▼ ");
        name = scanner.nextLine();
        System.out.println(name + "が入力されました");
    }

そして、質問のプログラムに、上記のPlayerメソッドを呼び出すコードが見当たりません。
それなのに延々と繰り返しが起きるのだとすると、Playerメソッドを呼び出すコードを含むプログラムが質問に示されていないのだと思います。

質問に、全てのプログラムが提示されていますか？

Answer 4

ループを抜ける条件を満たしていないからでは。デバッグしてみてください。