Discussion :

[mrpopo]

Bonjour,

Je veux faire un jeu simple avec le fameux nombre à trouver entre 0 et 100. Le programme doit donc renvoyer en combien de coup le nombre à été trouvé.

Tout fonctionne normalement sauf que j'essaye désespérément de faire en sorte que l'utilisateur inscrit bien un nombre et non pas n’importe quoi comme des lettres, la touche entrée, etc... afin de pouvoir éviter les erreurs.

Le code n'est pas forcement code d'un professionnel du langage c# mais bon j'arrive à m'y retrouver comme cela :-)

Pour faire cela j'ai voulu utiliser l'instruction(int.TryParse (saisie , out valeurSaisie )), mais cela ne fonctionne pas du tout.

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
 
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
 
            Touche(); // Appel de la méthode Touche
 
        }
        static void Touche()
        {
            int valeurATrouver = new Random().Next(0, 100);
            int i = 1;
            bool trouve = false;
 
 
            Console.WriteLine(valeurATrouver); // Affiche le nombre à trouver 
 
            Console.WriteLine(" Veuillez saisir votre nombre (entre 0 et 100) : ");
 
                while (!trouve)
                {
 
                    string saisie = Console.ReadLine();
                    int saisie1 = Convert.ToInt32(saisie); // Converti le string Saisie en int pour faire le comparaison avec Valeur à trouver
                    int valeurSaisie;
 
                    if (int.TryParse (saisie , out valeurSaisie )) // Test mais la ca ne fait rien du tout
                    {
 
 
                            if (saisie1 == (valeurATrouver))
                            {
                                trouve = true;
                            }
                            else
                            {
                                i++;
                                if (saisie1 < valeurATrouver)
                                { Console.WriteLine("Le nombre à trouver est plus grand"); }
                                else Console.WriteLine("Le nombre à trouver est plus petit");
                            }
 
                    }
                        else
                        Console.WriteLine ("La valeur saisie est incorrecte ,veuillez recommencer ...");
 
                }
 
 
                Console.WriteLine("Vous avez trouvé le nombre en " + i + " coups");
 
        }
 
    }
 
}

Si vous pourriez m'éclairer, ce serait fort sympathique.



Le code est en attachement.

Merci

[tomlev]

Ton appel à Convert.ToInt32 est inutile puisque tu fais un int.TryParse derrière, et il peut échouer... si ça arrive, le programme se termine, et donc ton int.TryParse n'est jamais atteint. D'autre part, tu n'utilises jamais la variable valeurSaisie qui est le résultat du int.TryParse...

Il faut donc simplement que tu dégages le Convert.ToInt32 et la variable saisie1 qui ne servent à rien ; sinon ça semble à peu près correct.

Sinon, 2 petits conseils :

- utilise le débogueur pour résoudre ce genre de problème ; l'exécution pas à pas permet de voir facilement où est le bug.

- commenter son code, c'est bien, mais mettre des commentaires comme "Appel de la méthode Touche" ou "Converti le string Saisie en int", ça ne sert strictement à rien... on voit très bien ce que fait l'instruction sans avoir besoin du commentaire. Ne mets pas des commentaires juste parce qu'on t'a dit qu'il fallait en mettre ; mets en seulement quand c'est nécessaire à la bonne compréhension du code. Un code bien écrit peut souvent se passer complètement de commentaires ou presque, parce qu'on comprend ce qu'il fait juste en le lisant.

[mrpopo]

Merci du conseil. Cela marche maintenant avec le code suivant:

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
 
            Touche(); 
 
        }
        static void Touche()
        {
            int valeurATrouver = new Random().Next(0, 100);
            int i = 1;
            bool trouve = false;
            string saisie = string.Empty;
 
            Console.WriteLine(valeurATrouver); // Affiche le nombre à trouver 
 
            Console.WriteLine(" Veuillez saisir votre nombre (entre 0 et 100) : ");
 
                while (!trouve)
                {
 
                    saisie = Console.ReadLine();
                    int valeurSaisie;
 
                    if (int.TryParse (saisie , out valeurSaisie )) 
                    {
 
 
                        if (valeurSaisie == valeurATrouver)
                            {
                                trouve = true;
                            }
                            else
                            {
                                i++;
                                if (valeurSaisie < valeurATrouver)
                                { Console.WriteLine("Le nombre à trouver est plus grand"); }
                                else Console.WriteLine("Le nombre à trouver est plus petit");
                            }
 
                    }
                        else
                        Console.WriteLine ("La valeur saisie est incorrecte ,veuillez recommencer ...");
 
                }
 
 
                Console.WriteLine("Vous avez trouvé le nombre en " + i + " coups");
 
        }
 
    }
 
}

[tomlev]

N'oublie pas de marquer le topic comme