saisie getline string

Version imprimable

Bonjour,

la fonction getline est zappée:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
#include <iostream>
#include <string>
#include "entrees.hpp"
 
int main(){
  int choix;
  while( ! saisir_int(choix,"entrez un nombre: "))
    ;
  std::string X;
  saisir_string(X,"entrez une chaine");
  std::cout<<X<<std::endl;
}

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
64
65
66
67
#include <iostream>
#include <string>
#include <sstream>
#include <limits>
 
void vider_buffer();
 
void saisir_string(std::string &variable, const std::string &message){
  while(true){
    std::cout << message;
    std::getline(std::cin, variable); 
    if(std::cin.bad() || std::cin.eof()){
      std::cerr << "Erreur, saisie incorrecte." << std::endl;
      if(std::cin.eof()){
	break;
      }
      vider_buffer();
    }
    else if(std::cin.fail()){
      std::cerr << "Erreur, saisie incorrecte." << std::endl;
      vider_buffer();
    }
    break;
  }
}
 
bool saisir_int(int &variable, const std::string &message){
  std::string temp;
  while(true){
    std::cout << message;
    std::cin >> temp;
    if(std::cin.bad() || std::cin.eof()){
      std::cerr<<"Une erreur interne est survenue." << std::endl;
      if(std::cin.eof()){
	return false;
      }
      vider_buffer();
      continue;
    }
    else if(std::cin.fail()){
      std::cerr << "Erreur, saisie incorrecte." << std::endl;
      vider_buffer();
      continue;
    }
    vider_buffer();
    std::istringstream stream(temp);
    stream >> variable;
    if(stream.fail() || !stream.eof()){
      std::cerr << "Erreur, saisie incorrecte." << std::endl;
    }
    else{
      return true;
    }
  }
  return false;
}
 
 
void vider_buffer(){
  std::cin.clear();
  std::cin.seekg(0, std::ios::end);
  if(!std::cin.fail()){
    std::cin.ignore(std::numeric_limits<std::streamsize>::max());
  }
  else
    std::cin.clear();
}

Code:

1
2
entrez un nombre: 3
entrez une chaine

j'ai trouvé ces fonctions ici:
http://sdz.tdct.org/sdz/la-saisie-securisee-en-c.html

quelqu'un a une idée?

09/05/2024, 05h01
emmesse

Je me suis peut-être mal expliqué.
la ligne std::getline(std::cin, variable); est ignorée à l’exécution. variable est affecté de la chaine vide.
j'aurai besoin de saisir une chaine après avoir saisi un entier
quelqu'un aurai une idée?

si j'ajoute vider_buffer avant la saisie de la chaîne, cela ne change rien

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
void saisir_string(std::string &variable, const std::string &message){
  while(true){
    std::cout << message;
    vider_buffer();
    std::getline(std::cin, variable); 
    if(std::cin.bad() || std::cin.eof()){
      std::cerr << "Erreur, saisie incorrecte." << std::endl;
      if(std::cin.eof()){
	break;
      }
      vider_buffer();
    }
    else if(std::cin.fail()){
      std::cerr << "Erreur, saisie incorrecte." << std::endl;
      vider_buffer();
    }
    break;
  }
}

Code:

1
2
3
4
5
6
7
8
9
void vider_buffer(){
  std::cin.clear();
  std::cin.seekg(0, std::ios::end);
  if(!std::cin.fail()){
    std::cin.ignore(std::numeric_limits<std::streamsize>::max());
  }
  else
    std::cin.clear();
}

une idée?

09/05/2024, 13h02
dalfab
Bonjour,

Les fonctions de saisie sont inutilement complexes. Entre autre avec des std::endl sur std::cerr qui n'est pas bufférisé, pas de std::flush après les std::cout!

Quant à la fonction vider_buffer(), elle utilise seekg() qui n'a pas de sens sur un un flux. Il faudrait plus simplement:
Code:

1 2 3 4 void vider_buffer(){ std::cin.clear(); std::cin.ignore( std::numeric_limits<std:streamsize>::max() ); }

Merci pour la réponse.

Avec le vider_buffer que tu me propose, le message "entrez une chaine" ne s'affiche plus. Le programme lit un entier et ne passe pas à la suite.

mes saisies compliquées sont expliqués ici:http://sdz.tdct.org/sdz/la-saisie-securisee-en-c.html

sinon,j'ai le même problème avec ce code
Code:

