probleme saisi float

Version imprimable

bonjour a tous,

voila, je suis entrain de faire une fonction de saisi de float.

voici son code :

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
float readFloat()
{
   float temp;
   char *sChaine,*sDebChaine;
   Booleen a;
   sChaine			=	(char *)malloc(10*sizeof(char));
   do
   {
       fflush(stdin);
       fgets (sChaine,10, stdin);
       sDebChaine  = sChaine;
       a           = VRAI;
       while( (*sChaine!='\0') )
       {
           if( (*sChaine>'9') && (*sChaine!='.') )
               a = FAUX;
           sChaine++;
       }
       if( a==FAUX ) {
           printf("ERREUR, veuillez recommencer : ");
           sChaine = sDebChaine;
       }
   } while(a==FAUX);
    temp = strtof (sChaine,NULL);
    free(sDebChaine);
    return temp;
}

le seul probleme, c'est quelle me renvoit toujours 0, donc que la conversion au niveau de strtof se passe mal alors que la chaine saisit est un reel,
quelqu'un aurait une petite idée?

Quelques remarques :

- strtof n'est que disponible en C99, utilise strtod avec des doubles à la place...
- on ne cast pas le retour de malloc
- faire ta boucle interne montre que tu n'as pas compris ou ne sais pas utiliser strtof(d) correctement...
- on ne fait pas de fflush sur un flux entrant

Bien que je ne vois pas comment ce test va vraiment t'aider :

Code:

if( (*sChaine>'9') && (*sChaine!='.') )

Je suis sûr qu'on arriverait à faire planter ton programme...

Une meilleure solution (mais pas encore parfaite) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 double readDouble() { double temp; char sChaine[128],*sFinChaine; do { printf("Veuillez entrer un nombre : "); fgets (sChaine,sizeof(sChaine), stdin); temp = strtod(sChaine,&sFinChaine); } while((sChaine==sDebChaine)||((*sFinChaine!='\n')&&(*sFinChaine!='\0'))); return temp; }
Jc

29/09/2006, 15h27
violette327

Salut !

Ton programme ne pouvais pas marcher, vu que dans la boucle, tu lis tous les caractères de ta chaine schaine et tu fais "avancer" le pointeur : schaine++.
Donc apres la boucle, tu convertis dans temp une chaine qui vaut NULL, donc tu as temp=0.
Si vraiment tu veux garder ta méthode, il faudrait que tu copie ta chaine avant le while ( strcpy(sInit, sChaine) ) et à la sortie tu convertis sInit en float ( temp=strtof(sInit) ).

Maintenant, je suis d'accord avec fearyourself il y a plus simple pour faire une saisie de float...
29/09/2006, 15h47
fearyourself

Citation:

Envoyé par violette327

Salut !
Ton programme ne pouvais pas marcher, vu que dans la boucle, tu lis tous les caractères de ta chaine schaine et tu fais "avancer" le pointeur : schaine++.
Donc apres la boucle, tu convertis dans temp une chaine qui vaut NULL, donc tu as temp=0.
Si vraiment tu veux garder ta méthode, il faudrait que tu copie ta chaine avant le while ( strcpy(sInit, sChaine) ) et à la sortie tu convertis sInit en float ( temp=strtof(sInit) ).

Ou juste faire

Code:

sChaine = sDebChaine;

Jc
29/09/2006, 17h45
dude666

ok je viens de tester la methode de fearyourself, mais j'ai toujours 0 dans temp.

est ce que cela pourrais venir de mon compilo : j'utilise dev c++ ?
29/09/2006, 18h18
Emmanuel Delahaye

Citation:

Envoyé par fearyourself

Une meilleure solution (mais pas encore parfaite) :

Il faut aussi tester si errno est différent de ERANGE.

Citation:

Envoyé par dude666

ok je viens de tester la methode de fearyourself, mais j'ai toujours 0 dans temp.

Comment tu le sais ?

Je parie pour un bug d'affichage (printf()).

Ceci fonctionne :

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
#include <stdio.h>
#include <stdlib.h>
 
double readDouble (void)
{
   double temp;
   char sChaine[128], *sFinChaine;
   do
   {
      printf ("Veuillez entrer un nombre : ");
      fflush (stdout);
      fgets (sChaine, sizeof sChaine, stdin);
      temp = strtod (sChaine, &sFinChaine);
   }
   while ((sFinChaine == sChaine)
          || ((*sFinChaine != '\n') && (*sFinChaine != '\0')));
   return temp;
}
 
 
int main ()
{
   double n = readDouble ();
 
   printf ("n = %f\n", n);
   return 0;
}

Code:

1
2
3
4
5
6
7
8
 
Veuillez entrer un nombre : 12,34
Veuillez entrer un nombre : 12a
Veuillez entrer un nombre : a12
Veuillez entrer un nombre : 12.34
n = 12.340000
 
Press ENTER to continue.

29/09/2006, 22h07
dude666

ok ca marche merci