Optimiser un do-while pas top top.

Version imprimable

Hello People!
Alors voila j'ai un petit soucis, j'ai pour "mission" d'écrire un programme qui, dans un menu, propose les choix suivants :
– convertir un montant HT en montant TTC ;
– convertir un montant TTC en montant HT ;
– terminer l’application.
Si l’utilisateur demande une conversion, je doit lui demander de saisir un montant. Selon le cas, ce montant
sera multiplié ou divisé par 1,196, puis le résultat affiché.
Le menu est à nouveau proposé à l’utilisateur, jusqu’à ce qu’il choisisse de terminer l’application....

Bon en théorie je pense avoir "réussi" grâce à un do-while mais je trouve pas ça très propre,:aie:
en plus il est nécessaire de saisir deux fois la sortie du programme pour qu'elle s’exécute et j'aimerais savoir pourquoi.:weird:
Si quelqu'un pourrais me proposer une version optimisée de mon petit bricolage avec explications sa serais sympa :zoubi:
Voila mon code:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 { static void Main(string[] args) { int choix; float montant; do { Console.WriteLine("\n1 - Convertir un montant HT en montant TTC\n2 - Convertir un montant TTC en montant HT\n3 - Terminer l'application\nSaisissez 1, 2 ou 3:"); choix = Convert.ToInt16(Console.ReadLine()); if (choix != 3) Console.WriteLine("\nSaisissez votre montant à convertir:"); montant = Convert.ToSingle(Console.ReadLine()); if (choix == 1) Console.WriteLine("\nVotre montant TTC est de {0} Euros ", montant * 1.196); else if (choix == 2) Console.WriteLine("\nVotre montant HT est de {0} Euros ", montant / 1.196); } while (choix != 3); Console.WriteLine("A très bientôt!"); Console.ReadLine(); } }

Bonjour,

alors du coup pour ton problème de sortie de programme, il vient de la :

Code:

if (choix != 3)

Il n'y a pas de crochets, donc par défaut, seule la prochaine ligne sera soumise a la condition, et donc

Code:

montant = Convert.ToSingle(Console.ReadLine());

sera exécutée, quel que soit la valeur de choix.

