Mise en oeuvre d'un pattern, pb de conception

Bonsoir,

Dans le cadre de la mise en place d'un moteur de prise en charge de règles métiers à destination d'une application web, j'ai découvert le Pattern Specification avec la note de blog suivante (Le concept de cette pattern y est bien expliqué)

http://blog.xebia.fr/2009/12/29/le-p...regles-metier/

Cela correspond bien à mon besoin, cependant j’espérais pouvoir le rendre dynamique, de façon à parser les règles 'à chaud' et les appliquer, sans être obligé de modifier le code.

Je parviens à parser mes règles, j'en construis les objets adéquats (dérivant de LeafSpecification) que je place dans une liste.

Avant de complexifier plus le traitement, je cherche à chainer ces règles à l'aide de la fonction and(), ce qui au final me renvoie un objet AbstractCompositeSpecification.

Sur cet objet, je devrais pouvoir vérifier l'ensemble de mes règles avec la fonction isSatisfiedBy(), mais il semble que je parte dans un traitement infini..

Je bloque un peu, je suis un peu à la limite de mes capacités d'abstraction.. Mais j'aimerais comprendre..

Quelqu'un aurait une piste ?

Poste ton code qu'on jette un œil.

Je vais essayer :p

Toutes mes classes règles dérivent de la classe suivante

Code:

---

1 2 3

public abstract class RuleCategory extends LeafSpecification<Category> { }

---

Exemple d'une de mes classes "règles"

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

public class RuleCategoryMatch extends RuleCategory { private Category categoryMatch;   public void setCategoryMatch(Category categoryMatch) { this.categoryMatch = categoryMatch; }   @Override public boolean isSatisfiedBy(Category candidate) { if(candidate.equals(categoryMatch)) { return true; } return false; }   @Override public String toString() { return "Rule Category Match - match cat " + categoryMatch; } }

---

Exemple d'ajout d'une régle à la liste de règle

Code:

---

1 2 3 4

RuleCategoryMatch rule = new RuleCategoryMatch(); rule.setCategoryMatch(myCategory) listRules.add(rule);

---

fonction qui empile recursivement les regles

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

private AbstractCompositeSpecification recurseBuildRule(ArrayList <RuleCategory> list, RuleCategory curRule) { if(list.get(0) != null) { RuleCategory rule = list.iterator().next();   list.remove(0); if(list.size() > 0 && list.get(0) != null) { return (AbstractCompositeSpecification)curRule.and(recurseBuildRule(list, curRule)); } } return curRule; }

---

Puis cette portion de code fait appel à la fonction récursive pour construire l'empilement de règle

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

RuleCategory rule = (RuleCategory) listRules.iterator().next(); listRules.remove(0);   RuleCategory finalRule = (RuleCategory) recurseBuildRule(listRules, rule);   if (finalRule.isSatisfiedBy(testCategory)) { System.out.println("Rules are OK"); } else { System.out.println("Rules are KO"); }

---

Cela fonctionne mais je me demande si il n'y a pas moyen de concevoir cela autrement, par exemple en créant une classe FinalRule, qui "s'auto-construirait" en tant que classe finale, et à laquelle je passerais ma liste de règle au contructeur..

le tout s'appuie sur le schéma suivant
http://blog.xebia.fr/wp-content/uplo...cification.png

Merci de m'avoir lu, c'est bien difficile de décrire de tels problèmes :)

Merci !

En fait, je viens de trouver un facon de le faire fonctionner, mais je ne trouve pas ca élégant.. Enfin je suppose qu'il y a plus propre et fiable.

Je modifie mon post précedent

La méthode recurseBuildRule(ArrayList <RuleCategory> list, RuleCategory curRule) me semble pas très bien codée. Une simple boucle aurait suffit à la place de la récursivité. Mais c'est une affaire de goût :lol:

Ton code bloque (ou boucle) sur l'exécution ou la construction de tes règles ?

Il manque le code de l'exécution de tes règles isSatisfiedBy() pour le AND ainsi que celui de la construction du and().

Effectivement, ta recursion ne fonctionne pas.

le code suivant te reverra systématiquement le même element car la méthode iterator() va te renvoyer un nouvelle Iterator à à chaque appel.

Code:

---

RuleCategory rule = list.iterator().next();

---

donc ton objet rule sera systématiquement le premier élément de ta liste.
Pour bien faire, il aurait fallu passer l'itérator en paramètre et non la liste... où simplement mettre cela dans une boucle.

Si tu regarde bien il enlève le premier élément juste après

Code:

---

list.remove(0);

---

Enfin de toute manière, ce n'est pas propre.

Salut,

Merci pour vos réponses,

J'ai pris compte de vos conseils et j'ai laissé tomber la récursion ; au final, oui, c'est plus simple avec un simple boucle

Je vais chercher à produite un objet règle à qui l'on passe la liste de règle, et qui une fois initialisé en représente l'agrégation..

Si tu regardes sur le site dont tu as posté le lien en début de topic ce sont les opérators qui gèrent l'agrégation.

Constructeur du AND

Code:

---

1 2 3 4

public AndSpecification(Specification<T>... specifications) { super(specifications); }

---

Et tu peux ensuite faire

Code:

---

1 2 3 4 5

Specification specif = new AndSpecification( new Regle1(), new Regle2(), ... );

---

Mmmm je crois que je vois
Je me complique la vie donc ? :)

En fait je n'ai pas l'habitude d'utiliser les arguments variables

C'est ca qui me perturbe je crois

Tu peux remplacer les arguments variables par une List si tu préfères.