Refactor `BooleanExpressionVisitor`
jakobbraun opened this issue · 1 comments
jakobbraun commented
Currently, BooleanExpressionVisitor uses visit and leave. I've got the feeling that it would be better style to only use visit in combination with recursive calls, but I'm not sure about that.
jakobbraun commented
By that we could also rewrite the current logic:
connect(and);
this.connectorDeque.push(" AND ");
to something more readable like
Sering.joing("AND", ...)