Problème avec strtod

Version imprimable

01/05/2007, 20h02
odsen.s

Problème avec strtod

Bonsoir,

Ca fait une heure que je me casse la tête sur un problème.

J'ai dans mon code une chaîne de caractères, que j'affiche, qui vaut "40.00".
Lorsque je veux convertir cette valeur dans un double grâce à strtod, la conversion n'est pas complète : la partie non convertie, que j'affiche, est ".00".

Avez-vous déjà rencontré ça ?
Quelles peuvent être les raisons pour que la conversion n'aille pas plus loin que la partie entière ?

Pas de problèmes particuliers comme ceci:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
 
int main (void)
{
   double a = 0.0;
   char * val = "40.00";
 
 
   a = strtod (val, NULL);
   printf ("%.02f\n", a);
 
 
   return EXIT_SUCCESS;
}

Montre ton code pour qu'on puisse voir ;)

01/05/2007, 21h22
josse95

Ne serait-ce pas un problème de setlocale ?

Quoique, je ne pense pas que strtod soit sensible au paramètrage local.

Voici le code de la fonction :

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
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
int lire_operation_fmt(FILE *fp_compte, size_t n_op, Operation *p_op)
{
   size_t i;
   int err = OK;
   char s_ligne[LONG_LIGNE] = {0};

   /* on se rend a la bonne ligne */

   fseek(fp_compte, 0, SEEK_SET);

   for(i = 0; i < n_op; i++)
   {
      if(fgets(s_ligne, sizeof s_ligne, fp_compte) == NULL)
      {
         err = OP_INEXISTANTE;
         break;
      }
   }

   if(err != OP_INEXISTANTE)
   {
      if(verif_fmt_ligne(s_ligne, sizeof s_ligne) == OK)
      {
         /* copie des valeurs */

         size_t j;

         for(i = j = 0; s_ligne[i] != ','; i++, j++)
         {
            p_op->s_date[j] = s_ligne[i];
         }

         p_op->s_date[j] = 0;
         i++;

         for(j = 0; s_ligne[i] != ','; i++, j++)
         {
            p_op->s_montant[j] = s_ligne[i];
         }

         p_op->s_montant[j] = 0;
         i++;

         for(j = 0; s_ligne[i] != ','; i++, j++)
         {
            p_op->s_motif[j] = s_ligne[i];
         }

         p_op->s_motif[j] = 0;
         i++;

         for(j = 0; s_ligne[i] != '\n'; i++, j++)
         {
            p_op->s_solde[j] = s_ligne[i];
         }

         p_op->s_solde[j] = 0;

         /* conversion en double pour montant et solde */
        

         char *p_conversion = NULL;
         errno = 0;
 
         /* l'erreur se produit ici */
         p_op->montant = strtod(p_op->s_montant, &p_conversion);

         if(p_conversion != NULL)
         {
             if(*p_conversion == 0)
             {
                 /* valeur trop grande */

                 if(errno == ERANGE)
                 {
                     err = ERR_VAL;
                 }

                 else
                 {
                     p_conversion = NULL;
                     errno = 0;

                     p_op->solde = strtod(p_op->s_solde, &p_conversion);

                     if(p_conversion != NULL)
                     {
                         if(*p_conversion == 0)
                         {
                             /* valeur trop grande */

                             if(errno == ERANGE)
                             {
                                 err = ERR_VAL;
                             }
                         }

                         else
                         {
                             err = ERR_CONV;
                         }
                     }
                 }
             }

             else
             {
                 err = ERR_CONV;
             }
         }
      }

      else
      {
         err = FORMAT_KO;
      }
   }

   return err;
}

int verif_fmt_ligne(const char* s_ligne, size_t t)
{
   /* format correct <=> (NB_VIRGULES) virgules dans la ligne */
   
   int nb_virgules = 0;
   size_t i;

   for(i = 0; i < t; i++)
   {
      if(s_ligne[i] == '\0')
      {
         break;
      }

      else if(s_ligne[i] == ',')
      {
         nb_virgules++;
      }
   }

   return nb_virgules - NB_VIRGULES;
}

