Javaのメソッドの一つをランダムに呼び出す

Question

Javaで、複数のメソッドから一つをランダムに呼び出したいです。

slime,Dragon,Devilの中からランダムに取り出して、最終的に『勇者は(10,100,300のどれか)のダメージを受けた！』と表示することは可能ですか？
なお、継承などの練習をしていたので継承していますが、特に意味などはありません。
お願いします。
インデントなど不慣れなので、分かりにくかったら申し訳ありません。

import java.util.*;
class Enemy {
    public static int attack(){
        return 100;
    }
}

class Slime extends Enemy {
    public static int attack() {
        return 10;
    }
}

class Dragon extends Enemy {
    public static int attack() {
        return 100;
    }
}

class Devil extends Enemy {
    public static int attack() {
        return 300;
    }
}

public class Main {
    public static void main(String[] args) {
        Slime slime = new Slime();
        Dragon dragon = new Dragon();
        Devil devil = new Devil();
        ArrayList<String> teki = new ArrayList<String>();
           teki.add("slime");
           teki.add("dragon");
           teki.add("devil");
        int Rand = (int)(Math.random()*3);
    
    String moji = teki.get(Rand);
    System.out.println("勇者は" + moji.attack() + "のダメージを受けた！");
        
    }
}

Answer 1

意味がないとおっしゃってる継承ですが、それを生かしてください。
ListをEnemy型で宣言して、各敵のオブジェクトを入れます。

public class Main {
    public static void main(String[] args) {
        Slime slime = new Slime();
        Dragon dragon = new Dragon();
        Devil devil = new Devil();
        ArrayList<Enemy> teki = new ArrayList<>();
           teki.add(slime);
           teki.add(dragon);
           teki.add(devil);
        int Rand = (int)(Math.random()*3);

        Enemy selected = teki.get(Rand);
        System.out.println("勇者は" + selected.attack() + "のダメージを受けた！");

    }
}

Comments

[porpora_0316]

Mainメソッドをこの通りにしてみたのですが、selectedの中身は『Devil@39a054a5』『Slime@39a054a5』『Dragon@39a054a5』と変わっているのにも関わらず表記は『勇者は100のダメージを受けた！』になってしまいます。

[momon-ga]

attack()は、staticメソッドなので、オーバーライドされません。

[swordone]

では、Enemyクラスとその子クラス全てからstaticを消してください。

[porpora_0316]

ありがとうございます。無事に出来ました。
まだオブジェクト指向を勉強し始めたばかりなので、staticの意味やオーバーライドについて詳しく知りませんでした。
もっと勉強します。

Answer 2

これ、コンパイル通ります?
あまりJavaはやっていないですが、コンパイル通らない気が...
String moji = ...;
... moji.attack() ...
っておかしくないですか? そのまま読むと
Stringのattackメソッドを呼び出す です。 String::attack() なんてありましたっけ?

また、コンパイルエラーになるかどうかわかりません ( 試したことないから ) が、
なぜ Enemyを継承しているのにそれぞれ attackメソッドをstaticでやっているんでしょうか?
それなら継承しないほうがいいと思いますが。(強みが薄れる。)

---

私なら Enemyをそれぞれ継承して Enemyの配列かリストを用意する。

その前に、Enemyクラスやそれ周辺を何とかしないと...

今のところインターフェースで良さそうなのでinterface として Enemyを定義。

interface Enemy{
    int attack();
}

で、これを継承（厳密には実装だけど。）してスライムやらのクラスを定義。

public class Slim implements Enemy{
   public Slim( int id ){ id_ = id; }
   public int attack(){ return ap_; }
   private int ap_; // attack point. (攻撃力)
   private int id_; // このオブジェクトの番号?
}

こうすれば、

Slim slim1 = new Slim(1);
int a = slim1.attack();
// a のダメージを受けた!

みたいにできる。

でもこれはあまりよくない。

オブジェクト指向設計　getter, setterを使うなとはどういうことかを読むと、「データはオブジェクトに任せるべき」らしいです。

よって仕様そのものを変更します。

Enemyインターフェースよりも Chara抽象( インターフェースでもいいけど ) にしたほうがいいかな。

参考: アブストラクト

public abstract class Chara{
  abstract boolean attack( Chara c );
  abstract boolean attacked( int n ); // 攻撃されたときの処理 
}

これを継承してSlimやらを定義。

public class Slim implements Enemy{
   public Slim( int id ){ id_ = id; }
   public boolean attack( Chara c ){
           if( c.attacked( ap_ ) ){ // 攻撃して相手が HP = 0 なら
                return true;
           }
   return false;
   }
   public boolean attacked( int n ){
      hp_ -= n; // hp_ = hp_ - n; と同義
      if( hp_ >= 0 ) return true;
   return false;
   }
   private int ap_; // attack point. (攻撃力)
   private int id_; // このオブジェクトの番号?
   private int hp_; // HP
}

のようにHPやらMPやらはオブジェクトが管理するようにする。

で、attackメソッドに攻撃対象のオブジェクトを渡せば攻撃側のap_ ( 攻撃力 ) による攻撃が差し引かれて...っていう風に。

// mainメソッド等の呼び出し元で
Slim slim1 = new Slim(1);
Hero hero = new Hero(1);
slim1.attack( hero ); // 主人公に攻撃
hero.attack( slim1 ); // スライム( スライム1 ) に攻撃

みたいにできる。

即席だからものすごい適当で、本来ならattackedは親であるabstractではprotectedでmainメソッドのような外部からは触れないようにしたり、

コンストラクタをprotectedかなんかにしてデザインパターンのFactoryパターンかなんかで

1. ファイルから前回の攻撃力やらHPやらのデータを取得
2. そのデータからオブジェクトを生成して返す

みたいな感じでやったりするはずだけど、あくまで例だから省略。(スルー)

ここまでが準備段階。

で、問題の

slime,Dragon,Devilの中からランダムに取り出して、最終的に『勇者は(10,100,300のどれか)のダメージを受けた！』と表示することは可能ですか？

ですけど、これは私なら先ほどのChara抽象の配列かリスト等を生成。

List<Chara> enemies = new ArrayList<Chara>(); // 敵用arrayリスト

で、オブジェクト指向を取り入れた言語では

子クラスでnewして 親で保持することが可能です。

本来なら ArrayList<Chara> ... = new ArrayList<Chara>(); となっているはずですが、
List<Chara> ... = new ArrayList<Chara>(); としますよね?

それと一緒です。

参考: インターフェース

ということで

// List<Chara> enemies =...;
enemies.add( new Slim1(1) );
enemies.add( new Slim2(2) );
enemies.add( new Dragon(1) );
...

と登録しておき、乱数を発行して

// int r; // すでに乱数は発行されているものとして
enemies[r].attack( hero );

のようにできる。

もちろん、"スライム1によってダメージ10を受けた..." 的な文章も出せるはずだけど、そこは質問者さんが工夫してね。

Comments

[porpora_0316]

細かく教えて下さってありがとうございます。
ベストアンサーは下の方にしたのですが、参考にさせていただきます。