Discussion :

[giggs]

Salut à tous, voila en fait j'ai un problème avec du code très très simple, en fait je me demande si ça ne vient pas de mon compilateur, voila 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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
#include <stdio.h>
 
int soustraire(int *i)
{
printf("tapez le chiffre a soustraire:\n");
int s;
scanf("%d",&s);
i-=s;
 
	return 0;
}
 
int additionner(int *i)
{
printf("tapez le chiffre a additionner:\n");
int s;
scanf("%d",&s);
i=i+s;
 
	return 0;
}
 
int main(void)
{
	char choix;
	int result;
	int *ptr;
	ptr=&result;
	printf("voici un programme qui se propose de soustraire et d'additionner deux valeures a la pricipale\n");
	printf("commencez par taper la valeure principale:\n");
	scanf("%d",&result);
	printf("par quoi voulez vous commencer, par additionner ou soustraire, tapez a ou s:\n");
	choix=getchar();
	 if(choix=='a')
	{
		printf("vous avez choisi d'additionner:\n");
		additionner(ptr);
	}
	else if(choix=='s')
	{
		printf("vous voulez soustraire:\n");
		soustraire(ptr);
	}
	else
	{
		printf("desole, la solution n'est pas prevue\n");
	}
	printf("voila vous avez fini l'operation, voici le resultat:%d\n",result);
	return 0;
}

Alors voila le soucis: lorsque je rentre la premiere valeure qui est censé pouvoir etre additionnée ou soustraite après au lieux de me donner la fonction ensuite choisie, il me renvoie directement la ligne du else, ce que je trouve très bizarre. Peut-ètre que c'est moi qui suis complètement hors sujet et que je ne comprend plus rien à la programmation (ce qui est probable).

PS: Ne pas s'affoler pour le peu de stabilité qu'offre le code avec les scanf et autre getchar(), je suis au courant des problèmes pouvant éventuelement apparaître (ce code n'est pas de moi d'origine et je cherche à aider un ami) mais il fallait que je le restitu tel quel pour bien comprendre...

Merci par avance.

[Flow_75]

Salut

Essaye de remplacer tes 'if' par un switch, par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
switch (choix)
             {
                case 'a' :
                    // fonction d'addition;
                case 's' :
                    // fonction de soustraction;
                default  : 
                    printf("desole, la solution n'est pas prevue\n");
              }

[giggs]

C'étais ma première idée, mais le problème reste entier, il me renvoyait alors directement au default (a la place du esle) et n'exécutait pas la fonction quand même...

[kamouminator]

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
int soustraire(int *i)
{
printf("tapez le chiffre a soustraire:\n");
int s;
scanf("%d",&s);
i-=s;
 
	return 0;
}
 
int additionner(int *i)
{
printf("tapez le chiffre a additionner:\n");
int s;
scanf("%d",&s);
i=i+s;
 
	return 0;
}

i est un pointeur sur int ! et tu lui ajoute (ou soustrait) s...c'est a *i qu'il faut ajouter ou soustraire, i contien l'adresse pas l'entier

sinon apres le scanf, il faut purger le stdin

[giggs]

Oui, un oublie la dessus, il suffit de rajouter un * avant les i mais ça ne résoud en rien le problème, la ça aurai juste enmpecher de faire les calculs sur la variable, et non pas empecher toute la fonction de se lancer... Bizarre tout ça, d'origine je suis censé aider quelqu'un qui est en première année d'iut et résultat des courses, c'est moi qui ai besoin d'aide, la il n'y a aucune raison valable pour que les fonctions ne s'exécutent pas, je vais essayer avec un autre compilateur peut-ètre c'est le mien qui débloque...

[kamouminator]

non, il faut purger apres chaque utilisation de scanf, c'est aussi pour ca que c'est deconceillé d'utiliser scanf !

[thewho]

Bonjour,

Les gaités de scanf (cherche un peu sur le forum, tu ne prux pas passer à travers, ça revient très souvent)

D'autre part, pour tes fonctions, pourquoi les avoir écrites comme ça ?

Code :

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

1
2
3
4
5
6
7
8
9
10
 
(int soustraire(int *i)
{
printf("tapez le chiffre a soustraire:\n");
int s;
scanf("%d",&s);
i-=s;
 
	return 0;
}

Pourquoi toujours renvoyer 0 plutôt que la valeur du résultat ?

(Je comprendrais bien que la valeur renvoyée ne soit pas le résultat du calcul, mais dans ce cas il faudrait que la valeur renvoyée prenne plusieurs valeurs, pour avertir l'appelant que tout s'est bien passé (ou mal).

De plus, ton paramètre i passé aux fonctions est un pointeur, et quand tu fais

Code :

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

i-=s;

tu enlèves s au pointeur, ce qui n'est sans doute pas ce que tu veux.

Autre chose, pourquoi lire la 2ème valeur à traiter dans les fonctions que tu appelles ?
Il me semblerait plus logique de lire les 2 valeurs dans le main, et faire des fonctions avec 2 paramètres, qui feraient le calcul, et renverraient le résult du calcul, plutôt que toujours 0 (car j'y reviens : renvoyer toujours la même valeur ne sert strictement à rien).

[kamouminator]

voila comment vider le stdin

Code :

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

1
2
3
4
int c;
while ( ((c = getchar()) != '\n') && c != EOF)
{
};

http://xrenault.developpez.com/tutoriels/c/scanf/

[giggs]

C'est bon, j'ai trouvé, c'était bien le compilateur qui avait un problème avec un autre ça marche très bien...
Thewho:
JE n'ai pas écrit ces fonctions comme ça, le but de ce topic n'était pas d'améliorer un bout de code dans le but de le rendre plus souple ou lisible, mais de comprendre pourquoi les fonctions ne s'exécutaient pas, voila pourquoi j'ai précisé dès le début qu'il ne fallait pas s'attarder sur le style de programmation médiocre, je suis au courant que les fonctions telles fgets sont préférable à :scanf ou getchar...

[kamouminator]

C'est bon, j'ai trouvé, c'était bien le compilateur qui avait un problème avec un autre ça marche très bien...

?? y'a des compilateur qui vident le flux entrant tout seuls apres le scanf ???

ps: autre chose, si c'est du c et non du C++, il faut declarer les variables en debut de fonction et non en plein milieu, sinon ca compile pas

[giggs]

Non, en fait le problème vient que j'ai voulu installé y a quelques jours la bibliothèque GTK+ et il s'en est suivi pleins de bug de la part de mon compilateur, donc il vas falloir que je le désinstalle puis le réinstalle.

[kamouminator]

Non...
tu as peut etre des problemes avec ton compilateur,
mais le fait que tu te retrouve directement sur le else ou sur le default viens bien du scanf...
ton code fais pareil chez moi !
au risque de me repeter....il faut purger