Discussion :

[Oliveuh]

Bonjour,

J'ai un petit problème de modélisation qui je pense doit être tout bête (mais moi aussi apparemment).
J'ai 3 classes ClasseA, ClasseB, ClasseC qui implémement une interface MonInterface.

Ensuite j'ai une autre classe :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public class maClasse {
  public maMethode(MonInterface obj) {
    if (obj.getClass="ClasseA")
    // TraitementA
 
    if (obj.getClass="ClasseB")
    // TraitementB
 
    if (obj.getClass="ClasseC")
    // TraitementC
  }
}

Comment faire pour éviter le getClass() et faire quelque chose de propre ?

Merci beaucoup.
[ Modéré par vedaer ]
-> Ajout des balises code
[ Merci de respecter les Règles du Forum ]

[snoop]

avec instanceOf...

[Oliveuh]

c'est pas un peu sale d'utiliser getClass ou instanceOf ?

[Ptibibi]

L'opérateur (il s'agit d'un opérateur et non d'une méthode) instanceof est fait pour des cas comme ça :

Code :

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

1
2
3
4
if (obj instanceof ClasseA)
{
...
}

Et ce n'est pas du tout sale au contraire.

[snoop]

oui, parce que sinon je ne vois pas trop comment faire autrement...

[Oliveuh]

Je me prends la tête dessus depuis trop longtemps donc allons y pour intanceOf !

Merci à vous !

[staul]

Si tu veux vraiment éviter le getClass, tu te contentes de caster et de récuperer les exceptions :

Code :

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

1
2
3
4
5
6
 
try {
ClassA a=(ClassA)obj;
// code
} catch (ClassCastException e) {
}

Mais c'est vrai que instanceOf est fait pour ça, donc c'est encore ce qu'il y a de plus propre (encore que des fois c'est marant de se prendre pour un goret :-)

[Alwin]

Regarde du coté de la méthode isAssignableFrom(), c'est un instanceof dynamique et non statique.

Code :

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

1
2
3
if( a.getClass().isAssignableFrom(b.getClass()) ) {
  ...
}

[adiGuba]

Salut,

Déclare une méthode dans ton interface :

Code :

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

1
2
3
public interface MonInterface {
      public void maMethode ();
}

Tu sera obligé de l'implémenter dans toutes les classes qui implementent MonInterface :

Code :

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

1
2
3
4
public Class ClasseA implement MonInterface {
      public void maMethode () {
             // TraitementA
      }

Code :

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

1
2
3
4
public Class ClasseB implement MonInterface {
      public void maMethode () {
             // TraitementB
      }

Code :

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

1
2
3
4
public Class ClasseC implement MonInterface {
      public void maMethode () {
             // TraitementC
      }

Ensuite il te suffit de faire :

Code :

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

obj.maMethode();

Où obj est de type MonInterface. La methode correspondante à la classe réel de l'objet obj sera appellé...
C'est la justement l'interet d'utiliser des interfaces !

a++

[narmataru]

Salut,
J'allais te proposer la même solution qu'adiGuba. Pourquoi rechercher les type de tes objets alors que tu sais pertinamment qu'ils implémentent la même interface ??

Il suffit de faire une méthode traitement() dans l'interface et de l'instancier comme tu l'entend dans chacun de tes sous-classes. De cette manière tu n'a plus de cast ni de instanceof et encore moins de getClass().getName() (pour ma part quand c'est nécessaire je préfère instanceof).

bonne soirée...