Problème avec les fonctions.

Version imprimable

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:

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.

18/12/2007, 14h43
souviron34
Code:

1 2 if (leChoix = 1)
en C, = est une ASSIGNATION
une comparaison se fait avec ==

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

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

18/12/2007, 15h08
souviron34

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

Utilises fgets , et enlève le "\n" à la fin du buffer...
18/12/2007, 15h23
emprex

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.
18/12/2007, 17h08
diogene

Citation:

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.

Après modification le code fonctionne merci beaucoup.
Mais comment puis-je faire pour valider mon choix et ensuite saisir le nombre à convertir, je voudrais que le menu soit bien distinct sur reste du programme pour lui ajouter d'autres options par la suite.

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
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
78
79
80
81
82
#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);
	if (leChoix == 1)
		convertiseurBinaireDecimal();
	else if (leChoix == 2)
		convertiseurDecimalBinaire();
	return leChoix;
}
 
int main(void)
{
 
	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();
	printf("\n");*/
	return 0;
}

18/12/2007, 21h45
fremen167

Citation:

Envoyé par emprex

Après modification le code fonctionne merci beaucoup.
Mais comment puis-je faire pour valider mon choix et ensuite saisir le nombre à convertir, je voudrais que le menu soit bien distinct sur reste du programme pour lui ajouter d'autres options par la suite.

Diogène se tue à te le répéter : utilise le fgets...

De toutes façons, ça te resservira !