tentative conversion string vers float

**firemax** · 21/08/2007, 02h03

Bonjour

Mon but est plutot simple , en faite je voudrais coder (ou recoder vu que je ne sais pas si elle existe) une fonction qui recupere un float a partir d'un pointeur qui pointe vers le premier caractere de la chaine mais ce n'est pas un point mais une virgule .

Exemple :
char *p;
p = "57,89";

sortie :
float f = 57.89;

J'ai rajouter des printf("OK"); pour tenter de trouver l'erreur , n'en tenez pas trop compte

Question :
1 - Ca segfault lors de l'apelle de my_getnbr , pourquoi ?
2 - Lorsque je code ca sans la fonction my_getnbr (ca me rajoute des lignes inutiles) la fonction convertofloat se termine pendant le premier while apres avoir convertis correctement la partie entiere (quand p[count] == ',' d'apres moi) , pourquoi ? Comment resoudre le probleme ?

Voila le code (attention ca pique au yeux ^^)

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
#include <stdio.h>
 
float converttofloat(char *p);
int  my_getnbr(char p);
 
int main ()
{
	char *pointeur;
 
	pointeur = "47,87";
	printf("%s\n",pointeur);
	printf ("%f\n",converttofloat(pointeur));
	return 0;
}
 
int my_getnbr(char p)
{
	if( (p >= '0') && (p <= '9') )
		return ((int)p - 48);
	else
		return (-1);
}
 
float converttofloat(char *p)
{
	int i=0 ; 
	int f=0 ; 
	int count=0;
	int countdec=0;
	float result = 0;
 
	printf("OK\n");
	/*Recuperation de la partie entiere*/
	while((p[count] != ',') || (p[count] != '\0') )
	{
		if(p[count] != ',')
		{ i = my_getnbr(p[count]) ; i *= 10; }
		count++;	
	}
 
	count++;
	printf("OK\n");	
 
	/*Recuperation de la partie decimale*/
	while(p[count] != '\0')
	{
		if(p[count] != ',')
		{ f = my_getnbr(p[count]) ; f *= 10; }
		count++;
		countdec++;
	}
 
	printf("OK\n");
 
	result =  f;
	while(countdec != 0)
	{
		result /= 10;
		countdec--;
	}
	result += i;
 
	return result;
}

Voila desoler si je me suis mal exprimer mais please help

Merci d'avance

**bi_turbo** · 21/08/2007, 02h12

premiere remarque

le prototypage float converttofloat(char *p);

printf ("%f\n",converttofloat(pointeur));

**bi_turbo** · 21/08/2007, 02h20

P est de type char

if( (p >= '0') && (p <= '9') ) tu compare p avec le caractere '0' ,'9' ou leur code ascii...
mon avi est que certainement tu veux que p €[0,9] pour les entiers.

**Emmanuel Delahaye** · 21/08/2007, 08h08

Envoyé par firemax

Mon but est plutot simple , en faite je voudrais coder (ou recoder vu que je ne sais pas si elle existe)<...>

strtod().

gl · 21/08/2007, 08h28

Envoyé par firemax

return ((int)p - 48);

pourquoi -48 ? Qu'est ce qui ne va pas avec -'0' ?

Envoyé par firemax

while((p[count] != ',') || (p[count] != '\0') )

La condition est toujours vraie, ce n'est pas || mais && qu'il faut utiliser.

Envoyé par firemax

{ i = my_getnbr(p[count]) ; i *= 10; }

Ici tu n'as que le dernier chiffre (puisque tu n'ajoutes pas les chiffres precedement convertis. En outre les unites deviennes des dizaines. Il ne faut pas multiplier ce que tu viens de convertir mais ce que tu avais deja converti avant de l'ajouter au chiffre courant.

Envoyé par firemax

{ f = my_getnbr(p[count]) ; f *= 10; }

Meme remarque

Envoyé par firemax

Code :

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

1
2
3
4
5
6
7
result =  f;
	while(countdec != 0)
	{
		result /= 10;
		countdec--;
	}
	result += i;

Pourquoi ne pas directement travaille en decimal ?

**firemax** · 21/08/2007, 11h35

Envoyé par Emmanuel Delahaye

strtod().

Merci

Mais vais la recoder quand meme (pour le plaisir ^^)

Envoyé par gl

pourquoi -48 ? Qu'est ce qui ne va pas avec -'0' ?

Bah on recupere sous forme de char donc '0' = 48 , '1' = 49 ... Non ?
Du coup pour le recuperer sous sa forme decimale on lui retire 48 , je me trompe ?

Envoyé par gl

La condition est toujours vraie, ce n'est pas || mais && qu'il faut utiliser.

Envoyé par gl

Ici tu n'as que le dernier chiffre (puisque tu n'ajoutes pas les chiffres precedement convertis. En outre les unites deviennes des dizaines. Il ne faut pas multiplier ce que tu viens de convertir mais ce que tu avais deja converti avant de l'ajouter au chiffre courant.

Ah ouai ^^' la j'ai bugger desole

Envoyé par gl

Pourquoi ne pas directement travaille en decimal ?

Comment ?

Envoyé par bi_turbo

premiere remarque

le prototypage float converttofloat(char *p);

printf ("%f\n",converttofloat(pointeur));

Envoyé par bi_turbo

P est de type char

if( (p >= '0') && (p <= '9') ) tu compare p avec le caractere '0' ,'9' ou leur code ascii...
mon avi est que certainement tu veux que p €[0,9] pour les entiers.

J'ai pas compris ce que tu voulais dire desoler .

Sinon voici le code actuel apres quelque modification :

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
#include <stdio.h>
 
float converttofloat(char *p);
int  my_getnbr(char p);
 
int main ()
{
	/*Essai heroiaue pour faire le triple d'un nombre 
	a virgule du type 47,87 pour lexemple*/
	char *pointeur;
 
	pointeur = "47,87";
	printf("%s\n",pointeur);
	printf ("%f\n",converttofloat(pointeur));
	return 0;
}
 
int my_getnbr(char p)
{
	if( (p >= '0') && (p <= '9') )
		return ((int)p - 48);
	else
		return (-1);
}
 
float converttofloat(char *p)
{
	int i=0 ; 
	int f=0 ; 
	int count=0;
	int countdec=0;
	float result = 0;
 
	printf("OK\n");
	/*Recuperation de la partie entiere*/
	while((p[count] != ',') && (p[count] != '\0') )
	{
		if(p[count] != ',')
		{ i *= 10 ; i += my_getnbr(p[count]) ; }
		count++;	
	}
 
	count++;
	printf("OK\n");	
 
	/*Recuperation de la partie decimale*/
	while(p[count] != '\0')
	{
		if(p[count] != ',')
		{ f *= 10 ; f += my_getnbr(p[count]) ; }
		count++;
		countdec++;
	}
 
	printf("OK\n");
 
	result =  f;
	while(countdec != 0)
	{
		result /= 10;
		countdec--;
	}
	result += i;
 
	return result;
}

gl · 21/08/2007, 12h32

Envoyé par firemax

Envoyé par gl

pourquoi -48 ? Qu'est ce qui ne va pas avec -'0' ?

Bah on recupere sous forme de char donc '0' = 48 , '1' = 49 ... Non ?
Du coup pour le recuperer sous sa forme decimale on lui retire 48 , je me trompe ?

* Retirer '0' marche tout aussi bien dans ton cas (comme tu l'as dit chez toi '0' = 48).
* Continue a fonctionner sur les architectures utilisant des charset ou '0' n'est pas egal a 48 (au hasard EBCDIC.
* Utiliser '0' est largement plus lisible et renseigne bien davantage sur le but du bout de code que d'utiliser 48.

[QUOTE=firemax;2419679]

Envoyé par gl

Sinon voici le code actuel apres quelque modification :

Pose-t-il toujours le meme probleme ?

**firemax** · 21/08/2007, 13h43

Non il ne pose plus probleme (desoler j'ai oublier de clicker sur resolu)

Si ce n'est que j'obtiens 47.869999 au lieu de 47,87 mais je vais me debrouiller (enfin j'espere)

J'ai modifier le code pour - '0' tu as raisons c'est beaucoup plus lisible

Merci tous le monde

gl · 21/08/2007, 14h08

Envoyé par firemax

Si ce n'est que j'obtiens 47.869999 au lieu de 47,87 mais je vais me debrouiller (enfin j'espere)

C'est normal, c'est lie a la representation interne des reels.
La plupart des reels ne peuvent pas etre represente de maniere exacte, le resultat des calculs fournit une approximation.

Si tu veux gagner en precision, utilises des double plutot que des float.

Sinon ton code presente encore des imperfections:

* Pas de gestion des chaines malformatees (presence d'un caractere non numerique)
* Pas de gestion du signe
* Mauvaise gestion de l'absence de ','
* gestion surprenante de la presence de plusieurs ',' dans la chaine