引数のモンスターがドラゴンだった場合Heroの体力、盾の防御力に関わらず1/3の確率で即死する処理

Question

多態性を利用して引数でモンスターを受け取り、「モンスターのダメージ - 盾の防御力」の分のダメージを受ける処理に修正。また、引数のモンスターがドラゴンだった場合Heroの体力、盾の防御力に関わらず1/3の確率で即死する処理を作成したいのです。

しかし、コンソールしてみると、「NoNameの現在HPは100だ」のみとなってしまい、処理ができていないのです。

ヒントとして、下記のコードで「1/3の確率で即死する処理と通常時の処理を付け足せば
できる」と言われても上手くできません。
どなたかヒントやご回答をお願い致します。
public void receiveAttack(Monster m, int damage) {
System.out.println(m.nameNum() + "の攻撃！");
if (m instanceof Dragon) {//

class Hero extends Character {
  Sword sword;
  Shield shield;
 
  public Hero() {
    this.name = "NoName";
    this.hp = 100;
    this.maxHp = 100;
    this.mp = 0;
    this.maxMp = 0;
    this.sword = new Sword();
    this.shield = new Shield();
  }
 
  public Hero(String name, int hp, int mp, String sName, int sDamage, String shName, int shDefense) {
    this.name = name;
    this.hp = hp;
    this.maxHp = hp;
    this.mp = mp;
    this.maxMp = mp;
    this.sword = new Sword(sName, sDamage);
    this.shield = new Shield(shName, shDefense);
  }
  
  public void receiveAttack(Monster monster) {
	// TODO 自動生成されたメソッド・スタブ
  }
}

import java.util.Random;

abstract class Character implements Human {
  String name;
  int hp;
  int maxHp;
  int mp;
  int maxMp;
  static Random random = new Random();
 
  public void showHp() {
    System.out.println(this.name + "の現在HPは" + this.hp + "だ");
  }
 
  @Override
  public void receiveAttack(Creature creature) {
    if(creature instanceof Dragon &&
    random.nextDouble()<1.0/3.0 
    //↑「1.0/3.0」実数で計算しないと、整数で計算されてバグになる。
    ){
      this.hp = 0;
    }
  }
 
  @Override
  public void receiveAttack(Monster m, int damage) {
    receiveAttack(m);
  }
}

interface Human extends Creature {
  void receiveAttack(Monster m, int damage);
}

interface Creature {
  void receiveAttack(Creature creature);
}

abstract class Monster implements Creature {
  String name;
  String num;
  int hp;
  int damage;
 
  public void run() {
    System.out.println(this.nameNum() + "は逃げ出した");
  }
 
  public String nameNum() {
    return (this.name + this.num);
  }
}

class Dragon extends Monster {
  public Dragon() {
    this.name = "";
    this.num = "A";
    this.hp = 500;
    this.damage = 100;
  }
 
  public Dragon(String num) {
    this.name = "";
    this.num = num;
    this.hp = 500;
    this.damage = 100;
  }
 
  @Override
  public void receiveAttack(Creature creature) {
  }
}

class Shield extends Guard {
  final String type = "";
 
  public Shield() {
    this.name = "";
    this.defense = 0;
  }
 
  public Shield(String name, int defense) {
    this.name = name;
    this.defense = defense;
  }
}

abstract class Guard {
  String type;
  String name;
  int defense;
}

class Sword extends Weapon {
  final String type = "";
 
  public Sword() {
    this.name = "";
    this.damage = 0;
  }
 
  public Sword(String name, int damage) {
    this.name = name;
    this.damage = damage;
  }
}

abstract class Weapon {
  String type;
  String name;
  int damage;
}

public class Main {
  public static void main(String[] args) {
    Hero hero = new Hero();
    Dragon dragon = new Dragon();
    hero.receiveAttack(dragon);
    hero.showHp();
  }
}

Answer 1

class Hero extends Character {
  ...  
  public void receiveAttack(Monster monster) {
	// TODO 自動生成されたメソッド・スタブ
  }
}

この receiveAttack() を消さないと、Character#receiveAttack() が呼ばれないように見えます。

Comments

[marinoskun]

ご回答ありがとうございます。
試しに、Hero.javaのreceiveAttack()を消して、Character.javaのreceiveAttack()を残してみたのですが、結果が変わりませんでした...