Update challenge visitors to accept logical exprs

nop · Aug 16, 2023 · 87c5486 · 87c5486
1 parent a1391d1
commit 87c5486
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/src/main/java/com/craftinginterpreters/lox/AstPrinter.java b/src/main/java/com/craftinginterpreters/lox/AstPrinter.java
@@ -26,6 +26,11 @@ public String visitLiteralExpr(final Expr.Literal expr) {
 		return expr.value.toString();
 	}
 
+	@Override
+	public String visitLogicalExpr(Expr.Logical expr) {
+		return parenthesize(expr.operator.lexeme, expr.left, expr.right);
+	}
+
 	@Override
 	public String visitUnaryExpr(final Expr.Unary expr) {
 		return parenthesize(expr.operator.lexeme, expr.right);

diff --git a/src/main/java/com/craftinginterpreters/lox/RpnPrinter.java b/src/main/java/com/craftinginterpreters/lox/RpnPrinter.java
@@ -26,6 +26,11 @@ public String visitLiteralExpr(Expr.Literal expr) {
         return expr.value.toString();
     }
 
+    @Override
+    public String visitLogicalExpr(Expr.Logical expr) {
+        return expr.left.accept(this) + " " + expr.right.accept(this) + " " + expr.operator.lexeme;
+    }
+
     @Override
     public String visitUnaryExpr(Expr.Unary expr) {
         return expr.right.accept(this) + expr.operator.lexeme;