Discussion :

[Pilloutou]

Bonjour,

je m'interroge sur un point.
J'ai une méthode qui doit ajouter un objet dans une liste si celui-ci n'existe pas déjà dans la liste.

Code c# :

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

void AddObject ( MyClass obj);

Si l'objet a été ajouté à la liste, il faut l'indiquer à l'objet appelant pour qu'il effectue une action.
Si l'objet n'a pas été ajouté à la liste, il faut aussi l'indiquer à l'objet appelant pour qu'il effectue une autre action.

Que vaut-il mieux faire ?
Utiliser un paramètre de retour de type bool:

Code c# :

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

1
2
3
4
5
 
if (AddObject(myObj))
      DoAThing();
else
      DoOtherThing();

Ou vaut-il mieux modifier la référence de l'objet pour la faire pointer vers nulle:

Code c# :

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

1
2
3
4
5
6
7
 
AddObject(ref myObj);
 
if (myObj==null)
     DoAThing();
else
     DoOtherThing();

Ou vaut-il mieux lancer une exception ?

Code c# :

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

1
2
3
4
5
6
7
8
9
10
 
try
{
      AddObject(myObj);
      DoAThing();
}
catch (AddObjectException ex)
{
      DoOtherThing();
}

avec

Code c# :

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

1
2
3
4
5
 
public class AddObjectException:Exception
{
    [..]
}

Pourriez-vous m'éclairer sur ce point ?

Merci bien à vous.

[Thomas Lebrun]

Avec ce que tu demandes, j'opterais pour le booléen: c'est le plus rapide à mettre en oeuvre.

L'idée de l'exception est pas mal non plus, mais d'après moi uniquement utile si tu comptes réutiliser ta classe d'exception plus tard

[Pilloutou]

Merci bien pour la réponse.

J'avais peur que le booleen fasse plus langage procédural comme le C et pas trop objet.

Je ne pense pas réutiliser l'exception MyObjectException.
J'ai mis cette exception juste pour l'exemple. Je pourrais tout aussi bien mettre simplement Exception plutôt que MyObjectException.

Donc dans ce cas, vu sous cet angle, tu me conseillerais d'utiliser le try catch même si comme je l'ai aussi toujours lu try catch est plus long à l'execution.
En fait, ne serait-ce pas une question d'habitude d'utilisation. Certains préféront utiliser l'exception, d'autre le retour booleen ?

[Nip]

Pour faire court, le try catch pour gerer un comportement normal de ton application, c'est non non non! Une application dans son mode de fonctionnement normal NE DOIT PAS lancer d'exception. Outre la baisse de perfs, semantiquement parlant ...ben c'est pas bien , une levee d'exception doit se produire exceptionnellement (ce qui n'empeche pas d'utiliser le try catch pour blinder son code). Si t'es dans une methode de validation quel est l'interet de lever une exception pour traiter un de tes cas a valider?

Le booleen est une meilleure solution dans ton cas
Ce que tu peux faire aussi c'est utiliser une hashlist ou dictionary, avec comme cle ton objet: tu verifies si ton objet est deja present dans le dictionnaire et tu l'ajoutes si necessaire.

[Pilloutou]

Ok je te remercie bien pour cette réponse fort interessante.

Pour faire court, le try catch pour gerer un comportement normal de ton application, c'est non non non! Une application dans son mode de fonctionnement normal NE DOIT PAS lancer d'exception.

Maintenant que tu m'en parles, je l'avais lu.
.NET 2.0 et C#2 Livre très interessant acheté suite aux conseils sur ce site de Morpheus.

Bah en tout cas merci encore.