ネストした関数

[nomaddo]

def f() {
  def g(x) {
    print(x);
  }
  g(12);
}
f();

コレを実行するとException in thread "main" com.github.kmizu.nub.NubRuntimeException: function gis not definedで実行時エラーに落ちます。
これはトップレベルの関数はEvaluator.javaにおいてトップレベルにある関数はevaluateメソッドにより関数の環境であるfunctionsに保存されるのですが、それ以外は無視してvisitDefFunctionメソッドはnullを返すためです。

デザインの問題なので

• トップレベルしか関数定義を許さないようにパーサーを変更する
• より広い範囲の定義をトラバースしてローカルな関数定義を許す

くらいの解決策をが思い浮かびます。

[kmizu]

実装がトップレベルの関数をなめて関数の環境に保存している実装になっている辺りからわかるかもしれませんが、そもそもネストした関数は実装する予定はありませんでした。パーザが許してるので実行時エラーになっちゃってるという問題です。

パーザの中ではトップレベルと関数の中を区別しないように定義するとシンプルなので現状そうなっている感じです。そして、パーザをこれ以上複雑にはしたくないので、ネストした関数はevalする前にチェックして排除するのが落とし所かなあと思っています。