Discussion :

[JohnNC]

Bonjour,

Dans une classe, j'ai une HashMap<String,Object>. Pour récupérer les valeurs de cette map, j'ai plusieurs méthodes (extrait) :

Code :

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

1
2
3
4
5
6
7
8
9
public Object getValeur(String nom){
 return map.get(nom);
}
public String getString(String nom) {
 return (String) map.get(Nom);
}
public Date getDate(String nom) {
 return (Date) map.get(nom);
}

Je voudrais savoir si je peux réunir ces méthodes en utilisant un "cast" en générique, du genre :

Code :

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

1
2
3
4
5
@SuppressWarnings("unchecked")
public K getObjet(String nom) {
 Object valeur = map.get(Nom);
 return valeur instanceof K ? (K) valeur : null;
}

J'avais lu dans un post, une méthode correspondant à ce que je cherche, pour convertir un objet non typé en objet paramétré, mais je ne retrouve plus le code :/

[Uther]

Visiblement la map peut contenir des objets de type différents. Donc je ne vois pas vraiment ce que tu souhaites tirer de l'utilisation des génériques dans ce cas.

[JohnNC]

De simplifier le code : Au lieu d'avoir n méthode pour "précaster" mes objets (getDate, getNumber, getString, ...), je n'en aurais plus qu'une (K getObjet).
Ainsi, si j'ajoute un nouvel objet, je n'ai pas besoin de rajouter une méthode "getXXXX" dans toutes mes classes utilisant ce système.

J'avais lu ce genre de méthode pour convertir une List non typée en List typée, évitant d'avoir des @suppressWarning trop fréquent.

[tchize_]

on ne peux pas tester la valeur à l'exécution d'une instance par rapport à un type générique, ni faire de typecasting vers les type générique.


Quand tu écrit

Code :

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

return (K)valeur

c'est exactement le meme code binaire qui est généré que pour

Code :

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

return valeur;

L'information sur les types des générique n'est plus présente à l'exécution, elle ne sert qu'au compilateurs

[adiGuba]

Salut,


Il faut impérativement spécifier le type à un moment donné.
Le problème étant que le typage Generics ne permet pas de connaitre le type réel utilisé au final...

Tu pourrais faire ceci :

Code :

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

1
2
3
4
	@SuppressWarnings("unchecked")
	public <K> K getObject(String key) {
		return (K) map.get(key);
	}

Mais si la compilation se passe bien, tu n'es pas protégé des ClassCastException... mais le pire c'est que l'exception ne survient pas dans la méthode mais en dehors lors de l'affectation !


La solution serait de passer le type souhaité en paramètre :

Code :

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

1
2
3
4
5
6
7
	public <K> K getObject(Class<K> clazz, String key) {
		Object object = map.get(key);
		if (clazz.isInstance(object)) {
			return clazz.cast(object);
		}
		return null;
	}

Du coup la vérification peut avoir lieu à l'intérieur de la méthode mais c'est un peu plus "lourd" à l'utilisation

a++

[Uther]

De simplifier le code : Au lieu d'avoir n méthode pour "précaster" mes objets (getDate, getNumber, getString, ...), je n'en aurais plus qu'une (K getObjet).
Ainsi, si j'ajoute un nouvel objet, je n'ai pas besoin de rajouter une méthode "getXXXX" dans toutes mes classes utilisant ce système.

Les génerics ne sont pas un solution à ça: ton K devra être défini a la création de la classe qui l'utilise, donc ton getObject() retournera toujours le même type d'objet pour chaque instance de la classe.
Tu ne pourrais donc plus stoker des objets de type différents dans cette map.

[Uther]

Mais si la compilation se passe bien, tu n'es pas protégé des ClassCastException... mais le pire c'est que l'exception ne survient pas dans la méthode mais en dehors lors de l'affectation !

On ne pourrait pas faire?

Code :

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

1
2
3
4
5
6
7
8
9
@SuppressWarnings("unchecked")
public <K> K getObject(String key) {
    try {
        K object = (K) map.get(key);
        return object;
    } catch (ClassCastException ex) {
        return null;
    }
}

[adiGuba]

On ne pourrait pas faire?

Non car à l'intérieur de la méthode le cast est purement logique et ne sert qu'à la compilation...

Grosso-modo ce code revient à faire ceci :

Code :

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

1
2
3
4
5
6
7
8
public Object getObject(String key) {
    try {
        Object object = (Object) map.get(key);
        return object;
    } catch (ClassCastException ex) {
        return null;
    }
}

Bref il n'y a pas de cast en réalité. Il est déplacé à l'appel de la méthode :

Code :

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

String str = getObject("key");

Qui devient donc implicitement :

Code :

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

String str = (String) getObject("key");

Bref l'exception ne pourra pas être intercepter à l'intérieur de la méthode

a++

[JohnNC]

D'accord, merci pour vos réponses, ça me convient parfaitement