Discussion :

[stof]

Bonjour,

J'ai une liste d'objets de types différents.

J'aimerais récolter certaines de leurs infos mais pour cela, j'ai besoin de caster dans le bon type à chaque fois ;:

Code :

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

Class castClass = obj.getClass();

mais après?

Ce genre de chose ne marche pas :

Code :

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

1
2
castClass el = (castClass )(obj)
castClass.getName() el = (castClass.getName())(obj)

[CastorJoyeux]

tu ne peux pas, le type dans le typecasting (entre les parenthèse) doit être connu à la compilation. Dans ton cas il faut faire:

Code :

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

1
2
3
4
5
 
if (obj instanceOf ClassA)
  uneMethode((ClassA)obj);
else if (obj instanceOf ClassB)
  uneMethode((ClassB)obj);

Evidement le mieux serait soit de déjà avoir le bon type pour obj afin d'éviter une casting, soit d'avoir 'uneMthode' qui prendre directement un ClasseMere

[stof]

J'ai un peu modifié la demande.

Mais c'est horrible, j'ai potentiellement 10 classes et ça me fait faire quelque chose d'assez lourd (et pas bon à maintenir...)

[Rei Ichido]

Question bête : uneMethode n'est pas déplaçable dans les classes A, B, etc ?
Parce qu'une méthode qui dépend du type de l'objet à l'exécution, typiquement ça se case dans l'objet lui même
Cela dit, il arrive souvent qu'on soit un peu coincés, comme quand il y a 2 types d'objets mouvants. Si on peut suffisamment séparer les traitements, ça passe, sinon, bonjour le instanceof !

[CastorJoyeux]

oui, ben dans ce cas faut revoir ton design Comme je l'ai dit, revoir le code pour qu'il puisse utiliser la classe mère. Si c'est la même méthode avec 10 classes différentes, on peut supposer que ces classes ont suffisament en commun que pour soit travailler avec la classe mère, soit travailler avec une itnerface commune aux 10 classes concernées.

j'avais déjà vu ce genre de code dans une librairie, ils faisaient ainsi, c'était affreux mais c'était leur solution:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public int uneMethode(ClassA classA) {....}
public int uneMethode(ClassB classB) {....}
//.....
public int uneMethode(ClassN classN) {....}
public int uneMethode(Object o) {
if (o instanceOf ClassA)
   return uneMethode((ClassA)o);
if (o instanceOf ClassB)
   return uneMethode((ClassB)o);
//.....
if (o instanceOf ClassN)
   return uneMethode((ClassN)o);
}

[stof]

Bon, OK; c'est vrai que la conception est un peu bizarre des fois mais je ne suis pas là pour tout casser

En fait je m'en suis sorti par

Code :

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

obj.getClass().getMethod("uneMethode").invoke(obj)

entouré d'un try catch (20 minutes pour comprendre pourquoi le compilateur ne voulait pas, ça donne le niveau...)

[CastorJoyeux]

attention que ce code
-> ne survivra pas à un refactoring
-> ne marchera plus si un jour on remplace uneMéthode(ClassA) par une Méthode(ClassMere)
-> que ca appelle 'uneMéthode' sur l'objet et non pas appeler uneMéthode en lui passant le'objet mentionné, comme je l'avais compris dans ton post Initial.

En effet, finalement, ton code est l'équivalent de faire

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public ClassA extends ClassMere{
  public void uneMéthode(){....}
}
public ClassB extends ClassMere{
  public void uneMéthode(){....}
}
public ClassMere {
   public abstract void uneMethode();
}
 
 
... 
ClassMere cm = ...;
cm.uneMéthode();