Discussion :

[Papy214]

Bonsoir,

Sur l'utilisation d'un EventHandler, StyleCop me dit que je devrais utiliser les generics pour plus d'efficacité mais je ne trouve pas la syntaxe pour le faire.

Voilà mon code original:

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
 
public partial class Form2 : Form
{
    public event TopInstantEventHandler OnTopInstant;
 
    public Form2()
    {
        InitializeComponent();
    }
 
    private void button1_Click(object sender, EventArgs e)
    {
        if (OnTopInstant != null)
        {
            OnTopInstant(this, new GenererTopInstantEventArgs(DateTime.Now));
        }
    }
}
 
public class GenererTopInstantEventArgs : EventArgs
{
    public DateTime Instant { get; set; }
 
    public GenererTopInstantEventArgs(DateTime moment)
    {
        Instant = moment;
    }
}
 
public delegate void TopInstantEventHandler(object sender, GenererTopInstantEventArgs e);

Voilà le message de StyleCop:

Target : WindowsApplication1.TopInstantEventHandler (IntrospectionTargetType)
Resolution : "Remove 'TopInstantEventHandler' and replace its
usage with EventHandler<T>"
Help : http://msdn2.microsoft.com/library/ms182178(VS.90).aspx (String)
Category : Microsoft.Design (String)
CheckId : CA1003 (String)
RuleFile : Design Rules (String)
Info : "Do not declare new delegates to be used as event handlers
when targeting a version of the .NET Framework that
supports generics. Use an instance EventHandler<T>
instead."

Problème: si j'écris

Code :

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

public delegate void EventHandler<T>(object sender, GenererTopInstantEventArgs e);

Le compilo ne trouve plus le type de handler TopInstantEventHandler.

Quelqu'un aurti un petit exemple à me donner ? Ou une correction de mon code ?

Merci d'avance

Papy !

[tomlev]

Ce que StyleCop veut dire, c'est que tu n'as pas besoin de déclarer un delegate spécifique pour ton évènement, car sa signature correspond à la signature "standard" pour les évènements en .NET et il existe déjà un delegate générique pour ça. Il suffit que tu supprimes la déclaration du delegate TopInstantEventHandler et que tu remplaces partout TopInstantEventHandler par EventHandler<GenererTopInstantEventArgs>

Par ailleurs, un petit conseil : essaie de respecter les conventions de nommage pour les évènements :
- pas de préfixe "On" pour l'évènement lui-même
- nom de l'évènement préfixé par "On" pour la méthode protégée qui déclenche l'évènement (si elle existe)
- nom de l'évènement suivi de EventArgs pour les paramètres de l'évènement
- nom de l'évènement suivi de EventHandler pour le delegate correspondant à ton évènement (mais apparemment StyleCop recommande le delegate générique, donc c'est inutile)

Au final, je verrais mieux ton code comme ça :

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
 
 
public partial class Form2 : Form
{
    public event TopInstantEventHandler TopInstant;
 
    public Form2()
    {
        InitializeComponent();
    }
 
    private void button1_Click(object sender, EventArgs e)
    {
        OnTopInstant(new TopInstantEventArgs(DateTime.Now);
    }
 
    // Les classes dérivées peuvent overrider cette méthode plutôt que de s'abonner à l'évènement
    protected virtual void OnTopInstant(TopInstantEventArgs e)
    {
        if (TopInstant != null)
        {
            TopInstant(this, e);
        }
    }
}
 
public class TopInstantEventArgs : EventArgs
{
    public DateTime Instant { get; set; }
 
    public TopInstantEventArgs(DateTime moment)
    {
        Instant = moment;
    }
}
 
// à supprimer si tu veux suivre les recommandations de StyleCop
public delegate void TopInstantEventHandler(object sender, TopInstantEventArgs e);

Si tu suis les conventions, ton code sera plus facile à utiliser et à maintenir par d'autres développeurs, car ce sont les mêmes conventions utilisées dans tout le .NET Framework

[Papy214]

Ok merci !

Et donc, avec les generics, ça donnerait ça:

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
 
public partial class Form1 : Form
{
 
    public event EventHandler<TopInstantEventArgs> TopInstant;
 
    public Form1()
    {
        InitializeComponent();
    }
 
    // Les classes dérivées peuvent overrider cette méthode plutôt que de s'abonner à l'évènement
    protected virtual void OnTopInstant(TopInstantEventArgs e)
    {
        if (TopInstant != null)
        {
            TopInstant(this, e);
        }
    }
 
    private void OnTopInstant(DateTime instant)
    {
        if (TopInstant != null)
        {
            TopInstant(e, new TopInstantEventArgs(instant));
        }
    }
 
}
 
public class TopInstantEventArgs : EventArgs
{
    public DateTime Instant { get; set; }
 
    public TopInstantEventArgs(DateTime moment)
    {
        Instant = moment;
    }
}

Right ??

[tomlev]

C'est ça !

[Papy214]

J'ai eu du mal à comprendre l'utilisation des génerics quand je me suis lancé dans C#. Au début, je pensais que ça ne pouvait servir qu'à faire des listes d'objets différents réunis par des méthodes communes, un peu comme les interfaces. En fait, c'est bien plus utile que ça.

Merci encore ...