switch et Partie entière

Version imprimable

Bonjour, j'ai un problème avec un switch qui doit me retourner une valeur entière selon un paramètre cylindree, mais malheureusement j'ai beaucoup de problemes avec mon switch :

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
                switch (cylindree)
                {
                    case cylindree <= 125:
                        pf = 1;
                        break;
 
                    case 125 < cylindree <= 175:
                        pf = 2;
                        break;
 
                    case 175 < cylindree <= 250:
                        pf = 3;
                        break;
 
                    case 250 < cylindree <= 350:
                        pf = 4;
                        break;
 
                    case 350 < cylindree <= 500:
                        pf = 5;
                        break;
 
                    case cylindree >  500:
                        if(cylindree%500 == 0) 
                            pf = 5 + (Math.Truncate(cylindree - 500)/125);
                        else pf = 5 + (Math.Truncate(cylindree - 500) / 125) + 1; 
                        break;
                }

Math.Trauncate comment ca fonctionne ?

normalement pour le calcul de la partie entiere, si la valeur apres le virgule est < 0.5 on arandit vers lla valeur avant le virgule sinon on arandit vers la valeur avant la virgule + 1 ???

Euh... ta syntaxe pour le bloc switch est complètement fausse : tu ne peux pas mettre des conditions comme ça dans les case, ça doit être des valeurs, genre ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 switch(cylindree) { case 50: ... break; case 125: ... break; case 500: ... break; default: ... break; }
Mais évidemment dans ton cas ce n'est pas adapté, tu ne vas écrire un case pour chaque valeur possible... Dans un cas comme ça, utilise une série de if/else :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 if (cylindree <= 125) { pf = 1; } else if (cylindree <= 175) { pf = 2; } else if (cylindree <= 250) { pf = 3; } ... else { ... }
En ce qui concerne le Truncate : dans ton code tu l'appliques à cylindree - 500, et non à (cylindree - 500)/125. Les parenthèses sont mal placées.
Truncate tronque un nombre décimal, c'est à dire que ça supprime la partie décimale, sans arrondir. Pour l'arrondi, utilise Math.Round

15/04/2009, 23h42
deejay2221
Comme Tomlev le dit, tu ne peux pas mettre des conditions comme celles-là et non pas seulement dans les switch.

Par exemple, ce code-là ne compile pas :
Code:

1 2 3 4 5 var a = 2; if (1 < a < 3) { // ... }
Le langage interprète cette ligne comme la suivant :
Code:

1 2 ((bool)(1 < a)) < 3 // alors on ne peut vérifier si (true < 3)
16/04/2009, 11h06
Msysteme

mais non, mon but c'est de tronquer : (cylindree-500)/125
16/04/2009, 11h20
Philippe Vialatte

Citation:

Envoyé par Msysteme

mais non, mon but c'est de tronquer : (cylindree-500)/125

Donc, en gros, tu veux faire :

Code:

Math.Truncate((cylindree - 500)/125)

Donc, ce que te disais tom ;)

Citation:

Envoyé par tomlev

En ce qui concerne le Truncate : dans ton code tu l'appliques à cylindree - 500, et non à (cylindree - 500)/125. Les parenthèses sont mal placées.
16/04/2009, 11h25
sisqo60
salut,

pour tronquer :
Code:

1 2 (Math.Truncate((cylindree - 500)/125));
tu tronquais cylindree-500 et ensuite tu divisais par 125. La division par 125 n'est pas souvent entière...

Code:

cylindree%500 == 0

n'est pas adapté dans ton cas à mon avis... Je m'explique :
ça veut dire que la cylindrée est 500k ou k est un réel. si k = 0 tu vas la condition est bonne.
Dans ton cas Math.Round(cylindrée/500); te donne le bon pf...
16/04/2009, 12h16
tomlev

Citation:

Envoyé par Msysteme