1 2 3 4 5 6 7 8 9 10 11 #include <iostream> int main(){ int n; std::cout<<"entrez un nombre"<<std::endl<<std::flush; std::cin>>n; std::cout<<"entrez une chaine"<<std::endl<<std::flush; std::string X; std::getline(std::cin,X); std::cout<<X<<std::endl<<std::flush; }
quand il demande une chaîne, getline est zappée et X ne contient rien:
Code:

1 2 3 4 5 6 $ ./a.out entrez un nombre 5 entrez une chaine $
une idée?

Ça marche avec std::cin.ignore:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
#include <iostream>
 
int main(){
  int n;
  std::cout<<"entrez un nombre"<<std::endl<<std::flush;
  std::cin>>n;
  std::cout<<"entrez une chaine"<<std::endl<<std::flush;
  std::string X;
  std::cin.ignore();
  std::getline(std::cin,X);
  std::cout<<X<<std::endl<<std::flush;
}

Code:

1
2
3
4
5
6
$ ./a.out 
entrez un nombre
54
entrez une chaine
bonjour toi
bonjour toi

J'ai abandonné les saisies compliquées car ça ne marchait toujours pas. Voici la dernière version:

Code:

1
2
3
4
5
#include "partie.hpp"
 
int main(){
  partie jeu;
}

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
#include <vector>
#include <string>
#include <algorithm>
#include <iostream>
#include <random>
 
#include "partie.hpp"
 
partie::partie(){
  std::random_device rd;
  std::mt19937 gen(rd());
  std::uniform_int_distribution<int> distrib_profession(0,11);
  int choix;
  do{
    std::cout<<"Combien de joueurs pour cette partie? "<<std::endl<<std::flush;
    std::cin>>choix;
    if(choix<1||choix>6)
      std::cout<<"le nombre de joeurs doit être compris enetre 1 et 6"<<std::endl;
  }while(choix<1||choix>6);
  std::string nom_joueur;
  int numero_profession;
  std::vector<int>profession_utilisee;
  for(int i=0;i<choix;i++){
    std::cout<<"nom du joueur "<<i+1<<": ";
    std::cin.ignore();
    std::cin>>nom_joueur;
    do
      numero_profession=distrib_profession(gen);
    while ( find(profession_utilisee.begin(),profession_utilisee.end(),numero_profession) != profession_utilisee.end());
    profession_utilisee.push_back(numero_profession);
    std::cout<<"Joueur "<<nom_joueur<<", votre profession est "<<liste_professions[numero_profession]<<std::endl;
    joueur temp(liste_professions[numero_profession]);
    liste_joueurs.push_back(temp);
    liste_noms.push_back(nom_joueur);
  }
}

17/05/2024, 10h57
Médinoc

L'autre solution étant de ne faire que des getline(), puis faire tes conversions sur la ligne saisie avec un istringstream.
Avec ça, tu ne devrais plus avoir de soucis de caractères "laissés" dans le flux d'entrée.
17/05/2024, 12h33
emmesse

un exemple?

Un exemple:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
#include <iostream>
#include <string>
#include <sstream>
 
int getline_main(){
	int n;
	std::string line;
	std::cout<<"entrez un nombre"<<std::endl; //De mémoire, endl implique déjà flush
	std::getline(std::cin, line);
	std::istringstream(line) >> n;
	std::cout<<"entrez une chaine"<<std::endl;
	std::string X;
	std::getline(std::cin,X);
	std::cout<<X<<std::endl;
	return EXIT_SUCCESS;
}

22/05/2024, 11h47
emmesse

oui ça marche
22/05/2024, 13h01
Luc Hermitte

Problème classique d'enchainement de getline après >>. La question est traitée dans la FAQ (comme plusieurs des autres que tu as postées)

https://cpp.developpez.com/faq/cpp/?...ligne-soit-lue
22/05/2024, 20h21
emmesse

que penser de ceci:

http://sdz.tdct.org/sdz/la-saisie-securisee-en-c.html
23/05/2024, 17h02
Luc Hermitte

Que de la même façon, j'avais mis dans la FAQ il y a fort longtemps tout ce qu'il faut pour vérifier les saisies clavier.
Sinon, trop de mots et pas lisible l'article. Parle de choses inutiles (seekg()), et d'autres à bannir (std::istream::getline -- à ne pas confondre avec std::getline)
23/05/2024, 18h14
emmesse

Merci pour les précisions, je prend note