Discussion :

[bouceka]

Bonjour, alors j'explique mon problème:
Dans un exercice on demande de créer une structure produit qui contient deux chaine: le code du produit et l'intitulé du produit.
Voici le code en version light :

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
 
#include <iostream>
using namespace std;
 
struct produit
{
    char code[10];
    char title[100];
};
 
 
void nvxprd(produit &prd,int &in)
{
    cout << "Entrez le code du produit: ";
    cin.getline(prd.code,10);
 
    cout << "Entrer l\'intitule du produit: ";
    cin.getline(prd.title,100);
 
    in++;
 
}
 
int main()
{
    bool flag = true;
    int choix=0,nb=0;
    produit liste[10];
 
    while(flag)
    {
        cout << "1. Ajouter un produit\n"
                "6. Quitter\n"
                "Votre choix: ";
        cin >> choix;
 
        switch(choix)
        {
            case 1:
                nvxprd(liste[nb],nb);
                break;
            case 6:
                flag = false;
            default:
                cout << "Commande incorrect !\n";
                break;
        }
    }
 
return 0;
}

Le problème est dans la fonction nvxprd où lorsque je choisit l'option 1 il ne me laisse pas saisir le code du produit .
Voila, j'attends vos réponse

[Elendil_BzH]

Bonjour,

Le problème n'est pas dans la fonction, j'ai pas testé le code mais il y a de grandes chances que ça vienne du 'cin >> choix;', si tu mets un getline à la place ça devrait marcher. Ou alors, tu peux rajouter ça après le 'cin >> choix;' : http://c.developpez.com/faq/cpp/?pag...e#SL_cin_clear

Je sais pas trop pourquoi ça fait ça, peut-être que le tampon n'est pas vide après un appel à 'cin >> choix;' et que du coup le premier getline ne fonctionne pas.

Et je rajouterais un break après la ligne 'flag = false; '

[swirtel]

Premièrement, ne PAS employer de char [] comme stockage de chaine de caractère.

Employez simplement des std::string.

Cela évite les buffer overflows, surtout les deux que j'ai vu son monumentaux.

Concernant le fait qu'il ne te donne pas la possibilité d'introduire une information pour le code du produit, vient du faite des résidus se trouvant toujours dans le buffer de std::cin.

Comme le dit Elendil_BzH, vide ton buffer.

[bouceka]

Elendil_BzH: effectivement cela venais du 'cin>> choix', apres avoir purger le buffer sa fonctionne parfaitement .

swirtel: en faite la contrainte de l'éxercice est justement d'utiliser les char[] autrement j'aurais utiliser les strings.


Merci à vous deux

[swirtel]

Tu ne l'avais pas précisé dans ton exemple.

Bonne soirée,