Discussion :

[astrofan]

Bonjour, j'ai un petit problème...

Voilà mon code source, et mon problème est que mon programme ne récupère pas correctement le contenu d'une variable string en float...

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
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
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
#include <iostream>
#include <cmath>
#include <string>
#include <fstream>
#include <sstream>
#include <windows.h>
 
using namespace std;
 
string LireFichier(void)
{
 string cours;
    char const*nom = "cours.scrola";
    ifstream txt(nom, ios::in);
    if(txt)
    {
        getline(txt, cours);
    }
    txt.close();
    return cours;
}
 
int main()
{
    string cours=LireFichier();
 std::stringstream ss;
 ss<<cours;
 double cours_dol;
 cours=cours_dol;
 debut:
    system("cls");
    int choix;
    int cho;
    cout<<"Bonjour et bienvenue dans l'utilitaire de conversion de devises.";
    cout<<"\n\nVoulez-vous mettre ";
    printf("%c", 133);
    cout<<" jour les cours du dollar?\nOu voulez-vous sp";
    printf("%c", 130);
    cout<<"cifier vous m";
    printf("%c", 136);
    cout<<"me le cours actuel?\n\n   1.Mettre ";
    printf("%c", 133);
    cout<<" jour.\n   2.Sp";
    printf("%c", 130);
    cout<<"cifier le cours.\n   0.Continuer sans mettre ";
    printf("%c", 133);
    cout<<" jour.\n\n\n\nVotre choix : ";
    cin>>cho;
    switch(cho)
    {
        case 1:
        {
            system("cls");
   cout<<"Veuillez patienter pendant que le programme\nt";
   printf("%c", 130);
   cout<<"l";
   printf("%c", 130);
   cout<<"charge tous les cours actuels...\n\n   10 secondes restantes...";
   Sleep(750);
   system("cls");
   cout<<"Veuillez patienter pendant que le programme\nt";
   printf("%c", 130);
   cout<<"l";
   printf("%c", 130);
   cout<<"charge tous les cours actuels...\n\n   9 secondes restantes...";
   Sleep(750);
   system("cls");
   cout<<"Veuillez patienter pendant que le programme\nt";
   printf("%c", 130);
   cout<<"l";
   printf("%c", 130);
   cout<<"charge tous les cours actuels...\n\n   8 secondes restantes...";
   Sleep(750);
   system("cls");
   cout<<"Veuillez patienter pendant que le programme\nt";
   printf("%c", 130);
   cout<<"l";
   printf("%c", 130);
   cout<<"charge tous les cours actuels...\n\n   7 secondes restantes...";
   Sleep(750);
   system("cls");
   cout<<"Veuillez patienter pendant que le programme\nt";
   printf("%c", 130);
   cout<<"l";
   printf("%c", 130);
   cout<<"charge tous les cours actuels...\n\n   6 secondes restantes...";
   Sleep(750);
   system("cls");
   cout<<"Veuillez patienter pendant que le programme\nt";
   printf("%c", 130);
   cout<<"l";
   printf("%c", 130);
   cout<<"charge tous les cours actuels...\n\n   5 secondes restantes...";
   Sleep(750);
   system("cls");
   cout<<"Veuillez patienter pendant que le programme\nt";
   printf("%c", 130);
   cout<<"l";
   printf("%c", 130);
   cout<<"charge tous les cours actuels...\n\n   4 secondes restantes...";
   Sleep(750);
   system("cls");
   cout<<"Veuillez patienter pendant que le programme\nt";
   printf("%c", 130);
   cout<<"l";
   printf("%c", 130);
   cout<<"charge tous les cours actuels...\n\n   3 secondes restantes...";
   Sleep(750);
   system("cls");
   cout<<"Veuillez patienter pendant que le programme\nt";
   printf("%c", 130);
   cout<<"l";
   printf("%c", 130);
   cout<<"charge tous les cours actuels...\n\n   2 secondes restantes...";
   Sleep(750);
   system("cls");
   cout<<"Veuillez patienter pendant que le programme\nt";
   printf("%c", 130);
   cout<<"l";
   printf("%c", 130);
   cout<<"charge tous les cours actuels...\n\n   1 secondes restantes...";
   Sleep(750);
   system("cls");
   cout<<"Veuillez patienter pendant que le programme\nt";
   printf("%c", 130);
   cout<<"l";
   printf("%c", 130);
   cout<<"charge tous les cours actuels...\n\n   0 seconde restante...";
   Sleep(750);
   system("scroladl <a href="http://www.scrola.com/cours/cours.scrola" target="_blank">http://www.scrola.com/cours/cours.scrola</a>");
            system("cls");
            break;
        }
        case 2:
        {
            system("cls");
            cout<<"Quelle est la valeur du cours du dollar?  ";
            cin>>cours_dol;
            break;
        }
        case 0:
        {
            break;
        }
        default:
        {
            system("cls");
            cout<<"\n\n\n\n\nVous avez entr";
            printf("%c", 130);
            cout<<" un mauvais mot-cl";
            printf("%c", 130);
            cout<<"...";
            system("pause>nul");
            goto debut;
            break;
        }
    }
    double euro=0, dol=0;
    int choix2;
    cout<<"\n\n\nAppuyez sur une touche pour continuer...";
    system("pause>nul");
    convert:
    system("cls");
    cout<<"\n\n\n   1.Euro>>Dollar\n   2.Dollar>>Euro\n\n\nQuel est votre choix? ";
    cin>>choix2;
    switch(choix2)
    {
        case 1:
        {
            system("cls");
            cout<<"\n\nLe cours actuel du dollar est : "<<cours_dol<<".\n\nQuel est la somme en euros que vous voulez\nconvertir? ";
            cin>>euro;
            dol = euro * cours_dol;
            cout<<"\n\nAu cours actuel ( "<<cours_dol<<" ), "<<euro<<" euros valent "<<dol<<" dollars.\n";
            system("pause>nul");
            goto fin;
        }
        case 2:
        {
            system("cls");
            cout<<"\n\nLe cours actuel du dollar est : "<<cours_dol<<".\n\nQuel est la somme en dollars que vous voulez\nconvertir? ";
            cin>>dol;
            euro = dol / cours_dol;
            cout<<"\n\nAu cours actuel ( "<<cours_dol<<" ), "<<dol<<" dollars valent "<<euro<<" euros.\n";
            system("pause>nul");
            goto fin;
        }
        default:
        {
            system("cls");
            cout<<"\n\nUne erreur est survenue... C'est d";
            printf("%c", 150);
            cout<<" au fait que vous ayiez entr";
            printf("%c", 130);
            cout<<" un mauvais mot-cl";
            printf("%c", 130);
            cout<<"...\nRecommencez.";
            system("pause>nul");
            goto debut;
        }
    }
    int choix3;
    fin:
 cout<<"\n\n\n   Tapez 1 pour recommencer le programme.\n\n   Tapez 0 pour quitter.\n\n     Votre choix : ";
 cin>>choix3;
 switch(choix3)
 {
  case 1:
  {
   goto debut;
   break;
  }
  case 0:
  {
   break;
  }
  default:
  {
   cout<<"\n\n                ERREUR !\nVous avez entr";
   printf("%c", 130);
   cout<<" un mauvais mot-cl";
   printf("%c", 130);
   cout<<"...\n Recommencez...";
   system("pause");
   system("cls");
   goto fin;
   break;
  }
 }
    end:
    return 0;
}

Par exemple, le problème récupère dans un fichier nommé "cours.scrola", mais le récupère mal... le contenu de "cours.scrola", c'est 123, et il me récupère 4.4864546x10(puissance -43)

Y'a peut etre un petit problème...sniff

[kinji1]

Si tu veux mettre ce qu'il y a dans ton fichier dans cours_dol, c'est plutôt :

Code :

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

1
2
3
4
5
6
string cours = LireFichier();
std::stringstream ss;
ss << cours;
double cours_dol;
ss >> cours_dol;

[koala01]

Salut,

Ton code appelle pas mal de remarques:

• Un ifstream est d'office un fichier en lecture. Il ne sert donc à rien d'indiquer ios::in en deuxième paramètre
• Quand tu crée une variable de type ifstream, le fichier est automatiquement fermé lorsque la variable est détruite... ce qui arrive d'office lorsque tu quitte la portée dans laquelle la variable est déclarée. Il n'est donc a priori pas nécessaire de fermer explicitement ton fichier ici
• La fonction LireFichier, à moins qu'elle ne fasse partie des spécifications de l'exercice, ne présente qu'un intérêt relativement limité, au vu de la petitesse du code... Par contre, il pourrait vraiment sembler intéressant de créer une fonction qui s'occupe de l'affichage du menu et de la récupération du choix de l'utilisateur
• Si tu sais pertinemment que ton fichier va contenir un réel, pourquoi ne pas lire directement un réel dans le fichier Cela te facilitera grandement la vie
• Pense à utiliser les boucles, cela pourra t'éviter pas mal de lignes de codes (je penses par exemple à ton "compte à rebours")
• Même quand tu sais que tu aura besoin d'une chaine de caractères "C Style" (tableau de caractères terminé par un '\0'), il est fortement conseiller d'utiliser, malgré tout, une chaine de type std::string. Tu pourras récupérer le tableau de caractères en utilisant la méthode c_str() sur ta variable
• printf est une instruction C, il est fortement conseillé de lui préférer l'utilisation des flux standard (cin/cout)... C'est beaucoup plus facile à gérer, et il y a même possibilité de la formater
• Méfie toi de l'affichage des caractères de valeur supérieure à 127... Leur représentation graphique dépendra fortement de la table de caractères utilisée (chez moi, le caractères 130 est le "é"... était-ce ce que tu veux :question)
• La solution de "facilité" est de tout simplement écrire sans accents... une autre solution pouvant consister en l'utilisation des flux "unicode" (wcout et autres similaires)
• Evite tant que possible l'utilisation de l'instruction system("Une commande")... c'est clairement un PAC qui t'emmène droit vers une portabilité excécrable
• Préfère la programmation structurée, avec de belles boucles - que ce soit des boucles "Tant que", des boucles "Jusque" ou des boucles itératives - à l'utilisation horrible pour la lecture de goto (penses qu'un code est beaucoup plus souvent lu qu'il n'est écrit )
  
