Effectuer un calcul sur un GtkEntry

Version imprimable

Bonjour à tous,

Donc je vous explique mon problème...
J'ai dans une fenetre un gtk_entry et un boutton..

L'utilisateur est senser entre un prix d'achat et lorsqu'il clique sur le bouton, il execute le calcule pour avoir le prix final, et va changer la valeur de mon gtk_entry en y affichant le resultat...

Seulement le problème est le suivant :
- cela fonctionne, mais il ne prend pas en compte les chiffre derrière la virgule ( si j'entre 9 ou 9.99 cela me donne le meme resultat)
- Secondo il me donne une erreur de segmentation dans la console...

Voici le code et dite moi se que vous en pensser :)
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 void Btn_Calcul_Prix() { const char *text = NULL; double Entre = 0, Calc1 = 0, Result = 0; text = gtk_entry_get_text(GTK_ENTRY(pEntryWPrix)); Entre = strtol (text, NULL, 10); Calc1 = Entre / 0.6; Result = Calc1 * 1.21; char *Retour; sprintf(Retour,"%0.2f",Result); gtk_entry_set_text(GTK_ENTRY(pEntryWPrix),Retour); }

06/01/2009, 20h30
gerald3d

Surement un problème de police. Pour confirmer essaies de mettre une virgule à la place du point.
06/01/2009, 20h39
dogua

Citation:

Envoyé par gerald3d

Surement un problème de police. Pour confirmer essaies de mettre une virgule à la place du point.

Sa fait la même choses :s
06/01/2009, 23h57
teuf13

Tu utilises mal sprintf (son utilisation est d'ailleurs à bannir autant que possible ;). Je t'invite à regarder du côté de g_strdup_printf (n'oublie pas de faire un g_free sur la chaîne de caractère renvoyée quand tu n'en as plus besoin).

strtol renvoie un entier, donc tu demandes explicitement au système de te fournir en entier, ça serait un bug s'il te donnait un nombre flottant ;) Regarde du côté de strtof/strtod plutôt.

En ce qui concerne les avis sur le code, je n'utiliserais qu'une seule variable flottante pour faire tous les calculs vus que les résultats intermédiaires n'ont pas l'air de t'être utiles.
07/01/2009, 12h19
dogua

Citation:

Envoyé par teuf13

Tu utilises mal sprintf (son utilisation est d'ailleurs à bannir autant que possible ;). Je t'invite à regarder du côté de g_strdup_printf (n'oublie pas de faire un g_free sur la chaîne de caractère renvoyée quand tu n'en as plus besoin).

strtol renvoie un entier, donc tu demandes explicitement au système de te fournir en entier, ça serait un bug s'il te donnait un nombre flottant ;) Regarde du côté de strtof/strtod plutôt.

Ok merci je vais regarder a sa...

Citation:

Envoyé par teuf13

En ce qui concerne les avis sur le code, je n'utiliserais qu'une seule variable flottante pour faire tous les calculs vus que les résultats intermédiaires n'ont pas l'air de t'être utiles.

Sinon jme doute, mais c'etait pour etre plus claire :) je remet tout en un calcul apres..
07/01/2009, 16h18
dogua

:?
Pourriez vous eventuellement me montrer dans mon vout de code comment je doit utiliser strtod ou strtof car je ne parvien pas a compiler en utilisant ceci.

Autre problème, j'ai essayer de compiler sur une autre machine... sa compile mais alors la sa calcul plus rien sa me donne une erreur de segmentation???

Je suis perdu :cry:
07/01/2009, 20h13
teuf13

Si t'as encore le sprintf, c'est normal les erreurs de segmentation... Pour strtof/strtod, sans voir les msgs d'erreur, je peux pas aider :)

Citation:

Envoyé par teuf13

Si t'as encore le sprintf, c'est normal les erreurs de segmentation..

Pourtant, avec le meme code sur mon portable sa tourne sauf que j'ai aucune precision... Mais sur les Workstation au boulot sa me met Erreur de segmentation... (peut etre la glib trop ancienne.. aller savoir)

Sinon rectification, Si j'utilise :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 void Btn_Calcul_Prix() { const char *text = NULL; double Entre = 0, Calc1 = 0, Result = 0; text = gtk_entry_get_text(GTK_ENTRY(pEntryWPrix)); Entre = atof (text);// je perd ma precision ici Calc1 = Entre / 0.6; Result = Calc1 * 1.21; char *Retour; sprintf(Retour,"%0.2f",Result); gtk_entry_set_text(GTK_ENTRY(pEntryWPrix),Retour); }
Sa fonctionne, mais il faut quand meme que je mette une virgule et pa un point, sinon sa passe pas... Existerait-il un moyen pour convertir le point en virgule ou de faire en sorte que cela passe aussi avec un point?

Et alors que pensser vous du faite que sa foire sur la vielle debian qu'il y a sur les Workstation au boulot?

07/01/2009, 22h59
teuf13

Bon, écoute, encore une fois, ton utilisation de sprintf est erronnée, tu corromps la mémoire de ton programme, ce qui conduit à des plantages aléatoires. Donc le comportement que tu obtiens est parfaitement normal! Je t'encourage une nouvelle fois à oublier ce sprintf et à utiliser g_strdup_printf à la place.
Pour ces histoires de . et de , tu peux utiliser g_ascii_strtod qui attend un . mais je te rappelle qu'en France on met une virgule pour séparer la partie entière de la partie décimale d'un nombre ;) Sur un système en anglais, strtod s'attendra à avoir un point.
08/01/2009, 08h03
dogua

Citation:

Envoyé par teuf13

mais je te rappelle qu'en France on met une virgule pour séparer la partie entière de la partie décimale d'un nombre ;) Sur un système en anglais, strtod s'attendra à avoir un point.

Je suis belge :p.. on fait jamais comme les autres :lol:

Ok ben pour le reste je vais voir avec se que tu ma dit ;-)
09/01/2009, 16h49
dogua

Bon ben avec

Code:

g_ascii_strtod

Il garde la precision avec le point... IL ne me reste plus a la limite de faire une espece de condition qui check si c'est une virgule ou un point qui a été entré, et le tour est joué :)

Je mettrai le code final ici pour les autres qui comme moi on eu des difficulté. Ainsi il pourons voir comment on peut procéder :)

Encore Merci

PS: Pour se qui est des workstation au boulot elles vont etres reinstaller en Ubuntu 8.10, avec les dernière version des lib, donc sa devra fonctionner ;-)
10/01/2009, 23h24
teuf13

Citation:

Envoyé par dogua

PS: Pour se qui est des workstation au boulot elles vont etres reinstaller en Ubuntu 8.10, avec les dernière version des lib, donc sa devra fonctionner ;-)

Si par là tu veux dire que la mise à jour va corriger les pbs de plantage de ton appli à cause du sprintf, ne t'étonne pas si ton programme se met à planter le jour où tu rajoutes une nouvelle fonction ou bien où tu réorganises un peu le code...