Discussion :

[stujava]

Bonjour,

Je suis confronter à du code assez "nébuleux" et j'aimerai pouvoir l'éclaircir, j'a déjà quelques idées mais j'aimerai votre avis.

Voici donc comment se présente le code, il y une suite de if{} else{} renfermant les mêmes appels et presque le même code si ce n'est quelques initialisations différentes et quelques paramètres différents, la seule chose importante qui diffère dans chacun des if est un test définissant le type d'objet en entrée.

Mon souhait serait de supprimer les IF et de le remplacer par une interface qui serait implémentée dans chacunes des classes. En tenant compte que dans ma classe appelante je n'aurais qu'une ligne qui appelerait une méthode dans l'interface et qui serait automatiquement dirigée vers la bonne classe.

Qu'en pensez-vous ? Avez-vous des idées, suggestions, remarques ?

Merci d'avance pour votre aide.
Stujava

[william44290]

Met ton code

[michon]

Comme ca, j'ai envie de dire : "oui, pourquoi pas".

Après je pense que le code en question serait surement utile afin d'y voir plus clair, et pourquoi pas ta proposition de changement. Car suivant les appels et autre, cela peut être intéressant ou non...

[stujava]

Désoé mais je ne peux mettre mon code sur le forum.

[william44290]

Pourquoi, tu travailles pour les Services Secrets ?

un extrait devrait suffire

[stujava]

Non je n'ai pas cette chance de travailler pour les services secret mais je suis sur mon machine perso et pas sur la machine où se trouve le code et impossible de transférer le code d'une machine à l'autre

[michon]

C'est malheureux...

Je ne vois donc pas comment nous pourrions te donner notre opinion sans savoir vraiment de quoi il s'agit...

Sache qu'on ne souhaite pas tout ton projet, juste ton enchainement de boucle, quitte à en changer le nom des variables et méthodes si cela te convient mieux... mais sans support, on ne pourra pas apporter de vrais reflexion, juste des suppositions... ce qui ne te fera pas avancer.

[stujava]

Je comprends pas de souci.

[professeur shadoko]

si je comprends bien tu cherches à faire un truc dans ce genre :

Code :

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

1
2
3
4
 
interface Faiseur {
        Truc faisLeAvec(Machin machin) ;
}

tes différents objets implantent cette interface et après tu veux faire :

Code :

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

1
2
3
4
 
   for(Faiseur acteur : lesDifférentsActeurs){
            listeRes.add(acteur.faisLeAvec(ça)) ;
   }

ça me semble une bonne idée pour remplacer des if(typeObjet)... c'est quoi ta question?

[Rei Ichido]

À noter que ça ne sera pas une "optimisation" au sens performances ; par contre, ça peut fortement améliorer la lisibilité et la facilité de maintenance du code.

En particulier, si comme tu le dis il y a des morceaux de code commun, n'hésite pas à le factoriser.

[tchize_]

Rei: ben on peu chipoter, si on passe de

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
if (x instanceof TypeA){
  TypeA a = (TypeA) x;
  a.machin();
  a.bidule();
  a.truc();
} else if (x instanceof TypeB){
  TypeB a = (TypeB) x;
  b.machin();
  b.bidule();
  b.truc();
}.....
} else if (x instanceof TypeZ){
  TypeZ z = (TypeZ) x;
  z.machin();
  z.bidule();
  z.truc();
}.....

à

Code :

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

1
2
3
4
5
public interface Generik {.....}
Generik g = (Generik) x;
g.machin();
g.bidule();
g.truc();

on va economiser quelques tests dans le cas Z

[Rei Ichido]

Effectivement, vu comme ça (je viens de relire le post initial que j'avais lu en diago), ça gagne epsilon ^^