Discussion :

[Ziarrow]

Est-ce que quelqu'un saurait pourquoi cin.getline et cin.ignore ne marcheraient pas...

exemple :

Code :

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

1
2
 
cin.getline(cle, 255);

Ca passe à la suite et je ne peux rien saisir.....

et autre exemple :

Code :

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

1
2
 
cin.ignore( numeric_limits<streamsize>::max(), '\n' );

Pareil, pas de pause, ça passe à la suite....

[Astraya]

Vérifie ici, si ton code correspond a l'exemple car avec ce que tu nous donnes, nous ne pouvons guère plus.

http://www.cplusplus.com/reference/i...tream/getline/

[Ziarrow]

J'ai vérifié et c'est exactement comme je l'ai écrit...

J'ai fais un petit test en mettant un cin.getline(variable) au début de mon code dans le int main() et ca marche, par contre dès que j'utilise cin.ignore ou cin.getline dans ma fonction, ca ne marche plus...

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
 
...
 
int main
{
...
switch(choix)
    {
      case 1: cryptxor();
                 break;
 
      ...
 
      case 4: default:
                 break;
    }
}
 
int cryptxor()
{
    char filesourc[50], filedest[50], cle[255];
 
    ...
 
   cout << "\n\t\t Entr\202 la cl\202 pour le cryptage : ";
   cin.getline (cle, 255);
 
   ...
 
   cin.ignore( numeric_limits<streamsize>::max(), '\n' );
 
   ...

Mes erreurs sont sans doute beaucoup plus visible pour vous maintenant....

[Astraya]

Et en essayant ceci :

Code :

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

1
2
 
cout << "\n\t\t Entr\202 la cl\2O2 pour le cryptage : " << endl ;

ça te permet de faire un retour a la ligne + un std::flush()

[koala01]

Salut,

De manière générale, en C++, tu aurais largement intérêt à utiliser la classe std::string plutôt qu'une chaine de caractères "C style" (un tableau de caractères terminé par '\0'), cela t'évitera bien des déboires.

De même, il devient alors plus intéressant d'utiliser la fonction libre getline plutôt que la fonction membre de cin qui porte le même nom.

Cela nécessitera cependant une petite étape supplémentaire, si tu souhaite utiliser un bloc switch... case car un tel bloc ne peut fonctionner qu'avec des valeurs numériques entières, mais, comme il est de toutes façons toujours intéressant de vérifier que l'utilisateur a introduit une valeur cohérente avec celle que l'on attend (on parle de subir le "test du singe" pour s'en assurer ), tu fera d'une pierre deux coups

Au final, tu pourrais avoir une fonction proche de

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
int choice()
{
    int c =0;
    /* on boucle sur la demande jusqu'à ce que l'on obtienne
     * quelque chose de cohérent
     */
    do
    {
        std::string rep;
        std::cout<<"veuilliez choisir ce que vous voulez faire"<<std::endl
                 <<"    1- machin chose"<<std::endl
                 <<"    2 brol"<<std::endl
                 <<"    3 bidule"<<std::endl
                 <<"votre choix :";
        std::getline(cin, rep);
        /* on passe par un flux de conversion */
        std::stringstream ss;
        ss<<rep;
        /* si l'utilisateur n'a pas introduit un chiffre, cela ne nous va
         * pas
         */
        if(! ss>>c)
            std::cout<<"Veuillez introduire une valeur numerique seulement"
                     <<std::endl;
       else
       {
            /* s'il a bien introduit une valeur numérique mais que 
             * celle-ci ne correspond pas à l'une des valeurs attendues,
             * cela ne nous va pas non plus :P
             */
            if(c!=1 && c!=2 && c!=3)
                std::cout<<"veuillez choisir entre 1, 2 et 3 uniquement"
                         <<std::endl;
       }while(c==1 || c==2 || c==3);
       return c;
}

Grâce à l'utilisation de la fonction libre et d'une chaine de type std::string, tu a la certitude que l'ensemble de ce qui pourra avoir été introduit par l'utilisateur aura, effectivement, été "utilisé" pour créer la chaine de caractères de la réponse.

Tu pourra alors utiliser cette fonction dans ta fonction principale sous une forme proche de

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
int main()
{
    int choix=choice();
    switch(choix)
    {
        /*...*/
    }
    /* et pour faire une pause avant de quitter
     */
 
    std::cout << "Appuyez sur entree pour continuer...";
    std::cin.ignore( std::numeric_limits<std::streamsize>::max(), '\n' );
    return 0;
}

Après, on me dira sans doute que l'on pourrait placer le code de cette fonction directement ans la fonction main au lieu de créer une fonction "rien que pour cela"...

Ce n'est pas tout à fait faux, mais c'est oublier que, dans l'idéal, une fonction doit se limiter à une seule responsabilité car, si elle en a plus d'une, c'est souvent qu'elle en a trop, et qu'il est toujours plus simple de s'y retrouver dans dix fonctions de 15 lignes que dans une seule fonction qui en fait 150

[Ziarrow]

D'accord, merci pour tous ces renseignements, cela répond clairement à mon problème.

Je me met de ce pas à changer tous ça..... et je vais essayer de trouver "l'art poétique" de Nicolas Boileau car j'ai cru avoir un éclair de compréhension autant dans la réponse qu'en lisant cette citation......

Encore merci.

[Ziarrow]

Je voudrais rajouter un détails car j'ai trouvé une solution au problème énoncé :

Apparemment, il y a au début du tampon du flux en entrée donc au début du tampon de std::cin, un changement de ligne ('\n') ou un retour de chariot ('\r') qui traîne...

Ainsi en faisant un appel à std::cin.peek(), cela retourne le premier caractère au début du flux en entrée...

Exemple :

Code :

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

1
2
3
4
5
6
7
 
   string var;
   char h = cin.peek();
   if (h == '\n' || h == '\r')
      cin.get(h);
    // lire une ligne à l'entrée standard, la mettre dans s
   getline (cin, var);

Cette solution à résolu mon problème qui à persisté après ce post....