From 30d2215800deafbdcfb134f1356777ec0139bb7a Mon Sep 17 00:00:00 2001
From: Michael Williamson <mike@zwobble.org>
Date: Thu, 1 Oct 2015 21:49:18 +0100
Subject: Use normal method call exec in interpreter entry point

---
 .../zwobble/couscous/interpreter/Interpreter.java  | 68 ++++------------------
 1 file changed, 12 insertions(+), 56 deletions(-)

(limited to 'src/main/java/org/zwobble/couscous/interpreter/Interpreter.java')

diff --git a/src/main/java/org/zwobble/couscous/interpreter/Interpreter.java b/src/main/java/org/zwobble/couscous/interpreter/Interpreter.java
index bc82fee..884a082 100644
--- a/src/main/java/org/zwobble/couscous/interpreter/Interpreter.java
+++ b/src/main/java/org/zwobble/couscous/interpreter/Interpreter.java
@@ -1,20 +1,17 @@
 package org.zwobble.couscous.interpreter;
 
 import java.util.List;
-import java.util.Optional;
-import java.util.stream.IntStream;
+import java.util.stream.Collectors;
 
 import org.zwobble.couscous.Project;
-import org.zwobble.couscous.ast.ExpressionStatementNode;
-import org.zwobble.couscous.ast.MethodNode;
-import org.zwobble.couscous.ast.ReturnNode;
-import org.zwobble.couscous.ast.visitors.StatementNodeVisitor;
+import org.zwobble.couscous.ast.ExpressionNode;
+import org.zwobble.couscous.ast.LiteralNode;
 import org.zwobble.couscous.values.InterpreterValue;
 
-import static com.google.common.collect.Iterables.find;
-import static java.util.stream.Collectors.toMap;
+import com.google.common.collect.ImmutableMap;
+
+import static org.zwobble.couscous.ast.StaticMethodCallNode.staticMethodCall;
 import static org.zwobble.couscous.interpreter.Evaluator.eval;
-import static org.zwobble.couscous.values.UnitValue.UNIT;
 
 import lombok.val;
 
@@ -26,52 +23,11 @@ public class Interpreter {
     }
     
     public InterpreterValue run(String className, String methodName, List<InterpreterValue> arguments) {
-        val clazz = project.findClass(className);
-        val method = find(clazz.getMethods(),
-            method -> method.getName().equals(methodName));
-        val environment = buildEnvironment(method, arguments);
-        
-        for (val statement : method.getBody()) {
-            val result = statement.accept(new Executor(environment));
-            if (result.isPresent()) {
-                return result.get();
-            }
-        }
-        return UNIT;
-    }
-
-    private Environment buildEnvironment(
-        final MethodNode method,
-        List<InterpreterValue> arguments) {
-        
-        if (method.getArguments().size() != arguments.size()) {
-            throw new WrongNumberOfArguments(method.getArguments().size(), arguments.size());
-        }
-        
-        val stackFrame = IntStream.range(0, method.getArguments().size())
-            .boxed()
-            .collect(toMap(
-                index -> method.getArguments().get(index).getId(),
-                index -> arguments.get(index)));
-        return new Environment(stackFrame);
-    }
-    
-    private class Executor implements StatementNodeVisitor<Optional<InterpreterValue>> {
-        private Environment environment;
-
-        public Executor(Environment environment) {
-            this.environment = environment;
-        }
-        
-        @Override
-        public Optional<InterpreterValue> visit(ReturnNode returnNode) {
-            return Optional.of(eval(environment, returnNode.getValue()));
-        }
-
-        @Override
-        public Optional<InterpreterValue> visit(ExpressionStatementNode expressionStatement) {
-            eval(environment, expressionStatement.getExpression());
-            return Optional.empty();
-        }
+        val argumentExpressions = arguments
+            .stream()
+            .<ExpressionNode>map(LiteralNode::literal)
+            .collect(Collectors.toList());
+        return eval(new Environment(project, ImmutableMap.of()),
+            staticMethodCall(className, methodName, argumentExpressions));
     }
 }
-- 
cgit v1.2.3