Discussion :

[Altenide]

Bonjour,

J'ai une méthode qui me récupère une liste d'objets via une requete hql. Tous ce passe bien hormis ce warning : Type safety: Unchecked cast from List to List<Utilisateur>

sur

Code :

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

(List<Utilisateur>) session.createQuery("from Utilisateur").list();

Je ne vois pas trop comment checker le cast en question ..

Avez vous une idée ?? merci

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
	@Override
	public List<Utilisateur> listTousLesUtilisateurs() {
		log.debug("Entering in  List<Utilisateur> listTousLesUtilisateurs()");
 
		Session session = HibernateUtil.getSession();
		try {
			List<Utilisateur> q = (List<Utilisateur>) session.createQuery("from Utilisateur").list();
 
			return q ;			
		}
		finally {
			session.close();
			log.debug("Exit listTousLesUtilisateurs()");
		}		
 
	}

[SucreGlace]

C'est juste un warning...

Le compilateur ne peut pas certifier que le cast se passera bien, et il te prévient.
Si toi tu es sûr que la liste retournée sera bien une List<Utilisateur>, tu peux gaiement ignorer ce warning, cela n'empêchera pas le code de compiler ni de s'exécuter.

Si tu ne veux plus le voir, tu peux utiliser l'annotation @SuppressWarnings("unchecked").

[Altenide]

merci, c'est bien ce que je fais la plupart du temps sans trop aimer utiliser les supressWarnings. Je me demandais comment faire différemment.

[thelvin]

Je ne vois pas trop comment checker le cast en question ..

Normal, c'est impossible avec un cast .

Vu que les données sont produites par hql, on ne sait pas une liste de quoi est produite. Il n'y a aucun moyen de prouver au compilateur que c'est bien une List<Utilisateur>.

Si tu es absolument certain que ce code ne peut pas et ne pourra jamais fournir autre chose que des Utilisateur dans la List, alors tu peux sans doute ajouter un @SuppressWarnings("unchecked") :

Code :

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

1
2
@SuppressWarnings("unchecked")
List<Utilisateur> q = session.createQuery("from Utilisateur").list();

Dans le cas contraire, il vaut mieux vérifier avant d'essayer de persuader le compilateur. Moi je me fais la méthode suivante :

Code :

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

1
2
3
4
5
6
7
8
public static <T> List<? extends T> entype(List<?> list, Class<T> type) {
  for(Object item : list) {
    type.cast(item); // generate a ClassCastException if invalid
  }
  @SuppressWarnings("unchecked")
  List<? extends T> typedList = (List<? extends T>)list;
  return typedList;
}

Qui peut ensuite s'utiliser comme ça :

Code :

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

1
2
List<? extends Utilisateur> q =
  entype(session.createQuery("from Utilisateur").list(), Utilisateur.class);

[Altenide]

exactement ce que je tentais de faire. merci pour le coup de main.

[adiGuba]

Salut,


Même sans vérification manuelle, en cas d'erreur tu obtiendras une ClassCastException lors de l'utilisation de la liste.


Si la quantité de données est importante, il est donc préférable d'éviter de vérifier le type de toutes les données de la collection.


a++

[thelvin]

Même sans vérification manuelle, en cas d'erreur tu obtiendras une ClassCastException lors de l'utilisation de la liste.

Oui, mais là on l'a au moment où le code demande que la liste soit typée.

(sauf si la liste qu'on a typée est encore utilisée ailleurs et que cet ailleurs peut lui insérer un objet du mauvais type, auquel cas la méthode que j'ai présentée est inadaptée. La seule "solution sûre" est de copier la liste non typée dans une liste typée.)

Si la quantité de données est importante, il est donc préférable d'éviter de vérifier le type de toutes les données de la collection.

Oui.

[Altenide]

Envoyé par adiGuba
Si la quantité de données est importante, il est donc préférable d'éviter de vérifier le type de toutes les données de la collection.

Ok, je vais m'en tenir au supressWarnings dans ce cas.

[adiGuba]

Justement : dès qu'on a un warning sur les Generics (ou qu'on utilise un @SuppressWarnings("unchecked")), le code n'est plus sûr à 100% du point de vue compilateur.

Donc c'est le développeur qui doit prendre cette responsabilité en charge. A partir de là si on est sûr que la liste renvoyé contiendra uniquement des objets d'un certain type il n'y a pas de soucis à caster, et la vérification est une peu inutile... si ce n'est détecter l'erreur un peu plus tôt.

a++

[Altenide]

ok. merci j'ai compris.