aiscript icon indicating copy to clipboard operation
aiscript copied to clipboard

fn.argsの命名を見直す

Open marihachi opened this issue 1 year ago • 3 comments

関数の仮引数はargsではなくparamsとする方が自然なため fn.argsはfn.paramsに変更したい。

marihachi avatar Aug 04 '24 09:08 marihachi

argsparamsより不自然だという気はあまりしませんが、そういう文脈があるのですか?

FineArchs avatar Aug 04 '24 22:08 FineArchs

不自然というほどではないと思いますが、一緒だと混乱するので区別したいです。 区別するときは実引数はargument、仮引数はparameterとなります。 現在のfnの呼び出しの実装ではargdefという変数名が使われていますが、これよりはparamの方がわかりやすいと思います。

参考までにESTreeのFunctionsの定義→https://github.com/estree/estree/blob/master/es5.md#functions

uzmoi avatar Aug 05 '24 03:08 uzmoi

なるほど、実引数と仮引数の呼び分けは初めて知りました。ありがとうございます。 そういうことであればparamsに変更するのは良さそうですね。

FineArchs avatar Aug 05 '24 09:08 FineArchs