Discussion :

[User.Anonymous]

J'utilise un EventHandler dans un setters pour propager un changement de valeur.

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
18
 
		public EventHandler OnTotalChanged;
 
		private double _total;
		public double Total
		{
			get
			{
				object o = _total;
				return o == null?0:_total;
			}
			set
			{
				_total += value;	
				//previent les abonnés que le total a ete modifie
				OnTotalChanged(value,EventArgs.Empty);
			}
		}

Seulement j'ai un plantage "Object reference not set to an instance of an object." ?!?

le paramètre value est bien rempli et normalement je n'ai pas besoin d'initiliser un event ?

What's matter ?

[mlebreton]

Dans le cas présent, tant qu'aucun évènement n'est attaché à OnTotalChange, le Handler est null. En effet, un EventHandler n'est rien d'autre qu'un délégué. Certe, une instance sera crée lors d'une première affectation d'un évènement, mais en attendant, dans le code il faut bel et bien s'assurer de l'existance d'une telle instance.

Généralement on utiliserait plutôt un code qui ressemblerait au suivant :

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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
publicclassTotalChangeEventArgs : EventArgs
{
  publicdouble NewValue = 0;
}
 
publicdelegatevoidTotalChangeEventHandler(object Sender, TotalChangeEventArgs e);
 
 
...
 
 
publicevent TotalChangeEventHandler OnTotalChanged;
 
protectedvoid DoTotalChange()
{
  if(OnTotalChanged!=null)
  {
       TotalChangeEventHandler e = new TotalChangeEventsArgs();
       e.NewValue = Total;
       OnTotalChanged(this,e); 
  }
}
 
privatedouble _total;
publicdouble Total
{
  get
  {
       object o = _total;
       return o == null?0:_total;
  }
  set
  {
       _total += value; 
       DoTotalChange();
  }
}

Au passage, il faut éviter d'utiliser le sender pour tansmetre des valeurs. Il vaut mieux respecter les conventions. Le sender permet de transmetre la référence de l'objet éméteur de l'évènement, l'EventArgs permet lui de transmettre tou le reste.

[BCmDev]

Quand tu notes

Code :

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

1
2
 
public EventHandler OnTotalChanged;

voulais-tu noter

Code :

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

1
2
 
public event EventHandler TotalChanged;

Si oui, tu dois vérifier qu'une inscription a été faite sur ton événement en testant simplement

Code :

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

1
2
3
4
5
6
 
if (this.TotalChanged!= null)
{
   // une inscription existe. Il faut déclencher l'événement.
   this.TotalChanged(...)
}

Attention à tes nommages (réserve la notation OnTotalChanged pour la méthode surchargeable déclenchant l'événement). De plus, le premier argument de l'événement devrait être l'élément qui a déclenché l'événement (couramment nommé sender). Si tu veux envoyer ton total, définis alors un nouveau jeu d'argument TotalChangedEventArgs.

[User.Anonymous]

Merci pour ces réponses rapides

J'ai utilisé la value à la place du sender car je ne voulais pas créer toute les fonctions délégués et utiliser ce que le frameworks me proposait ([Mode Fainéant : ON] )

Visiblement, il faut, même pour un évènement avec juste un paramètre, recréer toute l'architecture de l'événement.

C'est testé et maintenant ça marches au poil

[User.Anonymous]

Dans le cas présent, tant qu'aucun évènement n'est attaché à OnTotalChange, le Handler est null. En effet, un EventHandler n'est rien d'autre qu'un délégué. Certe, une instance sera crée lors d'une première affectation d'un évènement, mais en attendant, dans le code il faut bel et bien s'assurer de l'existance d'une telle instance.

Généralement on utiliserait plutôt un code qui ressemblerait au suivant :

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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
publicclassTotalChangeEventArgs : EventArgs
{
   publicdouble NewValue = 0;
}
 
publicdelegatevoidTotalChangeEventHandler(object Sender, TotalChangeEventArgs e);
 
 
...
 
 
publicevent TotalChangeEventHandler OnTotalChanged;
 
protectedvoid DoTotalChange()
{
   if(OnTotalChanged!=null)
   {
        TotalChangeEventHandler e = new TotalChangeEventHandler();
        e.NewValue = Total;
        OnTotalChanged(this,e); 
   }
}
 
privatedouble _total;
publicdouble Total
{
   get
   {
        object o = _total;
        return o == null?0:_total;
   }
   set
   {
        _total += value; 
        DoTotalChange();
   }
}

Au passage, il faut éviter d'utiliser le sender pour tansmetre des valeurs. Il vaut mieux respecter les conventions. Le sender permet de transmetre la référence de l'objet éméteur de l'évènement, l'EventArgs permet lui de transmettre tou le reste.

En relisant c'est TotalChangeEventsArgs e = new TotalChangeEventsArgs(); et non TotalChangeEventHandler sinon ça ira pas loin:p

[BCmDev]

Evidemment, il y a souvent une charge importante de travail à respecter des "standards", des "bonnes pratiques". Mais il faut à mon avis toujours avoir la maintenance à l'esprit. Qui se souviendra dans ce cas particulier que le sender est ton total ? Dans un mois, pas même toi... Bon courage !

[mlebreton]

En relisant c'est TotalChangeEventsArgs e = new TotalChangeEventsArgs(); et non TotalChangeEventHandler sinon ça ira pas loin:p

merci User.Anonymous, j'ai corrigé cette faute d'inatention dans le post correspondant.