Discussion :

[mamelouk]

Bonjour,

Est ce quelqu'un sait pourquoi lorsque j'utilise un stringstream pour convertir un string en double (ou int), ca marche bien la première fois, mais ensuite ca foire ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
string strX = "262";
string strY = "285";
double x, y;
 
ss << strX;
ss >> x;
 
ss << strY;
ss >> y;
 
cout << x << endl;
cout << y << endl;

affiche :

Code :

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

1
2
262
1.98281e-309

La lecture n'est pas destructrice du contenu ? Il faut appeler clear() ou qqchose dans le genre à chaque fois ?

Apparement oui. Ce monologue m'aura été très utile

[Laurent Gomila]

Attention, clear ne fait que remettre à zéro les flags d'erreur du flux (qui sont mis car à chaque extraction tu en atteint la fin). Pour vider le flux, c'est ss.str("").

Moi je ferais plutôt ça :

Code :

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

1
2
3
4
5
ss.str(strX);
ss >> x;
 
ss.str(strY);
ss >> y;

Par contre je ne sais pas si l'appel à str() replace aussi le curseur de lecture au début.

[mamelouk]

ok, mais ca marche pas quand le fais comme ca, tandis que quand je fais clear ca marche

[Laurent Gomila]

Ok, donc l'appel à str() ne replace pas le curseur de lecteur au début.

[Alp]

Dans ce cas il suffit juste de faire :

Code :

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

1
2
3
4
5
6
7
ss << strX;
ss >> x;
 
ss.str("");
 
ss << strY;
ss >> y;

je pense.

[mamelouk]

mmmm
moi je pense que ca marche pas
j'avais essayé un truc dans le genre.
pour l'instant clear() suffit pour ce que je fais

[Alp]

Tu as essayé ?
Il me semble que ça marche, mais sinon :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
// ton code qui est avant
// puis :
{
stringstream ss;
ss << strX;
ss >> x;
}
{
stringstream ss;
ss << strY;
ss >> x;
}

Tu peux même définir une macro/fonction inline pour que ce soit plus court... Du genre READ_FROM_SSTREAM(strY, y) ...

[mamelouk]

ben en fait j'avais essayé ss.str() avec le string suivante et ca marche pas.
après y'a la solution de recreer un stringstream à chaque fois comme tu dis...

j'ai finalement opté pour ca, j'appelle une fonction

int convert(string&)

qui declare le stringstream (il est donc détruit à chaque fin d'appel)

moi je dis quand meme que ca craint que ce soit aussi compliqué de faire des trucs aussi simple

[Alp]

Il ne faut pas faire ss.str() mais ss.str("") (très important!).

[mamelouk]

ben en fait j'avais essayé ss.str() avec le string suivante

ss.str(strY) évidement

ss.str() ne compile pas...

[Alp]

Non mais je veux dire il faut faire :

Code :

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

1
2
3
4
5
6
ss << strX;
ss >> x;

ss.str("");
ss << strY;
ss >> y;

Il faut bien des "" et non pas strY ou quoi que ce soit d'autre!

[mamelouk]

bon, pour te faire plaisir j'ai essayé :

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
#include <sstream>
#include <iostream>
 
using namespace std;
 
int main(void) {
    string strX("240"), strY("300");
    stringstream ss;
    int x, y;
 
    ss << strX;
    ss >> x;
    cout << x << endl;
 
    ss.str("");
    ss << strY;
    ss >> y;
    cout << y << endl;
}

Code :

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

1
2
240
-1073876376

j'ai remplacé str("") par clear() et ca marche

[Alp]

Il me semblait que ss.str("") aurait fait le nécessaire ...
Quoiqu'il en soit, le problème a été résolu

[mamelouk]

merci de ton aide

[koala01]

Salut,

Ceci dit, il y a facilement moyen de travailler autrement...

Pour l'exemple donné, je verrais bien deux solutions:

La première (et celle qui trouve plus grâce à mes yeux) et de placer toutes les valeurs dans le stringstream avant de commencer à le récupérer:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
string str1="256";
string str2="300";
 
stringstream ss;
ss<<str1<<" "<<str2;
int a;
int b;
ss>>a>>b;
cout<<a<<endl;
cout<<b<<endl;

La deuxième solution est, tout simplement, de travailler sur des stringstream qui sont dans des portées différentes

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
 
void fonct()
{
    string str1="256";
    string str2="300";
    int a;
    int b;
    {
        stringstream ss;
        ss<<str1;
        ss>>a;
    }
    /* comme on quitte la portée dans laquelle est déclaré ss, on peut 
     * le redéclarer et il sera remis à 0
     */
    {
        stringstream ss;
        ss<<str2;
        ss>>b;
    }
    cout<<a<<endl<<b<<endl;
}

[Matthieu Brucher]

Encore plus simple, boost::lexical_cast<int>(maChaine)

[mamelouk]

ah oui c'est vrai j'avais oublié la boost

[koala01]

Encore plus simple, boost::lexical_cast<int>(maChaine)

La question étant: est-ce réellement plus simple

Entre une dépendance, un espace de nomage et une instruction supplémentaire pour en arriver, en définitive au même résultat...

Je n'émet pas d'avis sur le sujet, mais je me pose réellement la question (dans ce cas bien précis du moins)

[mamelouk]

La question étant: est-ce réellement plus simple

Entre une dépendance, un espace de nomage et une instruction supplémentaire pour en arriver, en définitive au même résultat...

Je n'émet pas d'avis sur le sujet, mais je me pose réellement la question (dans ce cas bien précis du moins)

seulement une dépendance et une instruction, car ici l'espace de nommage n'est pas déclaré mais directement utilisé. On gagne plusieurs lignes de code, on gagne en lisibilité, et à mon avis on gagne surement aussi en performances

[Matthieu Brucher]

et à mon avis on gagne surement aussi en performances

En fait pas trop parce que lexical_cast utilise les stringstream.