ツクマテ

トリアコンタン さんが書きました:こんにちは！
まず、while文の仕様について確認しましょう。
https://developer.mozilla.org/ja/docs/W ... ents/while

while文は『判定式がtrueの間、処理を繰り返す』です。
判定式がfalseを返すとループを中断します。

問題は以下の通りです。
1. 判定式で代入『condition = true』している。
2. 本来ループを止めたいとき（hideでないとき）に逆にtrueを設定している。

修正案です。
仕様を完全に理解していないので要件通りに動作するとは限りませんが、無限ループは解消されると思います。

コード: 全て選択

var condition = true;
while (condition) {
    this.clear();
    this._troopId = troopId;
    this._enemies = [];
    condition = true;

    this.troop().members.forEach(function(member) {
        if ($dataEnemies[member.enemyId]) {
            var randomEnemyId = [];
            randomEnemyId = selectEnemyId($dataEnemies[member.enemyId]);
            var enemyId = member.enemyId;
            var x = member.x;
            var y = member.y;
            if (randomEnemyId === 0) {
                var enemy = new Game_Enemy(enemyId, x, y);
                enemy.hide();
            } else if (randomEnemyId) {
                var enemy = new Game_Enemy(randomEnemyId, x, y);
                condition = false;
            } else {
                var enemy = new Game_Enemy(enemyId, x, y);
                condition = false;
            };
            if (member.hidden) {
                enemy.hide();
            };
            this._enemies.push(enemy);
        };
    }, this);
    this.makeUniqueNames();
}

トリアコンタン さんが書きました:ただし、上記コードでも無限ループするパターンがあります。

それは、メモ欄に0だけ指定された場合です。

コード: 全て選択

<RandomEnemy:0>

おそらくこれは作者としては想定していない書き方だと思いますが、どのように扱うかは実装と言うよりは仕様決めの問題になると思います。

トリアコンタン さんが書きました:すみません、そこを問題点にあげるのを忘れてました。

var（constやletも）は『変数の宣言』です。つまり、conditionという名前の変数を使いますと宣言しています。
https://developer.mozilla.org/ja/docs/W ... ements/var

コード: 全て選択

var condition = true;

宣言は一番最初に使うときに一度だけすればOKです。
では、何度も宣言するとエラーになるのかというと、本来なら無視されるだけです。

今回のケースがなぜうまく動かないのか、それはローカル変数のスコープ（有効範囲）に関わる問題ですが、正しく理解するにはJavaScriptの仕様に関する知識が不可欠です。

下記で説明しますが、どうしても難しければ混乱を避けるためにも『変数の宣言は最初に使うときに一度だけ』と覚えておければOKです。
----------------------------------------------------------

簡単に説明すると、varで宣言したローカル変数のスコープは、宣言した関数の中（関数の中に関数が入っていればその中も含む）になります。
（ちなみにlet、constはさらに異なる仕様なのですが話が複雑になり過ぎるので割愛します）

実は、今回再定義した関数『setup』には入れ子になっている内部関数があります。
それはforEachの中です。

コード: 全て選択

this.troop().members.forEach(function(member) {

前述のとおり、varで宣言した変数のスコープは『宣言した関数の中』だけなので、forEachの中で宣言した変数は、forEachの中でだけ有効です。

簡略化するとこうなります。

コード: 全て選択

var condition = true;

this.troop().members.forEach(function(member) {
    var condition = false;
}, this);

console.log(condition); // trueが出力される。

----------------------------------------------------------

正直に告白しますと、まだ変数とクラスの違いさえ、読んでいてあやふやな感覚でいます。

randomEnemyIdは代入先と同じ名前ですが、これも別にすべきなのでしょうか？

(function() {
  //console.log(x); // 宣言前に表示しようとするとエラー
  let x = 100;
  console.log(y);  // undefined と表示される
  if (true) {
    let x = 300;
    console.log(x); // 300 と表示される
    var y = 50;
    console.log(y); // 50 と表示される
  }
  console.log(x); // 100 と表示される
  console.log(y); // 50 と表示される
})();

「途中から出現」の敵を指定するとエラーになります。

function selectEnemyId(arrayData) {
+  if (!arrayData.meta.RandomEnemy) {
+    return null;
+  }
  var pool = JsonEx.parse( `[${arrayData.meta.RandomEnemy}]` );
  var randomEnemyId = Number(pool[Math.randomInt(pool.length)]);
  return randomEnemyId;
};

Plasma Dark さんが書きました:

「途中から出現」の敵を指定するとエラーになります。

これはエラーメッセージから判断して、 RandomEnemy が指定されていない敵を出現させようとしていませんか。
つまり、 $dataEnemies[enemyId].meta.RandomEnemy が undefined になっているのだと思います。
JSON.parseは [undefined] をパースできずエラーになるので、 undefined の場合（つまり、ランダムに変化させたくない場合）のみ特別扱いしてあげましょう。
selectEnemyId 関数を以下のように修正することで治るんじゃないかと思います。

コード: 全て選択

function selectEnemyId(arrayData) {
+  if (!arrayData.meta.RandomEnemy) {
+    return null;
+  }
  var pool = JsonEx.parse( `[${arrayData.meta.RandomEnemy}]` );
  var randomEnemyId = Number(pool[Math.randomInt(pool.length)]);
  return randomEnemyId;
};

+と書いた行を追加するだけです。（もちろん、行頭の+は消してください）

        if (!arrayData.meta.RandomEnemy) {
            return randomEnemyId;
        };

        if (!arrayData.meta.RandomEnemy) {
            var randomEnemyId = null;
            return randomEnemyId;
        };

        if (!arrayData.meta.RandomEnemy) {
            var randomEnemyId = Number(1);
            return randomEnemyId;
        };