Généricité : erreur de compilation (avec Maven)

Version imprimable

Bonjour,

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

Interface :

Code:

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:

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:

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 :

Citation:

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.

07/05/2009, 11h05
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.
07/05/2009, 11h50
Invité

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

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

Code:

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. :?
07/05/2009, 11h56
Invité

Salut,

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

A plus

Salut,

Citation:

Envoyé par 84mickael

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:

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 ;)

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:

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:

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 !

07/05/2009, 12h16
Invité

Citation:

Envoyé par adiGuba

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. :)