Avec :

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
#define LONG_LIGNE 128
#define LONG_DATE 16
#define LONG_MONTANT 32
#define LONG_MOTIF 32
#define LONG_SOLDE 32
 
#define NB_VIRGULES 3
 
enum
{
   OK,
   FORMAT_KO,
   OP_INEXISTANTE,
   ERR_LONG,
   ERR_CONV,
   ERR_W,
   ERR_VAL
};
 
typedef struct Operation
{
   char s_date[LONG_DATE];
   char s_montant[LONG_MONTANT];
   double montant;
   char s_motif[LONG_MOTIF];
   char s_solde[LONG_SOLDE];
   double solde;
} Operation;

Le problème se pose à partir de la première conversion en double. (voir ligne en gras).
Comme le montre bien le code, la chaîne "op->s_montant" est lue à partir d'un fichier.
Elle est correcte, si je l'affiche, j'ai bien "40.00" comme dans mon fichier.

A noter quelque chose de très bizarre :
Tout marchait bien, sans erreur, quand j'utilisais cette fonction dans mon programme de gestion de budgets en console.
Comme tu as du le remarquer, Franck, je m'initie depuis quelques temps à GTK+.
Le problème de cette fonction est apparue lorsque je l'ai utilisée pour mon projet de refaire ce programme de gestion de budgets avec GTK+.
Pourtant :
- La fonction est exactement la même
- Le fichier lu par la fonction est le même

D'où mon incompréhension...

Voilà :)

Citation:

Ne serait-ce pas un problème de setlocale ?

Salut,
Qu'est-ce que "setlocale" ?
:)

01/05/2007, 21h27
josse95

setlocale (Windows) permet de spécifier le séparateur de décimale (. ou ,).
Mais il affecte printf, pas strtod à ma connaissance.
01/05/2007, 21h33
Emmanuel Delahaye
Citation:
Envoyé par Franck.H

Pas de problèmes particuliers comme ceci:

Code:

1 2 3 4 5 6 7 8 9 { double a = 0.0; char * val = "40.00"; a = strtod (val, NULL); printf ("%.02f\n", a); }
Oui, mais que donne

Code:

char * val = "40.50";

Tu as cassé le thermomètre (suppression du 2ème paramètre : NULL), ça ne signifie pas que le patient est guéri
01/05/2007, 21h53
Trap D

Citation:

Envoyé par josse95

setlocale (Windows) permet de spécifier le séparateur de décimale (. ou ,).
Mais il affecte printf, pas strtod à ma connaissance.

Il affecte aussi la lecture des nombres donc strtod.
01/05/2007, 22h02
josse95

Essaie ceci alors (si tu es sous Windows):

Code:

setlocale(LC_NUMERIC,".");

euh... je signale juste que ceci :
Code:

1 2 3 4 5 6 7 8 9 for(i = 0; i < n_op; i++) { if(fgets(s_ligne, sizeof s_ligne, fp_compte) == NULL) { err = OP_INEXISTANTE; break; } }
ne stockera que la dernière ligne du fichier en mémoire.....

8O

[EDIT]

Pardon ... que la première ligne...

[/EDIT]

02/05/2007, 10h19
Franck.H

Citation:

Envoyé par Emmanuel Delahaye

Oui, mais que donne

Code:

char * val = "40.50";

Tu as cassé le thermomètre (suppression du 2ème paramètre : NULL), ça ne signifie pas que le patient est guéri

Justement, chez moi je n'ai pas le moindre problème, même avec cette chaîne, j'ai quand même testé avant de poster ;) Sinon à part éventuellement un problème de locale, qu'il me semble avoir déjà vu sur ce forum y'a 1 ou 2 mois je vois pas d'erreur sur la conversion :?
02/05/2007, 18h38
odsen.s
Bonsoir à tous,

Citation:

Envoyé par josse95

Essaie ceci alors (si tu es sous Windows):

Code:

setlocale(LC_NUMERIC,".");

