Discussion :

[back63]

Salut,

J'ai actuellement un probleme avec une fonction censée convertir un entier long en une chaine de caractères :

La fonction a trois arguments : l'entier a convertir, le pointeur vers une chaine de caractères ou l'on doit ecrire la chaine, et la base.
Seulement, il y a un probleme , et quand j'essaye d'afficher la chaine a l'ecran, je n'obtient pas la chaine souhaitée.

Voici le 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
28
29
#include <iostream>
using namespace std;
 
void DecToChar(long, char*, int);
 
int main()
{
	long i;
	char* buf = new char[32];
	i = 16565;
	DecToChar(i, buf, 10);
	cout <<buf<<endl;
	system("pause");
}
 
void DecToChar(long Dec, char* out, int radix)
{
	char tableau[33];
	register int i;
	for(i = 31; Dec != 0 && i >= 1; i--)
	{
		tableau[i] = (Dec % radix) >= 10 ? 0x57 + (Dec % radix) : 0x30 + (Dec % radix);
		Dec /= radix;
	}
	tableau[32] = '\0';
	out = new char[32-i];
	register int j;
	for(j = 0; i < 33; out[j] = tableau[i], i++, j++);
}

et j'obtient (sur la console) :════════════════════════════════²²²²½½½½½½½½¯■¯■
Appuyez sur une touche pour continuer...

Aucun probleme de compilation.
Je pense qu'il y a un probleme au niveau du pointeur char, mais je ne suis pas sur.

PS : je suis un debutant, je ne suis peut etre pas au point sur certaines choses.

Merci.

[Eusebe]

Tu fais du C, pas du C++...

Si tu veux faire du C++, utilises plutôt des std::string et des flux pour convertir les données.

Si tu veux faire du C, demandes dans le forum C...

Mais déjà, en regardant très rapidement ton code, tu accède peut-être à l'élément 32 du tableau buf qui s'arrête à 31... (mais je n'ai pas regardé de très près...)

[olive_le_malin]

sinon tu peux aussi utiliser :
char *_ltoa( long value, char *string, int radix );

ou carrément sprintf()

[Médinoc]

Sauf qu'à mon avis, il était question de reproduire la fonction _itoa()...
Qui n'est pas standard, soit dit en passant...

[back63]

Sauf qu'à mon avis, il était question de reproduire la fonction _itoa()...

Oui en fait c'etait plutot ca...

[Médinoc]

À, j'ai trouvé le problème: Tu essaies de modifier out, mais c'est une variable locale.
Pour rester en C, il faudrait utiliser un pointeur de pointeur, ou pour rester conforme à la spec d'itoa(), toujours utiliser un buffer d'au moins 33 caractères.
Sans oublier que pour l'instant, on ne gère pas le signe négatif...

Déjà un petit code qui écrit sur out avec un bète strcpy()...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
void DecToChar(long Dec, char* out, int radix)
{
	char tableau[33];
	size_t i;
	tableau[33-1] = '\0';
	for(i = 33-1 ; Dec != 0 && i >= 1 ; )
	{
		int chiffre = (Dec % radix);
		i--;
		tableau[i] = ((chiffre >= 10) ? ('A' + chiffre-10) : ('0' + chiffre));
		Dec /= radix;
	}
	strcpy(out, tableau+i);
}

[back63]

bon, d'accord.
Merci a vous.