java

Question

下記のjavaソースコードはクラス型配列を使用してランダムでモンスターを2~5体作成してランダムで行動させてfalseの場合にrun()メソッドを実行した要素にnullを代入してすべてnullになったら終了するコードなのですが,このコードの実行例としてはこのような感じです.
ゴブリンAが現れた！

ゴブリンBが現れた！

ゴブリンCが現れた！

ウォーウルフAが現れた！

ゴブリンCナイフで斬りつける
ゴブリンAはトコトコ逃げた!
ゴブリンCはトコトコ逃げた!
ゴブリンBはトコトコ逃げた!
ウォーウルフAはトコトコ逃げた!

本来ならこの続きにすべてのモンスターは消えた!と表示されるはずなのですが,ここで止まってしまいます
原因を教えてください.お願いします

コード
```public class Task43 {

	public static void main(String[] args) {
		// 発生させるモンスターの数を2〜5の範囲で生成する
		int r = 0;
		Random random = new Random();
		while(r < 2){
			r = random.nextInt(5);
		}
		// 生成するモンスターを格納する配列を作成
		Monster[] monsters = new Monster[r]; 
		int[] cnt = {0,0,0};
		// モンスターをランダムに生成
		for(int i = 0;i < r;i++){
			int m = random.nextInt(2);
			if(m == 0) {
				monsters[i] = new Goblin((char)(65 + cnt[0]));
				cnt[0]++;
			}
			else if(m == 1){
				monsters[i] = new WarWolf((char)(65 + cnt[1]));
				cnt[1]++;
			}
			else{
				monsters[i] = new DeathBat((char)(65 + cnt[2]));
				cnt[2]++;
			}
		}
		// 生成したモンスターを表示
		showMonsters(monsters);
		
		// モンスターがランダムに行動
		do{
			if(monsters != null){
				int i = random.nextInt(monsters.length);
				boolean f = random.nextBoolean();
				if((f)&&(monsters[i] != null)){
					monsters[i].attack();
				}
				else if((!f)&&(monsters[i] != null)){
					monsters[i].run();
					monsters[i] = null;
				}
			}
				
		}while(monsters != null);
		// 全てのモンスターが逃げ出したら表示
		System.out.println("\nモンスターは全て逃げ出した！");
	}

	/**
	 * モンスターの一覧を表示するクラスメソッド
	
	 */
	private static void showMonsters(Monster[] monsters) {
		for(int i = 0;i < monsters.length;i++){
		System.out.println(monsters[i] + "が現れた！\n");
		}
	}
}

Answer 1

動作させて確認はしていませんが、次のように変更をすると良いと思います。

do {
  ...
} while(monsters != null); 

の部分を

while(countLivingMonster() > 0) {
   ...
}
// 生きているモンスターの数を調べる。
public static int countLovingMonster(Monster[] monsters) {
    int count = 0;
    for (Monster m : monsters) {
        if (m != null) {
            count++;
        }
    }
    return count;
}

のようにします。

でも、swordone さんの回答にあるように
Monster を配列でなく ArrayList で保持するようにし、
生きているモンスターだけを ArrayList に保持する
ほうが良いとおもいます。
あるいは、ArrayList の内容を変化させる代わりに、
Monster クラスに getStat() メソッド(モンスターが生きているかなどの状態を得るメソッド)
を追加し、
生きているモンスターの数を数えて while を抜けるかを判断するようにする
という案もあります。

Answer 2

while(monsters != null);

これは、monstersという「配列」がnullにならないとループを抜けません。
monstersの中身が全てnullになるのとは意味が違います。
中身がすべてnullになっても、配列自体はまだ存在するため、ループから抜けていません。
(つまり、「止まっている」のではなく、無限ループに陥っている)

配列ではなく、ArrayListを使うといいと思います。
発生したモンスターをArrayList<Monster>に入れ、
リストのサイズの範囲で乱数を発生させて行動モンスターを選び、
逃げたモンスターはリストから取り除き、
行動をリストの中身がなくなるまで繰り返す、という処理になります。
選んだ中身のnullチェックも必要なくなります。

他、気になる点をいくつか…

乱数で2~5を発生させたいなら、ループ処理ではなく、

r = random.nextInt(4) + 2;

の方がいいです。ループする必要なく1回で出せます。

//モンスターをランダムに生成
の部分で書いているこの文

            int m = random.nextInt(2); 

は、0か1しか返さないので、この後のif-else文のelseには絶対に飛びません。
2も欲しいのであれば、引数は3である必要があります。