Discussion :

[84mickael]

Bonjour,

J'écris un DAO générique avec Spring/Hibernate, mais j'ai un problème de compilation :

Interface :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
/**
 * Read an entity
 * @param <Entity> Entity class
 * @param entityClass Entity class
 * @param id Entity ID
 * @return Entity
 * @throws DataAccessException Data access error
 * @throws IllegalArgumentException Null entityClass or null id
 */
public <Entity extends AbstractEntity> Entity read(Class<Entity> entityClass, Serializable id)
    throws DataAccessException, IllegalArgumentException;

Implémentation :

Code :

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

1
2
3
4
5
6
7
@Transactional(readOnly = true)
@Override
public <Entity extends AbstractEntity> Entity read(Class<Entity> entityClass, Serializable id) {
  ArgumentTools.checkNotNull(entityClass);
  ArgumentTools.checkNotNull(id);
  return GenericsTools.cast(getHibernateTemplate().get(entityClass, id)); // ERROR
}

Méthode utilitaire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
/**
 * Object cast
 * @param <I> Input type
 * @param <O> Output type
 * @param i Input object
 * @return Output object
 */
@SuppressWarnings("unchecked")
public static <I, O extends I> O cast(I i) {
  return (O) i;
}

Ca compile avec Eclipse, mais seulement parfois avec Maven :

type parameters of <O>O cannot be determined; no unique maximal instance exists for type variable O with upper bounds Entity,java.lang.Object

Une idée de la correction à apporter ?

Merci.

[CyberChouan]

Ton code est effectivement incorrect.

La généricité ne te permet pas de faire un cast vers un des types paramétrés (de même que tu ne peux pas en créer d'instance : "new O()" ne compilera pas).

De plus, ton code est fonctionnellement bizarre... Ton cast en un sous-type sans vérification a pas mal de chances de lever une ClassCastException.

[84mickael]

Dommage, ça me paraissait puissant et simple à utiliser !

La signature de la méthode utilisée (classe HibernateTemplate) :

Code :

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

public Object get(Class entityClass, Serializable id) throws DataAccessException

Si je l'utilise ainsi, ça obligera à faire un cast pour chaque appel de méthode.

[Invité]

Salut,

Ne peux tu pas passer un objet Class<O> en paramètre et utiliser la méthode cast de Class ?

A plus

[adiGuba]

Salut,

Ca compile avec Eclipse, mais seulement parfois avec Maven :

Que veux-tu dire par "parfois" ? Le même code peut compiler ou pas aléatoirement ???
Ou c'est la manière dont tu appelles la méthode qui entre en jeu ?



Sinon pourquoi ne pas utiliser la méthode cast de ton Class :

Code :

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

1
2
3
4
5
6
	public <E extends AbstractEntity> E read(Class<E> entityClass,
			Serializable id) {
		ArgumentTools.checkNotNull(entityClass);
		ArgumentTools.checkNotNull(id);
		return entityClass.cast( getHibernateTemplate().get(entityClass, id) );
	}

a++

PS : Evites d'utiliser des noms "long" pour les types paramétrés ce n'est pas très lisible : un ou deux lettres majuscules font parfaitement l'affaire tout en étant plus clair

[84mickael]

Ca marchait déjà comme ça (ma fonction GenericsTools.cast permet juste de ne pas éparpiller des @SuppressWarnings, mais elle ne fonctionne pas dans tous les cas) :

Code :

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

1
2
3
4
5
6
7
8
@SuppressWarnings("unchecked")
@Transactional(readOnly = true)
@Override
public <Entity extends AbstractEntity> Entity read(Class<Entity> entityClass, Serializable id) {
	ArgumentTools.checkNotNull(entityClass);
	ArgumentTools.checkNotNull(id);
	return (Entity) getHibernateTemplate().get(entityClass, id);
}

Mais alors comme ça c'est parfait (je ne connaissais pas la fonction Class.cast) :

Code :

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

1
2
3
4
5
6
7
@Transactional(readOnly = true)
@Override
public <Entity extends AbstractEntity> Entity read(Class<Entity> entityClass, Serializable id) {
	ArgumentTools.checkNotNull(entityClass);
	ArgumentTools.checkNotNull(id);
	return entityClass.cast(getHibernateTemplate().get(entityClass, id));
}

Merci !

[84mickael]

Que veux-tu dire par "parfois" ? Le même code peut compiler ou pas aléatoirement ???
Ou c'est la manière dont tu appelles la méthode qui entre en jeu ?

Bizarrement, le problème ne se posait pas à chaque "mvn install". Je ne saurais pas vraiment le reproduire, mais peu importe puisque vous m'avez proposé une meilleure solution.