Discussion :

[CompuTux]

Bonjour à tous !

Dans le cadre de mon apprentissage de C#, je développe (en procédural et en mode console) un petit programme d'arithmétique qui, pour le moment, permet de décomposer un nombre entier naturel (un long) en produit de ses facteurs premiers.

Il s'agit d'une boucle principale dans la méthode Main du programme qui demande à l'utilisateur de saisir un mot clé, qui appelle la fonction correspondante.

Les mot clés sont rassemblés dans une enum Commandes :

Code :

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

enum Commandes { Aide, Décomposition, Quitter };

J'ai une fonction SaisieCommande() qui gère les commandes saisies par l'utilisateur :

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
public static Commandes SaisieCommande()
        {
            Console.Write("Entrez une commande (Aide, Quitter, etc.) : ");
 
            string saisie = Console.ReadLine();
 
            Commandes commandeSaisie = (Commandes)Enum.Parse(typeof(Commandes), saisie, true);
 
            foreach (Commandes commande in Enum.GetValues(typeof(Commandes)))
            {
                if (!commande.Equals(commandeSaisie))
                {
 
                    throw new ArgumentException("Vous avez entré une mauvaise commande !");
 
                }
            }
 
            return commandeSaisie;
        }

J'aimerais que cette fonction puisse gérer les erreurs de saisie utilisateur en renvoyant l'exception ArgumentException("Vous avez entré une mauvaise commande !").

Plusieurs questions se posent :

1) Est ce que l'exception choisie est la meilleure pour ce cas de figure ?
2) Comment faire pour comparer la saisie utilisateur à toutes les commandes possibles afin de savoir si l'exception doit être renvoyée ou pas ? Car actuellement mon programme renvoie l'exception même quand je saisie une bonne commande. J'en suis conscient !

Voilà, je vous remercie d'avance pour toute aide !

[tomlev]

N'utilise pas des exceptions pour ça, ce n'est pas fait pour ça ! Les exceptions, comme leur nom l'indique, sont faites pour gérer des cas exceptionnels/non prévisibles. Toi tu t'en sers pour faire du contrôle de flux, c'est une très mauvaise pratique... Le fait que l'utilisateur ait fait une erreur de saisie n'est pas une situation exceptionnelle, c'est quelque chose que tu peux vérifier et gérer proprement.

[CompuTux]

Merci pour ta réponse, mais mince alors ! Moi qui pensait avoir à peu près compris que les Exceptions pouvaient aussi être utilisées pour faire du contrôle de saisie....

En effet dans un MSDN Training Workbook de Microsoft, j'ai vu que l'auteur utilisait un ArgumentOutOfRangeException pour contrôler la saisie d'un numéro de jour dans une année :

Code :

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

1
2
3
4
5
6
7
8
9
Console.Write("Please enter a day number (between 1 and {0}) : ", maxDayNum);
 
                line = Console.ReadLine();
                dayNum = int.Parse(line);
 
                if (dayNum < 1 || dayNum > maxDayNum)
                {
                    throw new ArgumentOutOfRangeException("Day out of Range !");
                }

Voilà pourquoi je pensais utiliser l'ArgumentException pour gérer une erreur de saisie dans mon programme...

Visiblement j'ai toujours pas tout compris... Peux tu m'expliquer comment contrôler proprement la saisie utilisateur dans mon programme ?

[CompuTux]

Bon j'apprends, et donc j'ai suivi ton conseil, je t'en remercie, tu as raison car c'est très lourd les exceptions qui interrompent le programme !

Voici donc le code de ma solution :

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
#region Fonction SaisieCommande
        public static Commandes SaisieCommande()
        {
            // Cette fonction gère la saisie d'une commande par l'utilisateur
 
            Commandes commandeSaisie;
 
            Console.Write("Entrez une commande (Aide, Quitter, etc.) : ");
 
            string saisie = Console.ReadLine();
 
            bool saisieOK = Enum.TryParse(saisie, true, out commandeSaisie);
 
            if (saisieOK)
            {
 
                return commandeSaisie;
            }
 
            else
            {
                return Commandes.Erreur;
            }
 
 
        }
        #endregion

Il fonctionne bien et il n'est pas trop cochon je trouve...

Qu'en pensez vous svp ??

[tomlev]

Merci pour ta réponse, mais mince alors ! Moi qui pensait avoir à peu près compris que les Exceptions pouvaient aussi être utilisées pour faire du contrôle de saisie....

Ca peut être utilisé pour ça, mais en interceptant les exceptions qui se produisent quand tu essaies d'interpréter la saisie, pas en en lançant de nouvelles. Par exemple, si tu fais un DateTime.Parse et que ça lève une exception, tu l'interceptes pour détecter l'erreur de saisie (bon, c'est un peu bidon comme exemple, il vaut mieux utiliser TryParse de toutes façons...)

En pratique, on lève rarement des exceptions dans la partie "interface utilisateur" d'une application. En revanche, dans une application multi-couches il est fréquent de lever une exception dans une couche basse pour faire remonter un problème aux couches supérieures (et dans ce cas les couches supérieures doivent intercepter et gérer cette exception).

En effet dans un MSDN Training Workbook de Microsoft, j'ai vu que l'auteur utilisait un ArgumentOutOfRangeException pour contrôler la saisie d'un numéro de jour dans une année :

Code :

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

1
2
3
4
5
6
7
8
9
Console.Write("Please enter a day number (between 1 and {0}) : ", maxDayNum);
 
                line = Console.ReadLine();
                dayNum = int.Parse(line);
 
                if (dayNum < 1 || dayNum > maxDayNum)
                {
                    throw new ArgumentOutOfRangeException("Day out of Range !");
                }

C'est un très bon exemple... de ce qu'il ne faut pas faire
C'est d'ailleurs assez malheureux de voir un truc comme ça dans des tutos de MS

Voici donc le code de ma solution :

...

Qu'en pensez vous svp ??

C'est pas mal
Il n'y a pas vraiment une façon universelle de faire ça, ça dépend un peu du type d'application (console, windows, web...) et de son mode de fonctionnement. Dans ton cas, cette solution me semble assez adaptée