Problème avec strtod

**odsen.s** · 01/05/2007, 20h02

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 ?

**Franck.H** · 01/05/2007, 21h05

Pas de problèmes particuliers comme ceci:

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
#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

**josse95** · 01/05/2007, 21h22

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

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

**odsen.s** · 01/05/2007, 21h23

Voici le code de la fonction :

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
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 :

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
#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à

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

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

**josse95** · 01/05/2007, 21h27

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

**Emmanuel Delahaye** · 01/05/2007, 21h33

Envoyé par Franck.H

Pas de problèmes particuliers comme ceci:

Code :

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

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 :

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

   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

**Trap D** · 01/05/2007, 21h53

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.

**josse95** · 01/05/2007, 22h02

Essaie ceci alors (si tu es sous Windows):

Code :

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

setlocale(LC_NUMERIC,".");

**souviron34** · 02/05/2007, 01h38

euh... je signale juste que ceci :

Code :

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

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.....

[EDIT]

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

[/EDIT]

**Franck.H** · 02/05/2007, 10h19

Envoyé par Emmanuel Delahaye

Oui, mais que donne

Code :

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

   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

**odsen.s** · 02/05/2007, 18h38

Bonsoir à tous,

Envoyé par josse95

Essaie ceci alors (si tu es sous Windows):

Code :

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

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 ?

Envoyé par souviron34

euh... je signale juste que ceci :

Code :

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

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.....

[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.

**Jean-Marc.Bourguet** · 02/05/2007, 18h47

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...

**odsen.s** · 02/05/2007, 19h05

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 ?

**josse95** · 02/05/2007, 19h12

Pour le setlocale:

Code :

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

#include <locale.h>

setlocale est compatible POSIX, Win32 et ANSI C

**odsen.s** · 02/05/2007, 19h33

Merci.

J'ai essayé :

Code :

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

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.

**Jean-Marc.Bourguet** · 02/05/2007, 20h23

Envoyé par odsen.s

Merci.

J'ai essayé :

Code :

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

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 :

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

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).