Discussion :

[Heziva]

Bonjour,

Je cherche à rendre mon code le plus "joli" possible, en utilisant notemment le polymorphisme. Après différentes étapes, j'obtiens le code suivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
MonRetour ceQueJeVeux;
if (maCondition1(monParam)) {
    ceQueJeVeux = monTraitementCas1(monParamDeFonction);
} else if (maCondition2(monParam)) {
    ceQueJeVeux = monTraitementCas2(monParamDeFonction);
} else if (maCondition3(monParam)) {
   ceQueJeVeux = monTraitementCas3(monParamDeFonction);
} else {
   ceQueJeVeux = maFonctionParDefaut(monParamDeFonction);
}
return ceQueJeVeux;

Est il possible de simplifier en utilisant une interface, et un GoF builder ou factory ? Je ne suis pas expert en deseign pattern. Comment le mettre en place concrètement?
Connaissez-vous une liste de diffusion dédiée aux problèmes de "beaux" codes?

Merci d'avance !

[dingoth]

Bonjour, si ce ne sont que 4 ifs, ça passe encore. Ce code est pas mal.

Si tu en as davantage, je te conseille d'opter pour une interface :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
public interface MonInterface {
  public boolean condition (Object monParamDeTest);
 
  public MonRetour executer (Object monParamDeFonction);
}

Et d'ensuite avoir une liste d'implémentation de ces interfaces, et de la parcourir afin de faire le test et d'exécuter son code si cette implémentation est apte à le faire.

Si tu dois gérer bien plus de tests que ceux-ci, envisage peut-être une chaîne de responsabilité. Ce patron permet de passer un objet à un premier composant de la chaine. S'il accepte d'exécuter son code, il le fera et renverra son retour. Sinon, il demandera au suivant dans la chaîne de faire son propre test et donner son retour, et ainsi de suite. Le dernier dans la chaîne est bien sûr le comportement par défaut.

En fait, ces deux comportements sont similaires, mais c'est une question de goût.