sstream et saisie cin

Version imprimable

Bonjour,

pourquoi b est toujours à 0 ?

une idée ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
#include <iostream>
#include <string>
#include <sstream>
 
int main(){
  std::cout<<"a= ";
  int a;
  std::string saisie;
  std::stringstream geek;
  std::cin>>saisie;
  geek<<saisie;
  geek>>a;
  geek.flush();
  std::cout<<"b= ";
  int b;
  std::cin>>saisie;
  geek<<saisie;
  geek>>b;
  std::cout<<a<<' '<<b<<std::endl;
}

Code:

1
2
3
4
$ ./a.out 
a= 56
b= 12
56 0

28/11/2025, 11h26
bacelar

Coincé par les CR-LF ?
28/11/2025, 12h36
Luc Hermitte

Hum...

Je n'utilise jamais les stringstream en I/O. J'utilise uniquement l'un ou l'autre.

Quant au reset, je fais ça avec `iss.str("")`, voire directement dans ton cas avec `iss.str(saisie)`.

Au passage, toujours tester les extractions. Ca permettrait de savoir quand cela commence à ne plus aller.

Hello,

Le problème vient du fait que geek.flush() ne sert pas à réinitialiser le flux pour une réutilisation.

Chronologie dans votre programme :

geek << saisie("56") : Le flux contient "56".
geek >> a : Le flux lit "56". Il arrive à la fin de la chaîne interne. Le drapeau eofbit (End Of File) est levé.
geek.flush() : Cette commande synchronise les tampons de sortie, mais elle n'efface pas les drapeaux d'erreur (EOF) et ne vide pas le contenu lu.
geek << saisie("12") : Vous ajoutez "12" au flux. Le flux contient maintenant techniquement "12" (ou une suite selon la position), mais il est toujours considéré comme "en erreur/fini" à cause du eofbit.
geek >> b : Comme le drapeau EOF (ou failbit) est actif, l'opération de lecture refuse de s'exécuter. b n'est pas modifié et reste à 0.

Pour réutiliser un stringstream, vous devez faire deux choses :

geek.clear() : Pour remettre les drapeaux d'état (comme EOF) à zéro.
geek.str("") : Pour vider le texte précédent du tampon (sinon vous allez ajouter "12" à la suite de l'ancien contenu s'il n'a pas été entièrement consommé).

Une proposition de code corrigé,
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 #include <iostream> #include <string> #include <sstream> int main(){ std::cout << "a= "; int a; std::string saisie; std::stringstream geek; // Première lecture std::cin >> saisie; geek << saisie; geek >> a; // --- CORRECTION ICI --- geek.clear(); // 1. Réinitialise les drapeaux d'erreur/EOF geek.str(""); // 2. Vide le contenu du buffer // ---------------------- std::cout << "b= "; int b; std::cin >> saisie; geek << saisie; // Maintenant le flux est propre et prêt geek >> b; std::cout << a << ' ' << b << std::endl; return 0; }
Si vous n'êtes pas contraint d'utiliser stringstream pour un exercice,
l'approche moderne en C++ (depuis C++11) pour convertir une string en intest d'utiliser std::stoi.

C'est plus simple et évite ces problèmes d'état :
Code:

1 2 3 // Remplacer toute la mécanique stringstream par : std::cin >> saisie; a = std::stoi(saisie);

merci à vous tous
voici le source final:

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
#include <iostream>
#include <string>
 
int main(){
  std::cout<<"Salaire : ";
  std::string saisie;
  std::cin>>saisie;
  int salaire=std::stoi(saisie);
  std::cout<<"Impôts : ";
  std::cin>>saisie;
  int impots=std::stoi(saisie);
  std::cout<<"Remboursement de l'hypothèque : ";
  std::cin>>saisie;
  int remb_hypotheque=std::stoi(saisie);
  std::cout<<"Remboursement du prêt étudian : ";
  std::cin>>saisie;
  int remb_etudiant=std::stoi(saisie);
  std::cout<<"Remboursement du prêt véhicule : ";
  std::cin>>saisie;  
  int remb_vehicule=std::stoi(saisie);
  std::cout<<"Remboursement des dettes de la carte de crédit : ";
  std::cin>>saisie;
  int remb_carte=std::stoi(saisie);
  std::cout<<"Autre dépenses : ";
  std::cin>>saisie;
  int autres_depenses=std::stoi(saisie);
  std::cout<<"remboursement prêt bancaire : ";
  std::cin>>saisie;  
  int remb_banque=std::stoi(saisie);
  std::cout<<"Dépenses enfants : ";
  std::cin>>saisie;
  int depenses_enfants=std::stoi(saisie);
  std::cout<<"Cashflow : ";
  std::cin>>saisie;
  int cashflow=std::stoi(saisie);
  std::cout<<"Caisse : ";
  std::cin>>saisie;
  int caisse=std::stoi(saisie);
  std::cout<<"Votre salaire est de "<<salaire<<std::endl;
  std::cout<<"Votre cashflow est de "<<cashflow<<std::endl;
  std::cout<<"Votre caisse est de "<<caisse<<std::endl;
  std::cout<<"Vos impôts sont de "<<impots<<std::endl;
  std::cout<<"Vos remboursements d'hypothèque sont de "<<remb_hypotheque<<std::endl;
  std::cout<<"Vos remboursements du pret étudiant sont de "<<remb_etudiant<<std::endl;
  std::cout<<"Vos remboursements du pret du véhicule sont de "<<remb_vehicule<<std::endl;
  std::cout<<"Vos remboursements des dettes de carte de crédit sont de "<<remb_carte<<std::endl;
  std::cout<<"Vos autres dépenses sont de "<<autres_depenses<<std::endl;
  std::cout<<"Vos remboursements du pret bancaire sont de "<<remb_banque<<std::endl;
  std::cout<<"Vos dépenses pour les enfants sont de "<<depenses_enfants<<std::endl;
  exit(0);
}

@fred1599 : je viens de faire l'expérience de la différence de la qualité des réponses humaine par repport à l'IA. L'IA m'a juste conseillé geek.clear();geek.str("")

28/11/2025, 19h58
fred1599

exit(0); nécessite la cstdlib

Sur certaines plateformes, il est possible que <iostream> ou <string> inclue en interne <cstdlib>.
Si c'est le cas, le symbole std::exit (ou ::exit par pollution de l'espace de noms global) sera disponible, et le code compilera par accident.

Un compilateur strict, respectant le principe que l'utilisateur doit déclarer ce qu'il utilise, rejettera le code avec une erreur :
"use of undeclared identifier 'exit'".
→ C'est le comportement attendu d'un code C++ correct et portable.

Remarque : La suppression de exit() est une solution architecturale supérieure.

Exemple de saisie utilisateur

Le code demande :
std::cout << "Salaire : ";

- Si l'utilisateur entre 2500 (suivi de Entrée), tout se passe bien.
- Mais si l'utilisateur entre 2500 Euros, voici la séquence désastreuse :

1. std::cin >> saisie lit jusqu'au premier espace → extrait "2500".
2. Le flux contient encore " Euros\n".
3. std::stoi("2500") réussit.
4. Le programme affiche "Impôts : ".
5. Le second std::cin >> saisie consomme immédiatement "Euros".
6. Le programme tente std::stoi("Euros") → exception.

Exceptions de std::stoi

Introduite en C++11, std::stoi lance des exceptions :

- std::invalid_argument : si aucune conversion possible ("Euros", "Hello", chaîne vide).
- std::out_of_range : si la valeur dépasse les limites de int (ex: 9999999999).

Problème : Sans try-catch, l'exception atteint main → std::terminate() → arrêt brutal.

Durée de vie et nettoyage

- En C++, les variables locales sont détruites à la fin de la fonction (Stack Unwinding).
- std::exit() termine immédiatement, sans détruire les variables locales.
- return 0; dans main garantit un nettoyage complet et sûr.
→ exit() doit être réservé aux erreurs irrécupérables.

EXIT_SUCCESS et EXIT_FAILURE

Le standard définit :
- EXIT_SUCCESS (généralement 0)
- EXIT_FAILURE

Utiliser return 0; est plus riche sémantiquement que exit(0);.

Problème de conception : Cashflow

Le programme demande :
- Revenus (Salaire)
- Dépenses (Impôts, prêts, etc.)
- Cashflow

Erreur conceptuelle : Le Cashflow est une donnée dérivée :
Cashflow = Revenus - Σ Dépenses

Risques :
- Erreur de calcul (l'utilisateur doit calculer lui-même).
- Incohérence (ex: Revenus=3000, Dépenses=2000, Cashflow=5000).

→ Un logiciel correct doit garantir l'intégrité des données.

Type utilisé : int

- int ne gère pas les décimales → pas de centimes.
- double ou float : erreurs d'arrondi binaires.
- Approche professionnelle : entiers représentant les centimes ou classes Decimal.

Conclusion : Conserver int est acceptable pour la simplicité, mais limité en précision.

Citation:

Problème de conception : Cashflow

oui tout à fait, où avais-je la tête

Citation:

Type utilisé : int

C'est le jeu "cashflow" de R. Kyiosaky. Les centimes n'interviennent pas
que penses-tu de ça:

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
#include <iostream>
#include <string>
 
int saisie_int(std::string const &message);
 
int main(){
  int salaire=saisie_int("Salaire : ");
  int cashflow=salaire;
  int impots=saisie_int("Impots : ");
  cashflow-=impots;
  int remb_hypotheque=saisie_int("Remboursement de l'hypothèque : ");
  cashflow-=remb_hypotheque;
  int remb_etudiant=saisie_int("Remboursement du prêt étudiant : ");
  cashflow-=remb_etudiant;
  int remb_vehicule=saisie_int("Remboursement du prêt véhicule : ");
  cashflow-=remb_vehicule;
  int remb_carte=saisie_int("Remboursement des dettes de la carte de crédit : ");
  cashflow-=remb_carte;
  int autres_depenses=saisie_int("Autre dépenses : ");
  cashflow-=autres_depenses;
  int remb_banque=saisie_int("Remboursement prêt bancaire : ");
  cashflow-=remb_banque;
  int depenses_enfants=saisie_int("Dépenses enfants : ");
  cashflow-=depenses_enfants;
  int caisse=saisie_int("Caisse : ");
  std::cout<<"Votre salaire est de "<<salaire<<std::endl;
  std::cout<<"Votre cashflow est de "<<cashflow<<std::endl;
  std::cout<<"Votre caisse est de "<<caisse<<std::endl;
  std::cout<<"Vos impôts sont de "<<impots<<std::endl;
  std::cout<<"Vos remboursements d'hypothèque sont de "<<remb_hypotheque<<std::endl;
  std::cout<<"Vos remboursements du pret étudiant sont de "<<remb_etudiant<<std::endl;
  std::cout<<"Vos remboursements du pret du véhicule sont de "<<remb_vehicule<<std::endl;
  std::cout<<"Vos remboursements des dettes de carte de crédit sont de "<<remb_carte<<std::endl;
  std::cout<<"Vos autres dépenses sont de "<<autres_depenses<<std::endl;
  std::cout<<"Vos remboursements du pret bancaire sont de "<<remb_banque<<std::endl;
  std::cout<<"Vos dépenses pour les enfants sont de "<<depenses_enfants<<std::endl;
  return 0;
}
 
int saisie_int(std::string const &message){
  std::string saisie;
  bool saisie_ok;
  do{
    saisie_ok=true;
    std::cout<<message;
    std::getline(std::cin,saisie);
    for(size_t i=0;i<saisie.size();i++)
      if(saisie.at(i)<'0' || saisie.at(i)>'9'){
	saisie_ok=false;
	std::cout<<"Entrez une valeur numérique."<<std::endl;
	break;
      }
  }while( ! saisie_ok);
  return stoi(saisie);
}

La fonction std::stoi ne supporte pas les chaînes vides et termine brutalement le programme si elle en reçoit une.

Votre boucle rejette tout caractère qui n'est pas un chiffre (0-9). Cela signifie que :

- Nombres négatifs impossibles : Si l'utilisateur a un déficit ou veut entrer une dette, il ne peut pas écrire -500 car le signe "-" sera rejeté.
- Espaces : Si l'utilisateur tape "100 " (avec un espace avant), c'est rejeté.

Voici une proposition pour la fonction saisie_int

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
 
#include <iostream>
#include <string>
 
//... (votre main reste identique)
 
int saisie_int(std::string const &message) {
    std::string saisie;
    while (true) { // On boucle indéfiniment jusqu'à un return
        std::cout << message;
        if (!std::getline(std::cin, saisie)) {
             // Gestion d'erreur critique (ex: Ctrl+D / fin de flux)
             return 0; 
        }
 
        // 1. Vérifier si vide
        if (saisie.empty()) {
            std::cout << "Erreur : vous n'avez rien saisi." << std::endl;
            continue; // On recommence la boucle
        }
 
        // 2. Vérifier si c'est numérique (accepte le signe - au début)
        bool est_numerique = true;
        size_t start_index = 0;
        if (saisie.at(0) == '-') {
            start_index = 1;
            if (saisie.size() == 1) est_numerique = false; // Juste "-" n'est pas un nombre
        }
 
        for (size_t i = start_index; i < saisie.size(); i++) {
            if (saisie[i] < '0' || saisie[i] > '9') {
                est_numerique = false;
                break;
            }
        }
 
        if (est_numerique) {
            try {
                return std::stoi(saisie);
            } catch (const std::out_of_range&) {
                std::cout << "Erreur : Le nombre est trop grand." << std::endl;
            }
        } else {
            std::cout << "Erreur : Entrez une valeur numérique valide." << std::endl;
        }
    }
}

Oui ça marche. Sinon tu as écrit saisie == '-' au lieu de saisie.at(0)=='-'.

La classe faillite n'existera pas dans le final. C'est juste pour tester.
voici mon faillite.hpp:

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
#include <string>
 
class faillite{
public:
  void cashflow_negatif();
  int saisie_int(std::string const &message);
  bool peut_continuer();
  int remb_banque();
  void saisie_pret_bancaire();
private:
  int salaire;
  int cashflow;
  int impots;
  int hypotheque;
  int remb_hypotheque;
  int pret_etudiant;
  int remb_etudiant;
  int pret_vehicule;
  int remb_vehicule;
  int carte_credit;
  int remb_carte;
  int autres_depenses;
  int pret_bancaire;
  int depenses_enfants;
  int caisse;
};

faillite.cpp:

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
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
#include <iostream>
#include <string>
#include "sortie_faillite.hpp"
 
void faillite::cashflow_negatif(){
  salaire=saisie_int("Salaire : ");
  cashflow=salaire;
  impots=saisie_int("Impots : ");
  cashflow-=impots;
  hypotheque=saisie_int("hypotheque : ");
  remb_hypotheque=saisie_int("Remboursement de l'hypothèque : ");
  cashflow-=remb_hypotheque;
  pret_etudiant=saisie_int("pret étudiant : ");
  remb_etudiant=saisie_int("Remboursement du prêt étudiant : ");
  cashflow-=remb_etudiant;
  pret_vehicule=saisie_int("pret véhicule : ");
  remb_vehicule=saisie_int("Remboursement du prêt véhicule : ");
  cashflow-=remb_vehicule;
  carte_credit=saisie_int("dettes sur la carte de crédit : ");
  remb_carte=saisie_int("Remboursement des dettes de la carte de crédit : ");
  cashflow-=remb_carte;
  autres_depenses=saisie_int("Autre dépenses : ");
  cashflow-=autres_depenses;
  saisie_pret_bancaire();
  cashflow-=remb_banque();
  depenses_enfants=saisie_int("Dépenses enfants : ");
  cashflow-=depenses_enfants;
  caisse=saisie_int("Caisse : ");
  std::cout<<"caisse "<<caisse<<'\n';
  std::cout<<"cashflow "<<cashflow<<'\n';
  std::cout<<"salaire "<<salaire<<'\n';
  std::cout<<"impôts "<<impots<<'\n';
  std::cout<<"hypothèque : "<<hypotheque<<'\n';;
  std::cout<<"Remboursement de l'hypothèque : "<<remb_hypotheque<<'\n';
  std::cout<<"pret étudiant"<<pret_etudiant<<'\n';
  std::cout<<"dette étudiant "<<remb_etudiant<<'\n';
  std::cout<<"pret véhicule"<<pret_vehicule<<'\n';
  std::cout<<"dette véhicule "<<remb_vehicule<<'\n';
  std::cout<<"prets carte de credit "<<carte_credit<<'\n';
  std::cout<<"dettes carte de credit "<<remb_carte<<'\n';
  std::cout<<"autres dépenses "<<autres_depenses<<'\n';
  std::cout<<"Pret_bancaire "<<pret_bancaire<<'\n';
  std::cout<<"dettte banque "<<remb_banque()<<'\n';
  std::cout<<"depenses pour les enfants "<<depenses_enfants<<std::endl;
 
 
  /*if(cashflow<0 && caisse<-cashflow)//si vous êtes en faillite
        if(peut_continuer())
      std::cout<<"Vous pouvez continuer."<<std::endl;
    else
    std::cout<<"Vous sortez du jeu."<<std::endl;*/
}
 
int faillite::saisie_int(std::string const &message) {
  std::string saisie;
  while (true) { // On boucle indéfiniment jusqu'à un return
    std::cout << message;
    if (!std::getline(std::cin, saisie)) {
      // Gestion d'erreur critique (ex: Ctrl+D / fin de flux)
      return 0; 
    }
 
    // 1. Vérifier si vide
    if (saisie.empty()) {
      std::cout << "Erreur : vous n'avez rien saisi." << std::endl;
      continue; // On recommence la boucle
    }
 
    // 2. Vérifier si c'est numérique (accepte le signe - au début)
    bool est_numerique = true;
    size_t start_index = 0;
    if (saisie.at(0) == '-') {
      start_index = 1;
      if (saisie.size() == 1) est_numerique = false; // Juste "-" n'est pas un nombre
    }
 
    for (size_t i = start_index; i < saisie.size(); i++) {
      if (saisie[i] < '0' || saisie[i] > '9') {
	est_numerique = false;
	break;
      }
    }
 
    if (est_numerique) {
      try {
	return std::stoi(saisie);
      } catch (const std::out_of_range&) {
	std::cout << "Erreur : Le nombre est trop grand." << std::endl;
      }
    } else {
      std::cout << "Erreur : Entrez une valeur numérique valide." << std::endl;
    }
  }
}
 
void faillite::saisie_pret_bancaire(){
  while(true){
    pret_bancaire=saisie_int("pret bancaire (multiple de 1000) : ");
    if(pret_bancaire % 1000 == 0)
      break;
  }
}
 
 
int faillite::remb_banque(){
  return pret_bancaire/10;
}

main.cpp

Code:

1
2
3
4
5
6
7
#include "sortie_faillite.hpp"
 
 
int main(){
  faillite banqueroute;
  banqueroute.cashflow_negatif();
}

30/11/2025, 18h30
fred1599

J'ai édité, bien vu...

Pour la suite je vous laisse faire, on sort du cadre de la demande initiale, n'hésitez pas à ouvrir un nouveau sujet si vous voulez parler de conception.