• lorsque tu utilise le test à choix multiples (switch... case), il est inutile d'entourer le bloc des case avec des accolades

Je pourrais te fournir un exemple de code corrigé, mais, comme j'ai un peu la flegme pour l'instant, je vais te laisser le temps d'assimiler les remarques qui précèdent pour les mettre en pratique toi même

Si vraiment, tu as un problème, je repasserai en mode "gentil codeur"

[astrofan]

Code :

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

ss >> cours_dol;

ne marche pas à la compilation.

Si tu sais pertinemment que ton fichier va contenir un réel, pourquoi ne pas lire directement un réel dans le fichier Cela te facilitera grandement la vie

Je veux bien, mais comment?

Méfie toi de l'affichage des caractères de valeur supérieure à 127... Leur représentation graphique dépendra fortement de la table de caractères utilisée (chez moi, le caractères 130 est le "é"... était-ce ce que tu veux :question)

Oui c'est ce que je veux^^

Sinon, pour toutes vos remarques, merci^^ J'essaierai d'y arriver...

[kinji1]

Code :

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

ss >> cours_dol;

ne marche pas à la compilation.

Je peux connaître le message d'erreur ? Parce que je trouve pas ça normal.

[koala01]

Code :

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

ss >> cours_dol;

ne marche pas à la compilation.

