攻撃メソッドの引数で呼んだオブジェクト

Question

現在Java言語でゲームを作っています。
2体のモンスターを作り、順番に攻撃させ、先に０にしたほうが勝ちという内容です。

モンスター２体にはそれぞれ名前、HP、引数でモンスターを呼びだしてHPをランダムにダメージを与えるＡttckメソッドを与え、
交互にAttackメソッドを実行し、お互いのHPどちらか０になるまで攻撃を続けると言った内容で以下のプログラムを書きました。

public class Monster {	

	private String name;

	private int hp;

	private Monster() {

	}


	private Monster(String name, int hp) {	

		this.name = name;
		this.hp = hp;

	}


	public String getName() {
		return name;
	}


	public int getHp() {
		return hp;
	}


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


	public void setHp(int hp) {
		this.hp = hp;
	}

	public void Attack(Monster m) {	//対象のモンスターを（オブジェクト）のHPをランダムに減らす

		System.out.println(this.name+"に攻撃！");



		Random damage = new Random();

		int	random= damage.nextInt(10)+5;


		this.hp = this.hp - random;

		if (this.hp < 0){
			this.hp = 0;
		}


		System.out.println(this.name + "に" +random + "ダメージ。残りHPは" + hp + "です");







	}


	public static void main(String[]args) {


		System.out.println("バトルスタート");

		Monster m1 = new Monster("スライム",100);

		Monster m2 = new Monster("ゴースト",100);







		for (;;) {




			m1.Attack(m2);


			if (m2.hp <= 0) {
				System.out.println("終了！スライムの勝利！");
				break;

			}


			m2.Attack(m1);

			if(m1.hp <= 0) {
				System.out.println("終了！ゴーストの勝利！");
				break;
			}

		}

	}


}

コード

しかし、この内容で実行したところ。
スライムに攻撃！　
スライムに●ダメージ！残りHPは●です。

ゴーストに攻撃！
ゴーストに●ダメージ！残りHPは●です。

引数で呼んだモンスターを攻撃するようにするにはどうすればよいのでしょうか？

Answer 1

Attackという関数が、引数にとったMonsterクラスのオブジェクトに対し攻撃するという意味なら、
攻撃の対象がthisになっているのが間違いです。

System.out.println(this.name+"に攻撃！");

thisは自分なので、自分への攻撃になります。
引数で取っている m に攻撃するようにしましょう。

System.out.println(m.getName() + "に攻撃！");

以下同様です。

あと、この件と直接関係ないですが、Monsterクラスの中でmain()を呼ぶのは、Monsterクラスとして不適切だと思います。
別途クラスを用意して、そちらからMonsterクラスのオブジェクトを作るのが適切と思われます。

Comments

[samhi]

返信遅くなってすみません
皆さま、回答ありがとうございます。

yukee様のアドバイスを基にもう一つ作ってみたいと思います

Answer 2

引数使い忘れてるああ

Answer 3

thisのname属性 は this.name で取得できます。
それでは mのname属性 はどのように取得すれば良いでしょうか？