Problème avec mon getLine() ?

Version imprimable

Bonjour,

Je souhaiterai saisir des vers, puis afficher ma strophe, comme ceci:

Code:

1
2
3
4
5
6
int main()
{
    Strophe S1;
    cin >> S1;
    cout << S1 << endl;
}

Voici une partie de mon (micro)projet. ;)
(j'ai supprimé les #include)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
class Vers {
 
    private:
        string suiteMots;
        string rime;
 
    public:
        Vers();
        Vers(string s);
        Vers(string s, string r);
        virtual ~Vers();
        virtual string getSuiteMots() const;
        virtual void saisie(istream& is);
        void affiche(ostream& os) const;
};
 
ostream& operator<<(ostream& os, const Vers& v);
istream& operator>>(istream& is, Vers& v);

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
Vers::Vers(){}
Vers::Vers(string sm){ suiteMots=sm;}
Vers::Vers(string sm, string r) { suiteMots=sm; rime= r;}
Vers::~Vers(){}
 
void Vers::saisie(istream& is) { 
    cout << "Entrez le vers." << endl; 
    getline(is,suiteMots); 
}
 
void Vers::affiche(ostream& os) const { os << "<<" << suiteMots << ">>";}
 
ostream& operator<<(ostream& os, const Vers& v) {
    v.affiche(os);
    return os;
}
 
istream& operator>>(istream& is, Vers& v) {
    v.saisie(is);
    return is;
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
class Strophe {
 
    private:
        Vers** suiteVers;
        int nbVers;
 
    public:
        Strophe();
        virtual ~Strophe();
        virtual void saisie(istream& is);
        virtual void affiche(ostream& os) const;
};
 
ostream& operator<<(ostream& os, const Strophe& s);
istream& operator>>(istream& is, Strophe& s);

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
Strophe::Strophe(){suiteVers=NULL; nbVers=0;}
Strophe::~Strophe(){if (suiteVers) delete[] suiteVers;}
 
void Strophe::saisie(istream& is) {
    if (suiteVers) delete[] suiteVers;
    cout << "Entrer le nombre de vers : " << endl;
    is >> nbVers;
    suiteVers = new Vers*[nbVers];
    for (int i=0; i<nbVers; i++) {
        Vers *v = new Vers();
        v->saisie(is);
        suiteVers[i] = v;
    }
}
 
void Strophe::affiche(ostream& os) const {
    for (int i=0; i<nbVers; i++) {
        suiteVers[i]->affiche(os); os << endl;
    }
}
 
ostream& operator<<(ostream& os, const Strophe& s) {
    s.affiche(os);
    return os;
}
 
istream& operator>>(istream& is, Strophe& s) {
    s.saisie(is);
    return is;
}

Au début lorsque je saisissai un vers, seul le premier mot était stocké.
Après une petite recherche, j'ai compris fallait utiliser getline().

Sauf que maintenant j'ai un autre problème, lorsque je dis a mon programme que je veux saisir x vers. Il va laisser le 1er vide et me demander de saisir les x-1 autres...

Concrètement,

Code:

1
2
3
4
5
6
7
8
9
10
11
Entrer le nombre de vers
3                         // saisie au clavier
Entrer le vers.
Entrer le vers.
Vers 1                  // saisie au clavier
Entrer le vers.
Vers 2.                // saisie au clavier
// affichage de ma strophe
<<>>
<<Vers 1>>
<<Vers 2>>

Je pense que le problème vient de mon "getline(is,suiteMots)" parce que si je le replace par "is >> suiteMots" je n'ai plus de ce problème.

Quelqu'un peut m'éclairer? ;)

06/02/2014, 17h06
JolyLoic

>> et getline ont parfois du mal à travailler ensemble :

Ton cin >> i ne va pas vider du tampon le caractère de saut de ligne. getline va donc lire le reste de la ligne : rien.

Utilise std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n'); Pour vider le tampon avant le premier getline.

Op.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
void Strophe::saisie(istream& is) {
    if (suiteVers) delete[] suiteVers;
    cout << "Entrer le nombre de vers : " << endl;
    is >> nbVers;
    suiteVers = new Vers*[nbVers];
    cin.ignore(256,'\n');
    for (int i=0; i<nbVers; i++) {
        Vers *v = new Vers();
        v->saisie(is);
        suiteVers[i] = v;
    }
}

Ca marche! Je te remercie. :)

