Discussion :

[Laurent Dardenne]

Salut,
dans le code suivant je code une régle de gestion :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
private byte mAge;
/// Age d'une personne
public byte Age
{
  get
  {
    return (mAge);
  }
  set
 {
    if ((value==0) || (value<=AgeMaximum))
      mAge = value;
    else throw new AgeIncorrectException();
 }
}

Selon certains (*) cette approche, déclencher une exception dans un accesseur, n'est pas recommandé.

J'aimerais donc savoir s'il existe une autre solution que celle-ci pour coder une régle de gestion sur une propriété ?

[smyley]

Selon certains (*) cette approche, déclencher une exception dans un accesseur, n'est pas recommandé.

Ben vu qu'il n'y a pas 36 000 façons de définir une variable, il devrait préférer un truc du genre

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
byte age = 0;
public byte Age
{
  get { return age; }
}
 
public virtual bool DéfinirAge(byte n_age)
{
  if(n_age >= 0 && n_age <= AgeMaximum)
  {
    age = n_age;
    return true;
  }
  else
    return false;
}

(ceci dit, je fait tout le temps des exceptions dans les accesseurs et bon )

[Kaidan]

Bonjour,

Dans le Framework .NET, y'a beaucoup d'accesseurs qui retournent des exceptions, je vois pas vraiment où est le problème. C'est clair qu'il vaut mieux toujours vérifier les variables dans le code appellant, mais ce n'est pas forcément possible quand on écrit des bibliothèque et l'exception reste le mécanisme principal pour remonter une erreur.

[smyley]

...

Je suis entièrement d'accord avec toi mais en fait j'ai pas eu la foi de lire le document cité par Laurent ... ce serai bien s'il nous citait le passage dans lequel il est question de cette "proscription"

[Laurent Dardenne]

je vois pas vraiment où est le problème.

Débutant sous C# j'essaie de suivre des régles de développement ou tout du moins de prendre de bonnes habitudes.
Les autres régles que j'ai pu trouvées dans l'ouvrage indiqué ne me posent pas de pb particulier.

[smyley]

Débutant sous C#

ça fait bizarre, avant c'était Delphi non ?

j'essaie de suivre des régles de développement ou tout du moins de prendre de bonnes habitudes.
Les autres régles que j'ai pu trouvées dans l'ouvrage indiqué ne me posent pas de pb particulier.

Non mais en fait une des principales règles avec les exceptions c'est qu'il ne faut pas en abuser ou s'en servir quand c'est pas franchement nécéssaire car le procéssus de lancement d'une exception est gourmand en ressources et que de plus, le code résultant d'une utilisation massive des exceptions sera probablement obscur et difficil à maintenir. Cependant lorsque c'est pour signaler une erreur de valeur ( même si on vérifie avant la valeur entrée par l'utilisateur ) les exceptions sont très efficaces ( même dans un accesseur ). Au moin on est sur de cours circuiter toute donnée invalide ( par exemple dans ton cas, il n'y aura aucun moyen de se balader avec des ages négatifs ) et, pour moi, mettre la vérification dans l'accesseur est une méthode simple et élégante, en éssayant de faire autre chose on se complique la vie .

[Laurent Dardenne]

ce serai bien s'il nous citait le passage dans lequel il est question de cette "proscription"

page 69

Code :

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

1
2
 
14. Throw the InvalidOperationException exception if a call to a property set accessor or method is not appropriate given the object's current state.

Voir aussi le livre "Pratique de .NET 2 et C#2" de Patrick Smacchia, page 409
"Il est conseillé de ne pas lancer d'exception à partir d'un accesseur, à moins quelle soit rattrapée dans l'accesseur même. En effet syntaxiquement le client n'a pas nécessairement conscience d'appeler un accesseur lors de l'accés à une propriété. Le client peut donc être surpris par une exception."

ça fait bizarre, avant c'était Delphi non ?

Oui mais je ne suis pas trop perdu
Et puis je continuerai d'utiliser Delphi sous Win32.

en essayant de faire autre chose on se complique la vie

C'est pas mon objectif. J'essaie de trouver une implémentation simple pour cette recommendation.

[SirJulio]

Salut,

je n'ai pas trouvé le passage en question dans le livre (pas mal btw), mais je pense que ce qu'ils ont voulu dire, c'est que, theoriquement, une propriété 'simple' (le cas general) n'est pas sensé lever une exception, contrairement à une methode. Cependant, si cette levée est indiquée, (par un comment intellisense par exemple, comme le fait le framework), je ne vois pas ou est le probleme, c'est meme preferable, AMHA, de lever l'exception à l'initialisation de la variable en question, plutot que de lancer une InvalidOperationException, par exemple, plus tard (qui perdra de son sens).

[Laurent Dardenne]

ceci dit, je fait tout le temps des exceptions dans les accesseurs

C'est vrai que cette approche facilite le codage. De plus cette régle de développement me semble difficile à respecter