Discussion :

[AWDAW]

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 :

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

[white_tentacle]

Code :

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

char chTabReste[];

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

[AWDAW]

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
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);
 
}

???

[Bousk]

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

[AWDAW]

Programmer ce n'est pas mettre quelques mots-clés bout à bout et prier que ça marche.
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

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

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