Comparaison dans un switch

Version imprimable

salut tout le monde
je veux executer des conditions mais sans passer par le long chemin des if et else if. Alors jai decide d'utiliser le switch mais sa ne prend pas.
Sa me dit que j'essais deffectuer une conversion entre un double et un typ bool du coup affiche une erreur et je sais pas comment faire
merci bien!

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
  double betrag = 15000;
 
            switch (betrag)
            {
                case  < 500:
 
                    Console.WriteLine("Es tut mir leid aber für diesen Einkauf bekommen Sie kein Rabatt!");
                    break;
 
                case ( betrag >= 2000 && < 5000):
 
                    Console.WriteLine("für diesen Einkauf erhalten Sie 5% Rabatt");
                    break;
 
                case (betrag >=500 && < 2000):
                    Console.WriteLine("Für diesen Einkauf erhalten Sie 2% Rabatt");
                    break;
 
                case  >= 5000:
                    Console.WriteLine("Für diesen Einkauf erhalten Sie 10% Rabatt");
                    break;
            }

22/10/2018, 09h58
PixelJuice

Bonjour,

En effet ton switch ne veut que des valeurs de type double. Le problème c'est que :

Code:

betrag >= 500

est une condition, qui est considérée comme un bool.

Du coup même si tu pouvais faire ça, tu serais limité au fait qu'il n'y aurait que deux case possible : true ou false. Quelle que soit la condition.

Il te faut passer par un if / else.
22/10/2018, 10h06
junior1998

Donc le c# ne le permet pas alors !!
C’est bizarre
22/10/2018, 10h40
PixelJuice

Citation:

Envoyé par junior1998

Donc le c# ne le permet pas alors !!
C’est bizarre

Non il ne le permet pas je pense, et les langages ou il est possible de le faire, je lis qu'il est fortement conseillé de passer par un if / else quand même, pour la lisibilité, etc ...

Pourquoi bizarre ? Un switch case est fait pour des cas d'utilisation très simples. Si tu as besoin de complexité, alors il te faudra passer par autre chose.

Personnellement, j'utilise très peu, voir pas du tout les switch case, hormis si il me faut évaluer des enums.

Bonjour,

On peut arriver à quelque chose d'approchant, en utilisant le pattern matching, qui permet de préciser une condition :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
double betrag = 15000;
 
switch (betrag)
{
    case double d when d < 500:
 
        Console.WriteLine("Es tut mir leid aber für diesen Einkauf bekommen Sie kein Rabatt!");
        break;
 
    case double d when (d >= 2000 && d < 5000):
 
        Console.WriteLine("für diesen Einkauf erhalten Sie 5% Rabatt");
        break;
 
    case double d when (d >= 500 && d < 2000):
        Console.WriteLine("Für diesen Einkauf erhalten Sie 2% Rabatt");
        break;
 
    case double d when d >= 5000:
        Console.WriteLine("Für diesen Einkauf erhalten Sie 10% Rabatt");
        break;
}

Attention, c'est une nouveauté disponible uniquement depuis C# 7.

Ou sinon, une manière ressemblante réalisable à partir du framework 3.5

Code:

1
2
3
4
5
6
7
8
9
10
11
Double number = 2018;
 
var choices = new Dictionary<Func<Double, Boolean>, Action>
{
    { x => x < 500 , () => Console.WriteLine("0-499") },
    { x => x < 2000 , () => Console.WriteLine("500-1999") },
    { x => x < 5000 , () => Console.WriteLine("2000-4999") },
    { x => x < Double.MaxValue , () => Console.WriteLine("5000+") }
};
 
choices.First(it => it.Key(number)).Value();

Citation:

Envoyé par PixelJuice

Pourquoi bizarre ? Un switch case est fait pour des cas d'utilisation très simples. Si tu as besoin de complexité, alors il te faudra passer par autre chose.

j'aurais tendance à dire non
le switch de c# est fait pour des cas d'utilisation très simple, mais dans d'autres langages pas forcément
en vb.net on peut faire
Code:

1 2 3 4 5 6 select case true ' conditions volontairements hétéroclites pour montrer l'étendue (non exhaustive) case d > 50 andalso d < 200 case typeof var3 is IMachin case coll.firstordefault(condition …) isnot Nothing case MaFctQuiReturnUnBool(var4) end select
et parfois c'est bien pratique de pouvoir faire des choses complexes avec peu de code qui plus est lisible (en plus vb.net n'a pas besoin de break à chaque case)
le switch par typage de c# amène un peu de souplesse et c'est déjà bien mais pas encore assez je pense ...

Citation:

Envoyé par popo

