【解決】リファクタのお願い：プラグインコマンドで同じパラメーター名を使いまわしたい

[ムノクラ]

試作品です。
https://github.com/munokura/MNKR-MZ-plu ... Command.js

これを拡張していく予定なのですが、コードの通り
variableID
variable
setup
を繰り返し使う事になりそうです。

let pluginName = "MNKR_VariableCommand";
の下に
let variableID = Number(args.variableID);
let variable = $gameVariables.value(args.variableID);
let setup = eval(args.setup);

と入れてみたのですが、argsに上手く追加できないでエラーになってしまいます。

このようなケースの簡略化方法がございましたら、ご指導いただきたく、お願いいたします。

[ムノクラ]

https://github.com/munokura/MNKR-MZ-plu ... Command.js

関数化を思い出して、入れてみました。
一応動いているようですが、変な処理や書き方になっていないでしょうか？

[Plasma Dark]

1回使ってそれきりではダメでしょうか。わざわざ variableID, variable, setup のスコープを広げる理由が見当たりません。

registerCommandに渡している関数の中だけでスコープを閉じてしまうほうが結果として見通しが良くなりそうです。

コード: 全て選択

PluginManager.registerCommand(pluginName, "addActor", args => {
  const actorID = $gameVariables.value(Number(args.variableID));
  const doSetup = args.setup === "true";
  if (actorID) {
    if (doSetup) {
      $gameActors.actor(actorID).setup(actorID);
    }
    $gameParty.addActor(actorID);
  }
});

[ムノクラ]

1回使ってそれきりではダメでしょうか。わざわざ variableID, variable, setup のスコープを広げる理由が見当たりません。

registerCommandに渡している関数の中だけでスコープを閉じてしまうほうが結果として見通しが良くなりそうです。

コード: 全て選択

PluginManager.registerCommand(pluginName, "addActor", args => {
  const actorID = $gameVariables.value(Number(args.variableID));
  const doSetup = args.setup === "true";
  if (actorID) {
    if (doSetup) {
      $gameActors.actor(actorID).setup(actorID);
    }
    $gameParty.addActor(actorID);
  }
});

スコープについて下記記事を読みました。
https://techacademy.jp/magazine/25697

設計思想として、「スコープは狭めたほうがトラブルが防ぎやすく、メリットがなければ広げるべきではない」ということのようですね。

関数化したのは、シンプルに「同じことを何度も書かないで済む」のでメンテナンスもしやすいかな？と思ったのですが、浅知恵だったようです。
実直に1つずつ書いていくことにします。

下記進める前にご指導いただき、助かりました。
ありがとうございました。