Discussion :

[bomehc]

Bonjour,

J'ai une question concernant l'écriture d'un code propre lié à un héritage.

J'ai une classe abstraite parente (MaClasseAbstraite), et deux classes filles qui étendent cette classe (Classe1 et Classe2).
Dans MaClasseAbstraite, j'ai une méthode (maMethode) qui teste différents cas:

Code :

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

1
2
3
4
5
6
7
8
9
public boolean maMethode(){
	if(cas1){
	        appelleAUnePremiereMethode();
	}
	else if(cas2){
		appelleAUneSecondeMethode();
	}
	...
}

Cette méthode convient tout à fait pour la Classe1, mais dans le cas de la Classe2, j'aimerais rajouter un «else if(cas3)» dans la liste pour appeler un nouveau traitement spécifique à ma Classe2.

Comment faire cela proprement ?

En effet, je peux réécrire maMethode dans Classe1 et Classe2, mais cela impliquerait beaucoup de duplication de code puisque tous les premiers «if, else if» seraient identiques dans ces deux classes.
Je peux également tester dans MaClasseAbstraite si l'objet courant est une instance de Classe2, et effectuer le nouveau test dans ce cas, mais la classe parente n'a a priori pas de raison d'avoir accès à ses classes filles...

Quelqu'un a-t-il une idée pour résoudre ce problème proprement ?

Je vous remercie par avance !

[thelvin]

Le plus propre, c'est au cas par cas. Ça dépend de ce qu'il y a d'unique à ta situation.

De manière générale, dans ce cas-là, on peut imaginer ça :

Modifier MaClasseAbstraite comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
protected void prepareMaMethode() {
  if(cas1){
    appelleAUnePremiereMethode();
  } else if(cas2){
    appelleAUneSecondeMethode();
  }  // fini, rien d'autre, la méthode s'arrête là
}
 
public boolean maMethode(){
  prepareMaMethode();
  ...
}

Puis, dans ta Classe2 :

Code :

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

1
2
3
4
5
6
7
protected void prepareMaMethode() {
  if(cas3){
    appelleUneTroisiemeMethode();
  } else {
    super.prepareMaMethode();
  }
}

[dinobogan]

Si Classe1 n'est ni dans cas1 ni dans cas2, maMethode() ne fait rien ?

[bomehc]

Je vous remercie pour vos réponses.

En fait, Classe1 est forcément dans cas1 ou cas2, et Classe2 peut-être soit dans cas1, cas2 ou cas3.

Ta solution me parait pas mal thelvin. L'idée de passer par une méthode intermédiaire permet de bénéficier de l'héritage tout en évitant la répétition de code. Je devrais arriver à me débrouiller avec ça !

Merci beaucoup !