normalement pour le calcul de la partie entiere, si la valeur apres le virgule est < 0.5 on arandit vers lla valeur avant le virgule sinon on arandit vers la valeur avant la virgule + 1 ???

Citation:

Envoyé par Msysteme

mais non, mon but c'est de tronquer : (cylindree-500)/125

Ben faudrait savoir... si tu veux tronquer, c'est Truncate, si tu veux arrondir c'est Round
16/04/2009, 12h54
Msysteme

le round il arandit au plus si la partie decimal > 0.5 sinon a la partie entiere de la valeure a arandir ???
16/04/2009, 13h47
tomlev

Citation:

Envoyé par Msysteme

le round il arandit au plus si la partie decimal > 0.5 sinon a la partie entiere de la valeure a arandir ???

oui, il arrondit à l'entier le plus proche

il existe :

Code:

1
2
3
Math.Round(cylindree, MidpointRounding.AwayFromZero);

Code:

1
2
3
Math.Round(cylindree, MidpointRounding.ToEven);

voila ce que j'ai fait finalement :

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
            if (gr.CodeCategorie == 1)
            {
 
                if (cylindree <= 125)
                    pf = 1;
                else if (125 < cylindree && cylindree <= 175)
                    pf = 2;
                else if (175 < cylindree && cylindree <= 250)
                    pf = 3;
                else if (250 < cylindree && cylindree <= 350)
                    pf = 4;
                else if (350 < cylindree && cylindree <= 500)
                    pf = 5;
                else
                {
                    if ((cylindree - 500) % 125 == 0)
                        pf = 5 + (int)((cylindree - 500) / 125);
                    else pf = 5 + (int)((cylindree - 500) / 125) + 1;
                }
            }
            else
            {
                if (codeCarburant.Equals("DI"))
                    pf = (int)(Math.Truncate(cylindree * 0.00401 * paramM));
                else
                    pf = (int)(Math.Truncate(cylindree * 0.00573 * paramM));
            }
 
            return pf;

21/04/2009, 15h10
Msysteme

Je crois que le math.truncate donne des résultats erronés, donc je veux essayé le round, mais j'ain pas compris le deuxieme parametre :

Code:

Math.Round(cylindree, MidpointRounding.AwayFromZero);

ou

Code:

Math.Round(cylindree, MidpointRounding.ToEven);

ce que je veux moi c'est : a,bb

si bb < 5 il doit me retourné a sinon il doit me retourné a + 1

voilà ce que j'ai fait pour le truncate :

Code:

pf = (int)(Math.Round(cylindree * 0.00401 * paramM / 30));
21/04/2009, 18h09
tomlev

Math.Truncate ne donne pas de "résultats erronés", il fait juste ce qu'il est supposé faire : tronquer la partie décimale

Round fait exactement ce que tu cherches. Tu n'as pas besoin du 2e paramètre, utilise la surcharge avec un seul paramètre (comme tu as fait en fait)
22/04/2009, 11h44
Msysteme

ok; mais a quoi sert les deuxieme paramètre ? juste pour information ?
22/04/2009, 12h00
sisqo60

:f1: + :google2: = :lahola:

http://msdn.microsoft.com/fr-fr/libr...trounding.aspx
22/04/2009, 12h05
tomlev

Citation:

Envoyé par Msysteme

ok; mais a quoi sert les deuxieme paramètre ? juste pour information ?

C'est juste pour indiquer comment tu gères le cas où le premier paramètre est exactement à mi-chemin entre deux entiers. Par exemple 2.5 est à mi-chemin entre 2 et 3, ce paramètre permet d'indiquer si ça sera arrondi en 2 ou en 3. Pour tous les autres nombres ça n'a pas d'influence, par exemple 2.1 sera toujours arrondi en 2, et 2.8 sera toujours arrondi en 3.
22/04/2009, 16h48
Msysteme

merci beacoup, c'est un exemple trés clair :D