Discussion :

[dude666]

bonjour a tous,

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

voici son code :

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

[fearyourself]

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 :

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

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

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

Une meilleure solution (mais pas encore parfaite) :

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

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

[fearyourself]

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 :

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

sChaine = sDebChaine;

Jc

[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++ ?

[Emmanuel Delahaye]

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 :

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

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

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.

[Emmanuel Delahaye]

Une meilleure solution (mais pas encore parfaite) :

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