Discussion :

[Msysteme]

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 :

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
                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 ???

[tomlev]

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 :

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
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 :

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
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

[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 :

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

1
2
3
4
5
var a = 2;
if (1 < a < 3)
{
    // ...
}

Le langage interprète cette ligne comme la suivant :

Code :

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

1
2
((bool)(1 < a)) < 3
// alors on ne peut vérifier si (true < 3)

[Msysteme]

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

[sisqo60]

salut,

pour tronquer :

Code :

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

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 :

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

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...

[tomlev]

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 ???

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

[Philippe Vialatte]

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

Donc, en gros, tu veux faire :

Code :

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

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

Donc, ce que te disais tom

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.