Cependant, je vois pas de quel saut de ligne on parle, en fait.

07/02/2014, 01h50
koala01

Salut,

Citation:

Envoyé par Jéjé34

Cependant, je vois pas de quel saut de ligne on parle, en fait.

Eh bien, typiquement, une entrée clavier est terminée par l'appui <enter>. Cet appui a pour double résultat de placer le caractère de retour chariot '\n' après le dernier caractère ajouté, et de signaler au flux qu'il est temps de se mettre à jour.

C'est de ce '\n' que l'on parle ;)

Ceci dit, tu ne devrais pas utiliser une valeur arbitraire comme 256 pour cin.ignore. La raison est toute simple : tu ne sais jamais ce que l'utilisateur pourra effectivement avoir voulu encoder, ni si la taille que tu indiques vouloir ignorer sera suffisante (l'utilisateur peut toujours vouloir ignorer la valeur arbitraire +1 caractères :aie:).

C'est la raison pour laquelle il est préférable d'utiliser std::numeric_limits<size_t>::max(), qui nous assure que la valeur arbitraire utilisée sera celle de la plus grande valeur numérique pouvant représenter une taille possible ;)

Merci. ;)

J'ai introduit la saisie de rimes.

Code:

1
2
3
4
5
void Vers::saisie(istream& is) {
    cout << "Entrez le vers puis la rime." << endl;
    getline(is,suiteMots);
    is >> rime;
}

Et utilisé numeric_limits<size_t>::max() comme vous me l'avez conseillé.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
void Strophe::saisie(istream& is) {
    if (suiteVers) delete[] suiteVers;
    cout << "Entrer le nombre de vers : " << endl;
    is >> nbVers;
    suiteVers = new Vers*[nbVers];
    for (int i=0; i<nbVers; i++) {
        Vers *v = new Vers();
        cin.ignore(numeric_limits<size_t>::max(),'\n');
        v->saisie(is);
        suiteVers[i] = v;
    }
}

Mais ca ne fonctionne pas. :cry:
Pour un petit programme qui me demande de saisir une strophe puis l'affiche, ca me donne:

Code:

1
2
3
4
5
6
7
8
9
Entrer le nombre de vers
2        // saisie
Entrer le vers puis la rime
Vers 11       // saisie du vers
Entrer le vers puis la rime
Vers 12       // saisie du vers
// affichage
<<>>
<< 11>>

Bref, il me laisse pas saisir mes rimes et m'affiche n'importe quoi. Alors que si je mets 256 qui est certes arbitraire et pas très logique mon programme fonctionne correctement:

Code:

1
2
3
4
5
6
7
8
9
10
11
Entrer le nombre de vers
2        // saisie
Entrer le vers puis la rime
Vers 11       // saisie du vers
11              // saisie de la rime
Entrer le vers puis la rime
Vers 12       // saisie du vers
12             // saisie de la rime
// affichage
<<Vers 11>>
<<Vers 12>>

07/02/2014, 11h39
JolyLoic

Il ne faut pas mettre le ignore dans ta boucle. Si tu mets ignore entre 2 getline, ça ne marchera pas.... Il est juste utile entre un << et un getline.

J'ai testé de le mettre ici.
Code:

1 2 3 4 5 6 void Vers::saisie(istream& is) { cout << "Entrez le vers puis la rime." << endl; cin.ignore(256,'\n'); getline(is,suiteMots); is >> rime; }
Code:

1 2 3 4 5 6 7 8 9 10 11 void Strophe::saisie(istream& is) { if (suiteVers) delete[] suiteVers; cout << "Entrer le nombre de vers : " << endl; is >> nbVers; suiteVers = new Vers*[nbVers]; for (int i=0; i<nbVers; i++) { Vers *v = new Vers(); v->saisie(is); suiteVers[i] = v; } }
Ca ne fonctionne toujours pas.

Dès que j'appuie sur Entrée après avoir saisie un vers, il passe directement au vers suivant sans me demander de saisir le rime.
Et lorsque j'affiche ma Strophe, elle est vide, aucun vers que j'ai saisie n'apparait.

Alors que si je met 256, ça fonctionne...
Ca doit donc venir du numeric_limits<size_t>::max(), mais pourquoi...:roll: