Discussion :

[didate]

Bonjour,

S'il vous plait, je cherche à savoir qu'es-ce qui est mieux entre les deux lignes de codes suivantes et pourquoi ?:

Code :

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

public Object execute(String Ps, boolean returnData,	Object... parameterValues)

Code :

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

public <T> Object execute(String Ps, boolean returnData,	T... parameterValues)

Exemple d'utilisation

Code :

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

this.execute("PS_TRUC", true, "V1",1,true);

sachant que lors de l'utilisation de la second, j'ai un warning @SuppressWarnings("unchecked")

Merci d'avance

[thelvin]

Question de point de vue, mais je dirais la première.

Le paramètre générique <T> est peu utile : il impose que tous les paramètres après les deux premiers soient de même type T.
Mais ce T n'est pas contraint et il a le droit de correspondre à Object, autant dire "pas de type commun," ce qui est le cas dans ton exemple ("V1", 1 et true étant de types différents.)

On pourrait le contraindre lors de l'appel :

Code :

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

1
2
3
4
5
this.<String>execute("PS_TRUC", true, "V1", "V2", "V3"); // Le <T> est forcé à String
 
// Ne compile pas :
// le <T> est forcé à String et 1 et true ne sont pas des String
this.<String>execute("PS_TRUC", true, "V1", 1, true);

Mais c'est léger.

Par ailleurs, s'il y a un @SuppressWarnings("unchecked") c'est que le générique n'est même pas vérifié et ne sert donc que pour contraindre l'appel à la méthode, si on pense à mettre une contrainte lors de l'appel (et qu'on en veut une, pas comme dans l'exemple donné.)
Bref, le générique a l'air plutôt vain.

[didate]

Merci thelvin ! je comprend mieux maintenant.

Qu'il est préférable d'utiliser le "T" si les différentes valeurs (T...parameterValues) sont de même type. Dans le cas contraire il est préférable d'utiliser "Object" .

J'ai testé le code suivant, et là le warning a disparu

Code :

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

this.<String>execute("PS_TRUC", true, "V1", "V2", "V3");

[Gugelhupf]

C'est une question de point de vue, mais ça dépend surtout du besoin.

Pour ma part, je n'aime pas utiliser Object, car je le considère comme un type passe partout.

[thelvin]

En l'occurrence, le <T> contraint par rien est tout autant un type passe-partout.
Attention à pas faire de cargo culte non plus...