ツクマテ

JsonEx.parseはどのバージョンから使用できるのか

Plasma Dark さんが書きました:目指している仕様は

敵遭遇ごとに、出現する敵キャラが、その敵キャラのメモ欄に記入したIDリストの中からランダムに選んだIDの敵キャラに化ける。
IDリストに0が混ざっていて、ランダムなID選択で0が選ばれた場合、その敵キャラは存在しないものとして戦闘する。
（ただし、戦闘開始時点で敵キャラが全く存在しない状況は作らない）

でしょうか。

しぐれん さんが書きました:このあたりの細かい書き方は宗教戦争なので不毛ですが、個人的に気になった点のみを。

乱数を取る部分をMath.randomInt()にした方が良い。
ツクールMVの独自関数ですが、より明確。

reIdが数値と配列のどちらもあり得る状態になっており、型が固定されてない。
無効値を用意したいのであれば、nullやNaNを使うべきで空配列は不適切。

プラグイン内部にあるvarは再代入が無いので、いずれもconstに置き換えが可能。

以上です。
ロジック部分でも改善の余地はありますが、ひとまずはこのあたりで。

        // var randomEnemyId = Number(pool[Math.floor(Math.random() * pool.length)]);
        var randomEnemyId = Number(pool[Math.randomInt(pool.length)]);

Plasma Dark さんが書きました:変数及び関数の命名については検討したほうが良さそうです。

reId はおそらく random enemy Id のことだと思いますが、こういった略語は現在の自分以外に伝わらない可能性が大いにあります。
よっぽど一般的に（あるいはツクール一般に）通用するもの以外、略語は用いないほうが良いです。

noteTag という命名になっていますが、この関数の役割はメモ欄（noteTag）に書かれた敵ID一覧から一つ選択して返す、ですよね。
全く省略せずに書くならば selectRandomEnemyIdFromNoteTag でしょうか。
流石に長過ぎると思うなら、 selectEnemyId とかでも良いかもしれません。
関数の名前は、それが何をするためのものか名前から想像できるよう、動詞で始めるケースが多いです。

0のときに出現しないようにする仕組みは、コード中にもあるこれを利用するのが楽だと思います。

コード: 全て選択

if (member.hidden) {
  enemy.hide();
};

余り知られていないかもしれませんが、RPGツクールMVの戦闘には「途中から出現」なる仕組みがあります。
これはそのために用意されたコードで、戦闘開始時には途中から出現フラグ（hidden）が立っている敵をいないものとして扱います。
バトルイベントには「敵キャラの出現」コマンドがあり、それが実行されない限り、hiddenフラグが立った敵キャラの存在はゲーム中に現れません。

return randomEnemyId;

ムノクラ さんが書きました:しぐれんさんには昼間のTwitterで、そして、今回Plasma Drakさんからも言われたとおり「0の時にenemy.hide();」を使う処理を入れました。
（コードが汚いのは、ちょっと置いておいて…）

結果、敵グループ内が全キャラがhideになると、自動的に勝利するという状況が生まれました。

解決方法のアイディアとして、hideではない敵キャラが配置された時に、內部変数に数値を入れ、その数値が0の場合は再度グループ生成をやりなおす…というのが浮かびましたが、妥当な方向でしょうか？

現時点でのコード（全キャラがhide未対策）をアップします。

追記
と思ったら、0か1なので、trueかfalseの方が良いですよね