[
https://issues.apache.org/jira/browse/OFBIZ-10593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16647939#comment-16647939 ]
Gil Portenseigne commented on OFBIZ-10593:
------------------------------------------
Hello [~mthl], i did review your final patch to refactor to a proper visitor pattern.
Great job ! I only found one typo into the well detailed Javadoc ;).
This implementation is far more understandable, and will hopefully ease OFBIZ-10579 improvement by [~suraj.khurana].
If nobody already did yet, we need to create a Jira to isolate EntityOperator and EntityConditionValue from EntityConditionBase. I'll check it out.
Thanks !
> ‘EntityConditionVisitor’ is a confused visitor pattern
> ------------------------------------------------------
>
> Key: OFBIZ-10593
> URL:
https://issues.apache.org/jira/browse/OFBIZ-10593> Project: OFBiz
> Issue Type: Improvement
> Affects Versions: Trunk
> Reporter: Mathieu Lirzin
> Assignee: Mathieu Lirzin
> Priority: Major
> Attachments: OFBIZ-10593_Implement-EntityConditionVisitor-properly.patch
>
>
> {{EntityConditionVisitor}} interface is supposed to implement the visitor pattern which supposes a set of classes meant to be visited using an {{accept}} method and a visitor with multiple {{visit}} method overloads (one for each visited class).
> Currently {{EntityConditionVisitor}} contains both {{accept}} and {{visit}} methods which make *no sense*
> {code:java}
> public interface EntityConditionVisitor {
> <T> void visit(T obj);
> <T> void accept(T obj);
> void acceptObject(Object obj);
> void acceptEntityCondition(EntityCondition condition);
> <T extends EntityCondition> void acceptEntityJoinOperator(EntityJoinOperator op, List<T> conditions);
> <L,R,T> void acceptEntityOperator(EntityOperator<L, R, T> op, L lhs, R rhs);
> <L,R> void acceptEntityComparisonOperator(EntityComparisonOperator<L, R> op, L lhs, R rhs);
> void acceptEntityConditionValue(EntityConditionValue value);
> void acceptEntityFieldValue(EntityFieldValue value);
> void acceptEntityExpr(EntityExpr expr);
> <T extends EntityCondition> void acceptEntityConditionList(EntityConditionList<T> list);
> void acceptEntityFieldMap(EntityFieldMap fieldMap);
> void acceptEntityConditionFunction(EntityConditionFunction func, EntityCondition nested);
> <T extends Comparable<?>> void acceptEntityFunction(EntityFunction<T> func);
> void acceptEntityWhereString(EntityWhereString condition);
> void acceptEntityDateFilterCondition(EntityDateFilterCondition condition);
> }
> {code}
> this confusion is visible in the {{EntityCondition}} which has both a {{visit}} and an {{accept}} method, even if it is only supposed to accept a visitor.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
Locked
