Optimiser un do-while pas top top.

**H0rus** · 24/07/2017, 16h47

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,

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.

Si quelqu'un pourrais me proposer une version optimisée de mon petit bricolage avec explications sa serais sympa

Voila mon code:

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
{
        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();
        }
    }

**PixelJuice** · 24/07/2017, 18h31

Bonjour,

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

Code :

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

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 :

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

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 :

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
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();
        }

**ericlm128** · 24/07/2017, 20h43

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

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
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!");
            }
        }

**François DORIN** · 24/07/2017, 21h41

Bonjour,

Autre proposition, avec un 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
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();
        }

**logeanto** · 25/07/2017, 16h30

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

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

**ericlm128** · 25/07/2017, 17h11

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 :

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

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

**H0rus** · 25/07/2017, 17h23

Aaaaah merci à tous pour les exemples et les explications! Longue vie à la commu des développeurs, the best ever!!!

**logeanto** · 27/07/2017, 14h22

Envoyé par ericlm128

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

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

**ericlm128** · 27/07/2017, 16h24

[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

Optimiser un do-while pas top top. [Débutant]

C#

Discussions similaires

Partager

Partager