Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

関数の引数が省略された時のエラーについて #1716

Closed
snowdrops89 opened this issue Aug 5, 2024 · 2 comments
Closed

関数の引数が省略された時のエラーについて #1716

snowdrops89 opened this issue Aug 5, 2024 · 2 comments

Comments

@snowdrops89
Copy link

まず、v3.5.3までは、これが問題なく動いていました。

「A」と「B」で挨拶。

●(AとBでCの)挨拶
    Cを表示。
    もし、C=「朝」ならば、
        「{A}:おっはー」を表示。
        「{B}:おはよー」を表示。
    違えば、
        「{A}:こんにちは」を表示。
        「{B}:こんにちは」を表示。
    ここまで。
ここまで。

このときCには空が入っていて、空の場合の処理を書くことで省略時のデフォルト値を設定することも出来ました。。
なでしこ3には、v1のような関数の引数が省略された場合のデフォルト値を設定する機能がないため、いろいろ裏技を模索して、引数を省略できる関数を作っていました。

ところが~‼️
v3.6.2以降、これは、

[エラー]expected expression, got ','

なるJSのエラーが表示されるようになってしまいました~😭

できれば元の通りに使えるようになると助かります。
もちろん関数の引数不足はエラーとして、裏技ダメ🙅とゆう事であればそれはそれで仕方ないのですが~🥲
その場合はエラーメッセージはなでしこの、何行目のこの関数の引数が足りないよ! という形にした方が良いと思います。

そして・・・v1のようにデフォルト値を明示して引数省略の指定ができるようにすることはだいぶ難しいのでしょうか。

@kujirahand
Copy link
Owner

こちらの問題、お騒がせしました。。。
引数の問題がある場合に壊れたJSが生成される場合がある問題 #1729 で修正しました!
最新のv3.6.15で修正されています。

本来は、v1の引数のデフォルト値の指定ができると良いですよね。検討します。
関数のデフォルト値を指定できるようにする #1732

@snowdrops89
Copy link
Author

確認しました!
ありがとうございます❤

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants