Réusinage et documentation de ‘EntityCondition’
L'organisation de la hiérarchie de classe EntityConditionBase
mériterait d'être repensée. Premièrement elle fait un usage abusif de l'héritage avec pour unique objectif de partager du code. Le diagramme suivant représente la hiérarchie de classes actuelle:
graph BT
EntityCondition-->EntityConditionBase
EntityOperator-->EntityConditionBase
EntityConditionValue-->EntityConditionBase
Il faut dans premier temps séparer EntityCondition
, EntityConditionValue
, et EntityOperator
en trois hiérarchies différentes:
-
Vider la classe EntityConditionBase
-
Supprimer l'héritage
La deuxième étape consiste à refactoriser et documenter EntityCondition
:
-
Transformer EntityCondition
en interface -
Remplacer la classe EntityConditionFunction
parEntityNotCondition
-
Supprimer le code mort -
Écrire la javadoc
On laisse de coté la refactorisation/documentation de EntityOperator
et EntityConditionValue
pour un autre ticket
Edited by Mathieu Lirzin