Ou sinon, une manière ressemblante réalisable à partir du framework 3.5

Pas con et très astucieux !

Par contre, il faut bien avoir des bornes qui ne se recoupent pas. S'agissant d'un dictionnaire, l'ordre dans lequel les clés sont énumérées est indéfini.

Donc ici, je changerai le code en :
Code:

1 2 3 4 5 6 7 8 9 10 11 Double number = 2018; var choices = new Dictionary<Func<Double, Boolean>, Action> { { x => x < 500 , () => Console.WriteLine("0-499") }, { x => 500 <= x && x < 2000 , () => Console.WriteLine("500-1999") }, { x => 2000 <= x && x < 5000 , () => Console.WriteLine("2000-4999") }, { x => 500 <= x && x < Double.MaxValue , () => Console.WriteLine("5000+") } }; choices.First(it => it.Key(number)).Value();

23/10/2018, 11h23
Max

Citation:

Envoyé par François DORIN

Pas con et très astucieux !

Très astucieux mais peut-être un peu too much dans ce contexte :D (la discussion a le tag [Débutant])

Citation:

Envoyé par François DORIN

Pas con et très astucieux !

Effectivement, c'est une vielle ruse que je n'avais pas utilisé depuis longtemps.
Je l'ai tapé de mémoire et tu as tout à fait raison sur la particularité du dictionnaire.
Petite rectification sur la dernière condition toutefois, il manque un zéro.

Code:

{ x => 5000 <= x && x < Double.MaxValue , () => Console.WriteLine("5000+") }

Citation:

Envoyé par Max

Très astucieux mais peut-être un peu too much dans ce contexte (la discussion a le tag [Débutant])

Effectivement, c'est un poil avancé pour un débutant, ça nécessite donc une explication.

Tout d'abord l'instruction est basé sur un dictionnaire (un couple clé-valeur dont la clé est unique mais pas forcément triée).
Ici, la clé et la valeur sont des délégués (en quelque sorte, des pointeurs vers des fonction).
Dans ce cas, la clé sera donc un pointeur vers une fonction prenant en paramètre un Double et renvoyant un Booléen.
Et la valeur sera un pointeur vers une fonction ne prenant pas de paramètre et ne retournant aucune valeur.

Le dictionnaire contient donc quatre pairs clé-valeur

Chaque clé est composé d'une expression lambda définissant le corps du premier délégué.
Comme le premier paramètre est un double, je l'ai appelé x, et comme le résultat est un booléen, seul le test logique (la comparaison du paramètre en entrée) est nécessaire.

Chaque valeur est composé d'une expression lambda définissant le corps du second délégué.
Puisque qu'il n'y a pas de paramètre en entrée, on place "()", puis le corps de l'action à réaliser (ici Console.WriteLine).

J'aurais pu déclarer les fonction une à une et écrire ceci, mais ça aurait été plus long et on aurait pas eu toutes les informations visibles d'un coup.
Code:

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 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 class Program { static void Main(string[] args) { Double number = 2018; var choices = new Dictionary<Func<Double, Boolean>, Action> { { TestInf500, WriteInf500 }, { TestBetwen500And1999, WriteBetween500And1999 }, { TestBetwen2000And4999 , WriteBetween2000And4999}, { TestSup5000 , WriteSup5000 } }; choices.First(it => it.Key(number)).Value(); Console.ReadKey(); } public static Boolean TestInf500(Double x) { return x < 500; } public static Boolean TestBetwen500And1999(Double x) { return 500 <= x && x < 2000; } public static Boolean TestBetwen2000And4999(Double x) { return 2000 <= x && x < 5000; } public static Boolean TestSup5000(Double x) { return 5000 <= x && x < Double.MaxValue; } public static void WriteInf500() { Console.WriteLine("0-499"); } public static void WriteBetween500And1999() { Console.WriteLine("500-1999"); } public static void WriteBetween2000And4999() { Console.WriteLine("2000-4999"); } public static void WriteSup5000() { Console.WriteLine("5000+"); } }
La ligne "choices.First(it => it.Key(number)).Value()" est composé comme ceci :
choice.First est une méthode Linq recherchant la première clé-valeur répondant aux critères passés via l'expression lambda.
it => it.Key(number) est mon expression lambda, elle signifie que parmi les clé du dictionnaire, je dois rechercher le délégué dont le retour matchera.
Pour la valeur 2018, le premier (et le seul) délégué qui répond true est le troisième.
.Value() est appliqué sur la pair clé-valeur trouvé par First, et me permet de récupérer la valeur de cette pair et donc le délégué qui affiche l'information dans la console.

J'espère avoir été suffisamment clair.