Quel fichier d'en-tête faut-il intégrer s'il te plaît ?
Mais je me demande, si c'était un problème de locale, pourquoi ma fonction fonctionnerait dans un projet et pas dans l'autre ?
Citation:
Envoyé par souviron34

euh... je signale juste que ceci :

Code:

1 2 3 4 5 6 7 8 9 for(i = 0; i < n_op; i++) { if(fgets(s_ligne, sizeof s_ligne, fp_compte) == NULL) { err = OP_INEXISTANTE; break; } }

ne stockera que la dernière ligne du fichier en mémoire.....

8O

[EDIT]

Pardon ... que la première ligne...

[/EDIT]
Euh, pourquoi ? A chaque appel de fgets, c'est bien la ligne suivante qui sera lue, non ?
(Exemple : n_op vaut 3, il y aura 3 appels de fgets, c'est la troisième ligne qui sera lue).
La fonction ne lit qu'une ligne à la fois, la ligne n_op.

J'ai effectué aussi les mêmes tests que Franck.H, avec la valeur entière et la valeur décimale séparées par un point, et la conversion se fait en entier. Le problème intervient uniquement dans ce projet GTK+, avec la même fonction qui fonctionnait bien auparavant et le même fichier à lire.
02/05/2007, 18h47
Jean-Marc.Bourguet

Citation:

Envoyé par odsen.s

Bonsoir à tous,
Quel fichier d'en-tête faut-il intégrer s'il te plaît ?
Mais je me demande, si c'était un problème de locale, pourquoi ma fonction fonctionnerait dans un projet et pas dans l'autre ?

Parce que l'utilisation de ta fonction suppose que ce qui determine la partie decimale est un point (ce qui est le cas dans la locale par defaut mais pas dans une locale francaise, c'est une virgule). Je suppose que gtk change la locale...
02/05/2007, 19h05
odsen.s

Citation:

Envoyé par Jean-Marc.Bourguet

Parce que l'utilisation de ta fonction suppose que ce qui determine la partie decimale est un point (ce qui est le cas dans la locale par defaut mais pas dans une locale francaise, c'est une virgule). Je suppose que gtk change la locale...

Ok.
Je vais donc essayer de changer la locale.
Y a-t-il une manière portable de le faire ?
02/05/2007, 19h12
josse95

Pour le setlocale:

Code:

#include <locale.h>

setlocale est compatible POSIX, Win32 et ANSI C
02/05/2007, 19h33
odsen.s
Merci. :)

J'ai essayé :
Code:

1 2 3 4 5 6 7 #include <locale.h> int main(void) { /* .... */ setlocale(LC_NUMERIC,"."); /* ..... */ }
Mais le problème persiste : .00 n'est pas converti.

EDIT : Pour info, que je fasse la setlocale ou non, si j'affiche mon double, j'ai "40,0000" avec une virgule.
02/05/2007, 20h23
Jean-Marc.Bourguet
Citation:
Envoyé par odsen.s

Merci. :)

J'ai essayé :

Code:

1 2 3 4 5 6 7 #include <locale.h> int main(void) { /* .... */ setlocale(LC_NUMERIC,"."); /* ..... */ }

Mais le problème persiste : .00 n'est pas converti.

EDIT : Pour info, que je fasse la setlocale ou non, si j'affiche mon double, j'ai "40,0000" avec une virgule.
Il faut t'assurer que tu changes bien la locale après gtk. Et que tu utilises un nom valide. Essaye avec
Code:

1 2 3 4 #include <locale.h> ... setlocale(LC_NUMERIC,"C"); ...
juste avant ta conversion qui foire, comme ça tu es sûr de te trouver entre le changement de gtk et la conversion et d'utiliser un nom de locale valide ("C" est la locale par defaut, et la seule qui soit définie par la norme).
03/05/2007, 14h06
odsen.s

Bonjour,

J'ai copié :

Code:

setlocale(LC_NUMERIC,"C");

juste après l'initialisation de GTK+, et ça fonctionne désormais :)

C'était donc bien un problème de locale.

Merci à tous ceux qui ont pris de leur temps pour m'aider.
:)