Problème avec les fonctions.

**emprex** · 18/12/2007, 14h39

Bonjour,
je souhaite faire un code qui laisse le choix à l'utilisateur entre une conversion binaire décimal ou l'inverse. Chaque fonction prise isolement fonctionne mais dans le cas ou je les assemble ca ne marche plus. Pourquoi ?

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
67
68
69
70
71
72
73
74
75
76
#include <stdio.h>
/****************************************************/
/*                                                  */
/* fonction qui convertit un binaire en décimal     */
/*                                                  */
/****************************************************/
int convertiseurBinaireDecimal()
{
	char leBinaire;
	int leDecimal = 0;
	do
	{
		scanf("%c",&leBinaire);
		if (leBinaire == '1')
			leDecimal = leDecimal * 2 + 1;
		else if (leBinaire == '0')
			leDecimal *= 2;
	}
	while (leBinaire != '\n');
	printf("%d\n", leDecimal);
	return leDecimal;
}
 
/****************************************************/
/*                                                  */
/* fonction qui convertit un décimal en binaire     */
/*                                                  */
/****************************************************/
int convertiseurDecimalBinaire()
{
   int leDecimal;
   int leBinaire = 2;
   scanf("%d", &leDecimal);
   while (leBinaire <= leDecimal)
      leBinaire *= 2;
   leBinaire /= 2;
   while (leBinaire != 0)
   {
      if (leBinaire <= leDecimal)
      {
         printf("1");
         leDecimal -= leBinaire;
      }
      else
         printf("0");
      leBinaire /= 2;
   }
   printf("\n");
   return leBinaire;
}
/****************************************************/
/*                                                  */
/*     fonction faire un choix                      */
/*                                                  */
/****************************************************/
/*int faireUnChoix()
{
	int leChoix;
	printf("Pour convertir un binaire en decimal taper 1:\n");
	printf("Pour convertir un decimal en binaire taper 2:\n");
	scanf("%d",&leChoix);
	return leChoix;
}*/
 
int main(void)
{
	int leChoix;
	/*faireUnChoix();*/
	printf("Pour convertir un binaire en decimal taper 1:\n");
	printf("Pour convertir un decimal en binaire taper 2:\n");
	if (leChoix = 1)
		convertiseurBinaireDecimal();
	else if (leChoix = 2)
		convertiseurDecimalBinaire();
	return 0;
}

Merci.

**souviron34** · 18/12/2007, 14h43

Code :

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

1
2
 
	if (leChoix = 1)

en C, = est une ASSIGNATION
une comparaison se fait avec ==

**emprex** · 18/12/2007, 15h05

j'ai fait la correction mais les fonctions ne s'executent pas.

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
67
68
69
70
71
72
73
74
75
76
77
#include <stdio.h>
/****************************************************/
/*                                                  */
/* fonction qui convertit un binaire en décimal     */
/*                                                  */
/****************************************************/
int convertiseurBinaireDecimal()
{
	char leBinaire;
	int leDecimal = 0;
	do
	{
		scanf("%c",&leBinaire);
		if (leBinaire == '1')
			leDecimal = leDecimal * 2 + 1;
		else if (leBinaire == '0')
			leDecimal *= 2;
	}
	while (leBinaire != '\n');
	printf("%d\n", leDecimal);
	return leDecimal;
}
 
/****************************************************/
/*                                                  */
/* fonction qui convertit un décimal en binaire     */
/*                                                  */
/****************************************************/
int convertiseurDecimalBinaire()
{
   int leDecimal;
   int leBinaire = 2;
   scanf("%d", &leDecimal);
   while (leBinaire <= leDecimal)
      leBinaire *= 2;
   leBinaire /= 2;
   while (leBinaire != 0)
   {
      if (leBinaire <= leDecimal)
      {
         printf("1");
         leDecimal -= leBinaire;
      }
      else
         printf("0");
      leBinaire /= 2;
   }
   printf("\n");
   return leBinaire;
}
/****************************************************/
/*                                                  */
/*     fonction faire un choix                      */
/*                                                  */
/****************************************************/
/*int faireUnChoix()
{
	int leChoix;
	printf("Pour convertir un binaire en decimal taper 1:\n");
	printf("Pour convertir un decimal en binaire taper 2:\n");
	scanf("%d",&leChoix);
	return leChoix;
}*/
 
int main(void)
{
	int leChoix;
	/*faireUnChoix();*/
	printf("Pour convertir un binaire en decimal taper 1:\n");
	printf("Pour convertir un decimal en binaire taper 2:\n");
	scanf("%d",&leChoix);
	if (leChoix == 1)
		convertiseurBinaireDecimal();
	else if (leChoix == 2)
		convertiseurDecimalBinaire();
	return 0;
}

**souviron34** · 18/12/2007, 15h08

maintenant tu as le problème avec scanf... fonction fortement déconseillée..

Utilises fgets , et enlève le "\n" à la fin du buffer...

**emprex** · 18/12/2007, 15h23

Merci beaucoup,
Je vais étudier cette fonction fgets, car ne ne l'ai jamais utilisée.
J'ai l'impression que je vais devoir écrire à nouveau les fonctions.

**diogene** · 18/12/2007, 17h08

Envoyé par emprex

J'ai l'impression que je vais devoir écrire à nouveau les fonctions.

Non, C'est un problème d'entrée au clavier : Lorsque tu fais un choix 1 ou 2 dans main, il reste dans le buffer du clavier le \n tapé pour terminer la ligne.
Le scanf ("%c",...) de convertiseurBinaireDecimal() lit un caractère, trouve ce \n et se termine aussitôt.
Si tu entres au clavier
1 1111000
au lieu d'essayer d'entrer
1
11110000
tu obtiendras la réponse 240 qui est correcte.
Le problème ne se pose pas pour le scanf("%d",...) de convertiseurDecimalBinaire parce que tu lis un entier et que les séparateurs qui précèdent la valeur sont dans ce cas sautés.

La solution est de purger le buffer d'entrée après le scanf du main. (rechercher fclean() par exemple sur le forum)

Indépendamment, étudier fgets est une bonne chose. Tu en auras besoin rapidement.

Problème avec les fonctions.

C

Vue hybride

Discussions similaires

Partager

Partager