Discussion :

[Morbo]

Salut,
Je commence tout juste à essayer de faire mes propres classes génériques, ou plutôt méthodes en l'occurence. Donc peut être que je ne prends tout simplement pas la bonne approche.

J'ai définis les interfaces suivantes :

Code :

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

1
2
3
4
public interface Persist
{
	public Object getKey();
}

Code :

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

1
2
3
4
public interface PersistWithIntegerKey extends Persist
{
	public Integer getKey();
}

Code :

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

1
2
3
4
public interface PersistWithStringKey
{
	public String getKey();
}

Et dans une classe de type "fabrique" j'ai créé les méthodes suivantes :

Code :

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

1
2
3
4
5
6
7
8
9
public <Ts extends PersistWithStringKey> Ts getObjet(Class<Ts> c, String stringId) throws SQLException
{
	return (Ts)sqlMap.queryForObject("get"+c.getSimpleName(),stringId);
}
 
public <Ti extends PersistWithIntegerKey> Ti getObjet(Class<Ti> c, Integer intId) throws SQLException
{
    	return (Ti)sqlMap.queryForObject("get"+c.getSimpleName(),intId);
}

Or l'appel suivant ne semble pas valide

Code :

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

DataModelManager.getInstance().getObject(RefMetier.class,refMetierId)

Alors que RefMetier implémente bien l'interface PersistWithIntegerKey :

Code :

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

public class RefMetier implements PersistWithIntegerKey

J'ai sans doute du louper un truc, donc si vous avez une idée .

[adiGuba]

Salut,


Quel est l'erreur exact ? Au passage j'espère que c'est une erreur de copie mais tu as getObjet et getObject...


Sinon au lieu de multiplier les interfaces, pourquoi ne pas utiliser directement ceci :

Code :

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

1
2
3
4
public interface Persist<T>
{
	public T getKey();
}

a++

[Morbo]

Arg, l'erreur de débutant... eclipse m'orientait vers autre chose, bref, c'était une belle erreur de syntaxe.
Et tu as également completement raison pour les interfaces : refactorting en cours!

[Morbo]

Hum, si je peux continuer la conversation :
Si je passe par des interfaces génériques comment va se traduire ?

Code :

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

public <Ti extends PersistWithIntegerKey> Ti getObjet(Class<Ti> c, Integer intId) throws SQLException

?

[Morbo]

Bon ok :

Code :

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

 public <Ti extends Persist<Integer>> Ti getObject(Class<Ti> c, Integer intId)

[adiGuba]

Et selon la signature de ta méthode queryForObject() (si tId est de type Object), tu pourrais même utiliser une seule et unique méthode getObjet() en utilisant deux types Générics :

Code :

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

1
2
3
4
public <T, P extends Persist<T>> P getObjet(Class<P> c, T tId) throws SQLException
{
    	return (Ti)sqlMap.queryForObject("get"+c.getSimpleName(),tId);
}

Avec T qui correspond au type de base (Integer ou String dans tes exemples), et P correspondant du type Persist correspondant

a++

[Morbo]

En effet, c'est bien mieux, merci .