aiscript icon indicating copy to clipboard operation
aiscript copied to clipboard

AiScriptErrorをErrorのサブクラスにするべきではない?

Open FineArchs opened this issue 1 year ago • 4 comments

https://github.com/aiscript-dev/aiscript/pull/278#issuecomment-1566320416 によると、JavascriptのErrorの生成にはスタックトレース収集などのコストがあると考えられるらしい AiScript上のエラーにJavascript上のスタックトレースの情報は特に必要ないと思われるため、Errorを引き継がなくてもいいのでは?

FineArchs avatar Oct 31 '24 02:10 FineArchs

AiScriptErrorが作られるのって、インタプリタが止まるときだけなんですよね。 制御構文のたびにthrowするなら邪魔なコストですが、ホスト側がキャッチできるエラーはErrorを継承しているべきだと思います。

uzmoi avatar Nov 24 '24 12:11 uzmoi

うーん、Errorを継承していて何か良いことってあります?

FineArchs avatar Nov 24 '24 14:11 FineArchs

catchした時に統一的な手法で処理出来る事か…

FineArchs avatar Nov 24 '24 14:11 FineArchs

そう考えると確かに別にErrorじゃなくする理由が無いな…

FineArchs avatar Nov 24 '24 14:11 FineArchs