Discussion :

[pubeldy]

Hello,

Etant donné que les paramètres optionnels n'existent pas en C#, je n'ai pas d'autre choix que d'utiliser la surcharge...
Dans mon cas, quand on appelle Request(X) il doit apeller Request(X,Y) qui contient tout le code métier.
Le problème, c'est que le paramètre Y est de type Enums.HistoryLogAction et je n'arrive pas à appeler Request(X,Y) en lui passant null pour valeur d'Y...

Est-ce la bonne manière de faire et si oui, comment résoudre ce problème?

Merci d'avance...

Code :

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

1
2
3
4
5
6
7
8
9
public void Request(Member request)
{
    Request(request, null);
}
 
public void Request(Member request, Enums.HistoryLogAction historyLogAction)
{
//Ici, la logique métier
}

[Kaidan]

Il faut ajouter une valeur à l'énumération pour gérer ce cas.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public enum HistoryLogAction {
   ...
   ...
   Unknown
}
 
public void Request(Member request) {
    Request(request, Enums.HistoryLogAction.Unknown);
}
 
public void Request(Member request, Enums.HistoryLogAction historyLogAction) {
   //Ici, la logique métier
}

[pubeldy]

Merci Kaidan,

j'y avais pensé. Par contre, ça me gêne un peu de ne pas pouvoir utiliser un bon "null" là ou effectivement, je n'ai pas de valeur à donner. Devoir émuler l'absence de valeur me parraît moins solide ... moins flexible... Mais si il n'y a pas d'autre solution, je ferai comme ça!

[Kaidan]

Si tu lis la documentation C#, tu verras que :

Le mot clé null est un littéral qui représente une référence nulle, c'est-à-dire qui ne fait référence à aucun objet.
null est la valeur par défaut des variables de type référence.

Hors, une énumération ne peut contenir que des types intégraux tel que int, char, long qui ne sont pas nullable (ce sont des types par valeur et non par référence).

[SaumonAgile]

En général, toutes les énumérations (jette un oeil dans la msdn) ont une valeur NotSet très utile pour gérer ce genre de cas justement.

[StormimOn]

En général, toutes les énumérations (jette un oeil dans la msdn) ont une valeur NotSet très utile pour gérer ce genre de cas justement.

C'est pas None plutôt ? Mais au final ca revient à la même chose ^^

[Sphax]

Et si tu tiens a pouvoir passer null et que tu fais du .net 2, alors tu peux utiliser les types nullables:

Code :

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

1
2
3
4
5
6
7
8
9
 
public void Request(Member request, Enums.HistoryLogAction? historyLogAction)
{
//Ici, la logique métier
if(historyLogAction != null)
{
    Enums.HistoryLogAction value = historyLogAction .value;
}
}

(note le ? dans le code)

[oyigit]

autre notation :

Code :

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

1
2
3
4
5
6
7
8
public void Request(Member request, Enums.HistoryLogAction? historyLogAction)
{
//Ici, la logique métier
if(historyLogAction.HasValue)
{
    Enums.HistoryLogAction value = historyLogAction .value;
}
}

[pubeldy]

Merci beaucoup pour vos avis. Je vais pencher pour le type nullable, c'est tout à fait ce que je cherchais...