Discussion :

[petitours]

Bonjour

Dans une classe je crée une méthode ayant des paramètres booléens

Code :

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

public void MaMethode(bool Param1)

Afin de faciliter l'utilisation de la classe j'aimerais fournir à IntelliSense les choix possibles et clarifier la synthase finale.
C'est à dire permettre de faire

Code :

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

MaMethode(Machin.Actif);

plutôt que

Code :

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

MaMethode(Truee);

Pour ça j'aurai bien fait ceci

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public enum Machin
    {
Actif= true,
Inactif
    }
 
public void MaMethode(Machin Param1)
{
}

Mais le enum est apparemment forcément un int

Comment puis je faire ça autrement ?


merci par avance

[Sehnsucht]

Personnellement je suis plutôt partisan de l'interface publique la plus claire (et simple d'utilisation) possible du coup je préfère 2 méthodes plutôt qu'une avec un flag soit :

Code :

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

1
2
3
4
5
ActiveMachin ()
DesactiveMachin ()
 
// au lieu de
AgitSurMachin(bool actif)

Surtout que rien n'empêche ces 2 méthodes de déléguer leur boulot à cette méthode qui prend un flag mais fait partie de l'interface privée de la classe.

Après si on reste sur le modèle établi ; je partirais plutôt sur un argument de type enum ; et c'est dans le corps de cette méthode qu'il y aurait un test ou un switch pour gérer les différents cas.

[lead8209]

Salut, pas moyen de faire autrement :

Code :

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

1
2
3
4
5
public enum Machin
{
    Inactif = 0
    Actif= 1,   
}

Mais franchement faire Enum sur 2 valeurs n'a pas vraiment de sens.
Un Enum te facilite la vie quand par exemple tu remontes un code d'erreur et que tu as 50 (en fait plus de 2 valeurs possibles.

Fais plutôt une fonction comme cela :

Code :

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

1
2
 
public void MaMethode(bool pEstActif)

Enfin tout dépend de e que tu veux faire. Avec l'exemple donné voici ma réponse

[ReiKiss]

Salut, si tu veux garder la structure de code que tu as donné, tu peux essayer cela :

Code :

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

1
2
3
4
5
public static class Machin
{
     public const bool Actif = true;
     public const bool Inactif = false;
}

et tu pourra utiliser :

MaMethode(Machin.Actif);

[petitours]

Merci a tous et en particulier à ReiKiss

C'est exactement ça que je cherchais à faire
Le but étant que là où on fait appel à cette méthode on ait une syntaxe plus claire et plus intuitive sans aller regarder à quoi correspond true et false.

Merci

[petitours]

en fait non :-(

En premier ca me dit que les type static ne peuvent pas etre utilisés en paramètres, ok, je vire static...

Après, j'ai l'intelliSence qui me propose bien ce que je voulais pour l'appel de la méthode mais
après ça me dit que je ne peux pas convertir un bool en "Machin"

[ericlm128]

Moi ceci fonctionne correctement chez moi (même avec la class static d'ailleur)

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
        private void button1_Click(object sender, EventArgs e)
        {
            test(Machin.Actif);     // "C'est vrai"
            test(Machin.Inactif);   // "C'est faux"
        }
 
        private void test(bool val)
        {
            if (val)
                MessageBox.Show("C'est vrai");
            else
                MessageBox.Show("C'est faux");
        }
 
        public struct Machin
        {
            public const bool Actif = true;
            public const bool Inactif = false;
        }