Voici ton programme optimisé (même si bon, on peut toujours optimisé du code mais c'est pas te prendre la tête dès le début avec des trucs complexes, des conventions, etc ..).

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
 static void Main(string[] args)
        {
            int choix;
            float montant = 0;
            const float TTC = 1.196F; // Utilise ici une variable constante, qui pourra être utilisé partout et si tu dois la changer, cela t'économisera du temps.
 
            do
            {
                Console.WriteLine("\n1 - Convertir un montant HT en montant TTC\n2 - Convertir un montant TTC en montant HT\n3 - Terminer l'application\nSaisissez 1, 2 ou 3:");
                choix = Convert.ToInt16(Console.ReadLine());
 
                if (choix != 3) // Ce IF englobe tout le reste, comme ça pas besoin de vérifier le choix si on sait que ça sera 3
                {
                    Console.WriteLine("\nSaisissez votre montant à convertir:");
 
                    montant = Convert.ToSingle(Console.ReadLine());
 
                    if (choix == 1)
                    {
                        Console.WriteLine("\nVotre montant TTC est de {0} Euros ", montant * TTC);
                    }
 
                    else if (choix == 2)
                    {
                        Console.WriteLine("\nVotre montant HT est de {0} Euros ", montant / TTC);
                    }
                }
            }
            while (choix != 3);
 
            Console.WriteLine("A très bientôt!");
            Console.ReadLine();
        }

Une autre proposition, sachant que tu en aura autant que de développeur.
J'ai ajouté la vérification d'erreur de saisi sur le menu et sur le nombre.

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
        static void Main(string[] args)
        {
            const float TTC = 1.196F;
 
            while (true)
            {
                Console.WriteLine("\n1 - Convertir un montant HT en montant TTC\n2 - Convertir un montant TTC en montant HT\n3 - Terminer l'application\nSaisissez 1, 2 ou 3:");
                string choix = Console.ReadLine();
 
                if (choix == "1" || choix == "2")
                {
                    Console.WriteLine("\nSaisissez votre montant à convertir:");
 
                    float montant;
                    // Vérifie si le montant est un nombre valide et le converti dans "montant"
                    if (float.TryParse(Console.ReadLine(), out montant))
                    {
                        if (choix == "1")
                            Console.WriteLine("\nVotre montant TTC est de {0} Euros ", montant * TTC);
                        else if (choix == "2")
                            Console.WriteLine("\nVotre montant HT est de {0} Euros ", montant / TTC);
                    }
                    else
                        Console.WriteLine("\nSaisis non valide : Nécessite un nombre !");
                }
                else if (choix == "3")
                {
                    Console.WriteLine("A très bientôt!");
                    Console.ReadLine();
                    break;
                }
                else
                    Console.WriteLine("\nSaisis non valide : Ne correspond pas a un choix du menu!");
            }
        }

Bonjour,

Autre proposition, avec un 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
29
30
31
static void Main(string[] args)
        {            
            float montant;
            bool stop = false;
 
            while(!stop)
            {
                string choix;
                Console.WriteLine("\n1 - Convertir un montant HT en montant TTC\n2 - Convertir un montant TTC en montant HT\n3 - Terminer l'application\nSaisissez 1, 2 ou 3:");
 
                choix = Console.ReadLine();
                switch(choix)
                {
                    case "1":
                        Console.WriteLine("\nSaisissez votre montant à convertir:");
                        montant = Convert.ToSingle(Console.ReadLine());
                        Console.WriteLine("\nVotre montant TTC est de {0} Euros ", montant * 1.196);                        
                        break;
                    case "2":
                        Console.WriteLine("\nSaisissez votre montant à convertir:");
                        montant = Convert.ToSingle(Console.ReadLine());
                        Console.WriteLine("\nVotre montant HT est de {0} Euros ", montant / 1.196);
                        break;
                    case "3":
                        stop = true;
                        break;
                }                                
            }            
            Console.WriteLine("A très bientôt!");
            Console.ReadLine();
        }

bonjour ici j'ai la version "jme la pete" qui abuse des opérateurs conditionnels (c'est comme ça que ça se dit ?)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 static void Main(string[] args)
        {
            int choix;
            float montant;
            string result = null;
            do
            {
                Console.WriteLine("\n1 - Convertir un montant HT en montant TTC\n2 - Convertir un montant TTC en montant HT\n3 - Terminer l'application\nSaisissez 1, 2 ou 3:");
                choix = Convert.ToInt16(Console.ReadLine());
                if (choix != 3)
                {
                    Console.WriteLine("\nSaisissez votre montant à convertir:");
                    montant = Convert.ToSingle(Console.ReadLine());
                    result = choix == 1 ? "\nVotre montant TTC est de " + (montant * 1.196) + " Euros " : choix == 2 ? "\nVotre montant TTC est de " + (montant / 1.196) + " Euros " : "\nErreur, le choix doit etre 1 ou 2";
                }
                Console.WriteLine(result);
            }
            while (choix != 3);
            Console.WriteLine("A très bientôt!");
            Console.ReadLine();
        }

en espérant avoir aidé.

25/07/2017, 17h11
ericlm128
Oui Opérateur conditionnel ou Opérateur ternaire.

Afin que H0rus comprenne, il s'agit d'une "espèce" de forme contracté de if...else retournant un résultat.

Exemple simple de code
Code:

1 2 3 int a = 0; a = true ? 1 : 2; // a vaut 1 car la condition est vrai a = false ? 3 : 4; // a vaut 4 car la condition est fausse
Voir :
https://msdn.microsoft.com/fr-fr/lib...v=vs.100).aspx
https://msdn.microsoft.com/fr-fr/lib...(v=vs.71).aspx

PS : Il y a une erreur sur "TTC" et "HT", quand ont se la pète ça ne pardonne ça :lol:
25/07/2017, 17h23
H0rus

Aaaaah merci à tous pour les exemples et les explications! Longue vie à la commu des développeurs, the best ever!!! 8-) 8-) 8-)
27/07/2017, 14h22
logeanto

Citation:

Envoyé par ericlm128

PS : Il y a une erreur sur "TTC" et "HT", quand ont se la pète ça ne pardonne ça :lol:

haha tout a fait d'accord je l'ai fait très vite mais cette solution n'est pas viable professionnellement car deja rien qu'avec 2 opérateurs conditionnels l'opération est dure a comprendre (du moins il faut se pencher dessus un petit moment )
mais je serais curieux si une fois compilé cela est different que des if else

Teo
27/07/2017, 16h24
ericlm128

[Performance] Opérateur conditionnel vs Opérateur ternaire

Non, cela n'a pas l'air de donner le même code MSIL. L'opérateur conditionnel à l'air plus lent aussi.

http://cc.davelozinski.com/c-sharp/w...onal-statement
https://stackoverflow.com/questions/...-if-else-block
https://stackoverflow.com/questions/...tor-in-c-sharp