Add local functions and closure

src/main/fr/celticinfo/lox/Interpreter.kt

@@ -62,7 +62,7 @@ class Interpreter: ExprVisitor<Any?>, StmtVisitor<Unit>{
     }
 
     override fun visitFunction(stmt: Function) {
-        val function = LoxFunction(stmt)
+        val function = LoxFunction(stmt, environment)
         environment.define(stmt.name.lexeme, function)
     }
 

src/main/fr/celticinfo/lox/LoxFunction.kt

@@ -1,34 +1,28 @@
 package fr.celticinfo.lox
 
-class LoxFunction : LoxCallable {
+class LoxFunction(
-    private val declaration: Function
+    private val declaration: Function,
-
+    private val closure: Environment
-    constructor(declaration: Function) {
+) : LoxCallable {
-        this.declaration = declaration
-    }
 
     override fun call(interpreter: Interpreter, arguments: List<Any?>): Any? {
-        val environment = Environment(interpreter.globals)
+        val environment = Environment(closure)
 
         for (i in declaration.params.indices) {
             environment.define(declaration.params[i].lexeme, arguments[i])
         }
 
-        try {
+        return try {
             interpreter.executeBlock(declaration.body, environment)
+            null
         } catch (returnValue: LoxReturn) {
-            return returnValue.value
+            returnValue.value
         }
-
-        return null
     }
 
-    override fun arity(): Int {
+    override fun arity() = declaration.params.size
-        return declaration.params.size
+
-    }
+    override fun toString() = "<fn ${declaration.name.lexeme}>"
 
-    override fun toString(): String {
-        return "<fn ${declaration.name.lexeme}>"
-    }
 
 }

src/test/fr/celticinfo/lox/InterpreterTest.kt

@@ -477,5 +477,114 @@ print fib(10);
             System.setOut(standardOut)
         }
     }
+
+    @Test
+    fun `Function should work with return statement in block with shadowing`() {
+        val standardOut = System.out
+        val outputStreamCaptor = ByteArrayOutputStream()
+
+        System.setOut(PrintStream(outputStreamCaptor))
+
+        try {
+            val code = """
+fun fib(n) {
+    var a = 1;
+    if (n <= 1) {
+        return n;
+    }
+    return fib(n - 1) + fib(n - 2);
+    
+}
+
+print fib(10);
+ """
+            val scanner = Scanner(code)
+            val tokens = scanner.scanTokens()
+            val parser = Parser(tokens)
+            val statements = parser.parse()
+            assertEquals(2, statements.size)
+
+            Interpreter().interpret(statements)
+            val output = outputStreamCaptor.toString().trim()
+            assertEquals("55", output)
+        } finally {
+            System.setOut(standardOut)
+        }
+    }
+
+
+    @Test
+    fun `Closures should work`() {
+        val standardOut = System.out
+        val outputStreamCaptor = ByteArrayOutputStream()
+
+        System.setOut(PrintStream(outputStreamCaptor))
+
+        try {
+            val code = """
+fun makeCounter() {
+    var i = 0;
+    fun count() {
+        i = i + 1;
+        print i;
+    }
+    return count;
+}
+
+var counter = makeCounter();
+counter();
+counter();
+counter();
+ """
+            val scanner = Scanner(code)
+            val tokens = scanner.scanTokens()
+            val parser = Parser(tokens)
+            val statements = parser.parse()
+            assertEquals(5, statements.size)
+
+            Interpreter().interpret(statements)
+            val output = outputStreamCaptor.toString().trim()
+            assertEquals("1\n2\n3", output)
+        } finally {
+            System.setOut(standardOut)
+        }
+    }
+
+    @Test
+    fun `Closures should work with shadowing`() {
+        val standardOut = System.out
+        val outputStreamCaptor = ByteArrayOutputStream()
+
+        System.setOut(PrintStream(outputStreamCaptor))
+
+        try {
+            val code = """
+fun makeCounter() {
+    var i = 0;
+    fun count() {
+        var i = 0;
+        i = i + 1;
+        print i;
+    }
+    return count;
 }
 
+var counter = makeCounter();
+counter();
+counter();
+counter();
+ """
+            val scanner = Scanner(code)
+            val tokens = scanner.scanTokens()
+            val parser = Parser(tokens)
+            val statements = parser.parse()
+            assertEquals(5, statements.size)
+
+            Interpreter().interpret(statements)
+            val output = outputStreamCaptor.toString().trim()
+            assertEquals("1\n1\n1", output)
+        } finally {
+            System.setOut(standardOut)
+        }
+    }
+}