Exercice d'algo - demande d'aide

Version imprimable

Bonjour, je commence le C#, j'essaye de faire un exercice, de base c'était de l'algo. Ce programme est censé demander 20 nombre successivement pour donner le nombre le plus grand et la position à la quelle il a était saisi , le hic c'est que ça n'en demande que 10 et la position donner est mauvaise, quelqu'un peut donc me corrigé s'vous plait?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 int N, i, PG, IPG; //Déclaration des variables IPG = PG = 0; for (i = 1; i <= 20; i++) //Boucle de 1 à 20 { Console.Write("Entrez un nombre: "); //Affichage N = Int32.Parse(Console.ReadLine()); //Lecture if (i == 1 || N > PG) //Si { PG = N; IPG = i; } i++; } Console.WriteLine("Le nombre le plus grand était: " + PG); //Le nombre le plus grand Console.WriteLine("Il a été saisi en position numéro: " + IPG); //Sa position

27/10/2015, 21h01
sevyc64

Tu incrément 2 fois ton index i.

Le 3ème paramètre du For est l'opération qui est faite automatiquement à la fin de la boucle. Ici, tu y met i++, donc ton index i sera automatiquement incrémenté à la fin de la boucle. Tu n'as donc pas besoin de rajouter l'incrémentation à l'intérieur de ta boucle.
27/10/2015, 21h41
Kuddi

Citation:

Envoyé par sevyc64

Tu incrément 2 fois ton index i.

Le 3ème paramètre du For est l'opération qui est faite automatiquement à la fin de la boucle. Ici, tu y met i++, donc ton index i sera automatiquement incrémenté à la fin de la boucle. Tu n'as donc pas besoin de rajouter l'incrémentation à l'intérieur de ta boucle.

Effectivement tu as totalement raison, merci pour ta rapidité! C'est autre chose que Ope..C..
27/10/2015, 21h46
Kuddi

Pendant que j'y suis, j'vais être chiant mais comment faire pour demander à l'utilisateur si il veut commencé le programme si choix==oui (que ça soit en majuscule ou en minuscule) est bien le programme commence, si choix==non le programme ne s’exécute pas est se ferme et une commande de fin, si l'utilisateur souhaite réessayer le programme se recommence depuis le début.
27/10/2015, 22h58
sevyc64

Console.ReadLine pour lire le choix de l'utilisateur,
Un bloc If..Then pour traiter le choix, style If non then exit (ou return, je sais plus à force de mélanger les langages)

Ensuite pour traiter minuscule ou majuscule, l'astuce est de passer la réponse de l'utilisateur, quel qu'elle soit, soit tout en minuscule avec ToLower sur la string, soit tout en majuscule avec ToUpper. Comme ça tu n'as ensuite qu'à traiter un seule cas, en minuscule, ou en majuscule suivant la conversion que tu as fait juste avant.

Citation:

Envoyé par sevyc64

Console.ReadLine pour lire le choix de l'utilisateur,
Un bloc If..Then pour traiter le choix, style If non then exit (ou return, je sais plus à force de mélanger les langages)

Ensuite pour traiter minuscule ou majuscule, l'astuce est de passer la réponse de l'utilisateur, quel qu'elle soit, soit tout en minuscule avec ToLower sur la string, soit tout en majuscule avec ToUpper. Comme ça tu n'as ensuite qu'à traiter un seule cas, en minuscule, ou en majuscule suivant la conversion que tu as fait juste avant.

J'ai pas trop réussis l'astuce du To.Upper et To.Lower, mais c'était "Environment.Exit(0);", Merci man. :)

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
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
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
 
namespace ConsoleApplication10
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("Bienvenue veut-tu commencer ce programme <oui> ou <non>");
            string choix, str_Recommencer;
            choix = Console.ReadLine();
 
 
            do //Début de la boucle de recommencement
            {
 
                if (choix == "non" || choix == "Non" || choix == "NON")
 
            {
                Environment.Exit(0);
            }
            else
            {
 
            }
 
 
 
            int u, i, PG, IPG; //Déclaration des variables
            string N;
 
            IPG = PG = 0;
            Console.WriteLine("Veuillez saisir 20 nombres succèsivement : ");
 
            for (i = 1; i <= 20; i++) //Boucle de 1 à 20
            {
 
                Console.Write("Saisie: ");
                N =Console.ReadLine(); //Lecture
                int.TryParse(N, out u);
                if (i == 1 || u > PG) //Si
                {
                    PG = u;
                    IPG = i;
                }
 
            }
            Console.WriteLine("Le nombre le plus grand était: " + PG);
            Console.WriteLine("Il a été saisi en position numéro: " + IPG);
            Console.ReadKey();
 
            Console.Write("\n\nVoulez-vous recommencer ? (o / n) :"); //On demande à l'utilisateur si il veut recommencer le programme
            str_Recommencer = Console.ReadLine(); //On lit la reponse de l'utilisateur
        }
            while (str_Recommencer == "oui" || str_Recommencer == "o");
            //Si l'utilisateur répond oui, on recommence le programme
        }
    }
}

28/10/2015, 07h38
jopopmk

Salut,

un truc qui va bien t'aider dans ton apprentissage et même après, c'est la doc MSDN ;) Voilà la fiche de String.ToUpper().

Avec une recherche du type "Classe.Methode MSDN" dans google tu devrais presque toujours l'avoir en premier résultat
(ce qui n'est pas nécessairement vrai avec qwant).
29/10/2015, 11h07
TheBlackReverand
Bonjour,

Sur le code suivant
Code:

1 2 3 4 5 6 7 8 if (choix == "non" || choix == "Non" || choix == "NON") { Environment.Exit(0); } else { }
le else n'est pas nécessaire car vide.
si je saisie nOn cela ne fonctionne pas, d'ou l'astuce de sevyc64
j'irais même plus loin, si ton utilisateur est trèèèès courtois et saisie "non monsieur" :D (comment ça je chipote?)

Voici ton code adapté pour géré les majuscules/minuscules ET la saisie de mot superflus.
Code:

1 2 3 4 if (choix.ToUpper().Contains("NON")) { Environment.Exit(0); }
si je saisie "nOn monsieur!"
le ToUpper() donne "NON MONSIEUR!"
le Contains recherche "NON" et renvoi True (j'ai trouvé une correspondance)

et je confirme, msdn for ever ;)
29/10/2015, 13h13
sevyc64
Citation:
Envoyé par TheBlackReverand

Voici ton code adapté pour géré les majuscules/minuscules ET la saisie de mot superflus.

Code:

1 2 3 4 if (choix.ToUpper().Contains("NON")) { Environment.Exit(0); }

si je saisie "nOn monsieur!"
le ToUpper() donne "NON MONSIEUR!"
le Contains recherche "NON" et renvoi True (j'ai trouvé une correspondance)

et je confirme, msdn for ever ;)
De même que si tu saisie Canon-ball ou Dénonciation ou ... (je te fais grâce de tout le dico) :mrgreen:
29/10/2015, 14h12
TheBlackReverand

oui ok j’avoue :D

Code:

choix.ToUpper().Split(',', ' ', '.', '!', ......).Contains("NON")

comment ça je tue des mouches avec un bazooka? :aie: