Convertisseur Decimal - Binaire

Version imprimable

Bonjour à tous et à tout.

Voila j'ai une question, ou plutôt un gros soucis avec mon code.

Pourriez-vous y jeter un coups d'oeuil j'arrive plus à le déboguer moi-même.

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
char* DecToBin(unsigned int inNbDec)
{
	/*********Déclaration et initialisations des variable locales***************/
 
	int inCompter(0);
	int	inCompter2 = 0;
	char chTabReste[];
 
 
	while (inNbDec != 0)
	{
		chTabReste[inCompter++] = inNbDec % 2;
		inNbDec /= 2;
 
	}
 
	while (inCompter2 <= inCompter)
	{
		chTabReste[inCompter2++] = chTabReste[inCompter];
		chTabReste[inCompter--];
	}
 
	return(chTabReste);
 
}

D'avance merci

01/12/2014, 18h42
white_tentacle

Code:

char chTabReste[];

Déclare un tableau, mais aucune mémoire allouée derrière. Pas étonnant que ça plante. Utilise std::vector à la place.

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
28
29
30
31
iostream.....etc...
#include <vector>
 
 
 
 
char* DecToBin(unsigned int inNbDec)
{
	/*********Déclaration et initialisations des variable locales***************/
 
	int inCompter(0);
	int	inCompter2 = 0;
	vector<char> chTabReste[];
 
 
	while (inNbDec != 0)
	{
		chTabReste[inCompter++] = inNbDec % 2;
		inNbDec /= 2;
 
	}
 
	while (inCompter2 <= inCompter)
	{
		chTabReste[inCompter2++] = chTabReste[inCompter];
		chTabReste[inCompter--];
	}
 
	return(chTabReste);
 
}

???

02/12/2014, 02h18
Bousk

Programmer ce n'est pas mettre quelques mots-clés bout à bout et prier que ça marche. :weird:
Ce code ne compile même pas...
02/12/2014, 09h21
AWDAW

Citation:

Envoyé par Bousk

Programmer ce n'est pas mettre quelques mots-clés bout à bout et prier que ça marche. :weird:
Ce code ne compile même pas...

Répondre au gens en essayant de décoder leur code ce n'est pas simplement leur répondre comme si on était des merdes. me sembles que ce forum est fait pour ça......tu réalises bien que c'est deja 1 qu'une fonction du programme avec comme entrée inNbDec et comme sortie chTabReste.

Merci aux passionné de prendre leur temps de me répondre et non pas la police du forum
02/12/2014, 10h32
Ehonn

La réponse de Bousk n'est pas très diplômate; mais ton code, non-complet, est jeté dans un message avec aucune explication, aucun question et aucune description du problème que tu rencontres, on ne sait même pas quelle est l'erreur donnée par le compilateur. Bref, la première chose à faire c'est de regarder comment on utilise std::vector.
Si tu as des soucis, n'hésites pas à poster des questions mais en expliquant le problème point par point, une chose à la fois (et on sera pour te répondre avec plaisir ;)).
02/12/2014, 10h44
white_tentacle

Comme tu le dis, les « passionnés » (en fait, pour certains, des professionnels qui font ça sur leur temps « libre »), répondent bénévolement aux questions.

La moindre des politesses est donc de :
- arriver avec des questions précises sur ce qui ne va pas
- présenter un code qui compile, ou éventuellement, s’il ne compile pas, les messages d’erreur du compilateur
- lire la FAQ et autres aides en tout genre avant de poser des questions

Ce forum n’est pas là pour fournir un cours de C++ (il y en a dans les ressources), ni pour faire les exercices à la place des étudiants (puisque manifestement, c’est un exercice). Comme tu as l’air d’être débutant, il y a de plus une section « débuter » où ce message aurait plus eu sa place, mais ça ça n’est pas très grave.

Donc ici, plusieurs problèmes, d’abord de forme :
- le code que tu montres n’est pas du C++, c’est du C
- si je te dis d’utiliser std::vector, je suppose que tu vas regarder comment on l’utilise. Pas juste « remplacer char[] par std::vector en priant pour que ça marche ». Comme l’a dit Bousk, coder, ça impose de savoir ce qu’on fait, et de la rigueur.

Et ensuite, sur le fond du code :
- allouer un tableau comme variable locale et le renvoyer, ça ne peut que planter (car la mémoire allouée sur la pile sera libérée)
- accéder à de la mémoire non allouée (puisque le tableau fait une taille de 0) ne peut que planter
- l’algorithme est faux (la deuxième boucle en particulier). Déroule le à la main sur un exemple simple (par exemple, le nombre 5) pour t’en convaincre.