pourtant cela devrait avec un code du genre de

Code :

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

1
2
3
4
5
6
 
std::string str="0.123";
std::stringstream ss;
ss<<str;
double cours_dol;
ss>>cours_dol;

Le tout, c'est qu'il faut, penser à inclure les fichier <string> et <sstream>

Si tu sais pertinemment que ton fichier va contenir un réel, pourquoi ne pas lire directement un réel dans le fichier Cela te facilitera grandement la vie

Je veux bien, mais comment?

Ben, tout simplement, en déclarant la variable qui sert à la lecture et le type de retour de la fonction comme étant du type que tu veux lire (un réel en l'occurence )

Méfie toi de l'affichage des caractères de valeur supérieure à 127... Leur représentation graphique dépendra fortement de la table de caractères utilisée (chez moi, le caractères 130 est le "é"... était-ce ce que tu veux )

Oui c'est ce que je veux^^

Ben, tu peux très bien envisager un code plus "C++" sous la forme de

Code :

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

1
2
3
 
cout<<"Veuillez patienter pendant que le programme"<<endl
    <<"t"<<(char) 130)<<"l"<<(char) 130<<"charge les fichiers"<<endl

Sinon, pour toutes vos remarques, merci^^ J'essaierai d'y arriver...

Fais mieux qu'essayer: met les en pratiques

[astrofan]

Merci Koala01