Discussion :

[zolafencer]

Bonjour, je souhaite faire un programme qui calcule la moyenne arithmétique de deux notes avec l'utilisation de sous programme intitulé moyenne et l'appeler dans le main

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
#include <stdio.h>
#include <stdlib.h>
float moyenne (float* a, float* b)
{
    float* moy;
    *moy=(*a+*b)/2;
    return *moy;
}
int main()
{
    int moy1, moy2, moytot;
    printf("saisissez la premiere moyenne\nmoy1=");
    scanf("%f\n",moy1);
    printf("saisissez la deuxieme moyenne\nmoy2=");
    scanf("%f\n", moy2);
    moytot=moyenne(moy1,moy2);
    printf("la moyenne des deux notes est \n");
    scanf("%f", moytot);
    return 0 ;
}

bien évidemment ça marche pas ;(
please help

[Sve@r]

Même problème qu'ici http://www.developpez.net/forums/d10...ice-langage-c/

Toutefois, dans ce cas précis, ta fonction n'ayant pas à modifier les éléments qu'elle reçoit, l'utilisation de pointeurs est inutile.

PS: pas de "\n" au scanf() et inutile de faire saisir la moyenne via scanf vu qu'elle est calculée par ta fonction...

[NeoKript]

Salut,

il faut compiler son code avant de poster...

Pour ton problème :

Code :

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

scanf("%f\n",moy1);

- scanf prend en paramètre des pointeurs, toi tu passe des int ... (alors qu'avec %f, scanf s'attend à dépiler un float...

Code :

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

1
2
3
4
5
6
float moyenne (float* a, float* b)
{
    float* moy;
    *moy=(*a+*b)/2;
    return *moy;
}

Ensuite, ta fonction moyenne prend en paramètre des pointeurs sur float, tu passe des int ... (même pas des pointeur sur int ...)

De plus le passage par pointeurs est inutile puisque tu retourne la valeur.

Voilà,
un conseil revois les pointeurs et les variables en générale.

Si tu as d'autres soucis hésites pas.

[droggo]

Hai,

Salut,

il faut compiler son code avant de poster...

Pour ton problème :

Code :

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

scanf("%f\n",moy1);

- scanf prend en paramètre des pointeurs, toi tu passe des int ... (alors qu'avec %f, scanf s'attend à dépiler un float...

Code :

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

1
2
3
4
5
6
float moyenne (float* a, float* b)
{
    float* moy;
    *moy=(*a+*b)/2;
    return *moy;
}

Ensuite, ta fonction moyenne prend en paramètre des pointeurs sur float, tu passe des int ... (même pas des pointeur sur int ...)

De plus le passage par pointeurs est inutile puisque tu retourne la valeur.

Voilà,
un conseil revois les pointeurs et les variables en générale.

Si tu as d'autres soucis hésites pas.

Il ne faut pas qu'il hésite, mais toi, tu devrais peut-être hésiter à répondre : ne pas corriger l'utilisation d'un pointeur non initialisé (pointeur qui est d'ailleurs absolument inutile).

[Sve@r]

Hai,

Il ne faut pas qu'il hésite, mais toi, tu devrais peut-être hésiter à répondre : ne pas corriger l'utilisation d'un pointeur non initialisé...

Attends, c'est pas sympa de le casser comme ça. C'est vrai qu'il est passé à coté de cette erreur (et moi aussi d'ailleurs) mais ce qu'il a dit était juste et correct...

[NeoKript]

Hai,

Il ne faut pas qu'il hésite, mais toi, tu devrais peut-être hésiter à répondre : ne pas corriger l'utilisation d'un pointeur non initialisé (pointeur qui est d'ailleurs absolument inutile).

Code :

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

1
2
float* moy; 
*moy=(*a+*b)/2;

Héhé, en effet je n'avais pas vu le pointeur non initialisé, ça m'apprendra à répondre trop vite ^^

[droggo]

Lai,

Désolé pour les termes utilisés, plutôt maladroits. Dans ma tête, ça voulait dire "tourner 7 fois la langue dans ta bouche avant de cliquer sur 'valider'".

(Et j'aurais dû faire de même, et corriger ma phrase avant de faire ce fameux clic. )

[zolafencer]

pas besoin de se disputer, merci tous pour vos efforts
j'ai pas compris pourquoi pas besoin de pointeurs, quand est ce qu'on a besoin alors dans les fonctions?
sinon j'ai fait une petite modification au niveau du code mais ça donne un résultat très bizarre!!!

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
#include <stdio.h>
#include <stdlib.h>
float moyenne (float* a, float* b)
{
    float* moy;
    *moy=(*a+*b)/2;
    return *moy;
}
int main()
{
    int moy1, moy2, moytot;
    printf("saisissez la premiere moyenne\nmoy1=");
    scanf("%f\n", &moy1);
    printf("saisissez la deuxieme moyenne\nmoy2=");
    scanf("%f\n", &moy2);
    moytot=moyenne(&moy1,&moy2);
    printf("la moyenne des deux notes est \n%f", &moytot);
    return 0 ;
}

encore merci les gars

[NeoKript]

pas besoin de se disputer, merci tous pour vos efforts
j'ai pas compris pourquoi pas besoin de pointeurs, quand est ce qu'on a besoin alors dans les fonctions?
sinon j'ai fait une petite modification au niveau du code mais ça donne un résultat très bizarre!!!

encore merci les gars

Salut, il faut vraiment que tu revois les pointeurs.

C'est normal que tu ai des choses bizares....

Code :

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

printf("la moyenne des deux notes est \n%f", &moytot);

-> Ici tu affiche l'addresse de la variable et non ça valeur.

Code :

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

1
2
3
4
5
6
float moyenne (float* a, float* b)
{
float* moy;
*moy=(*a+*b)/2;
return *moy;
}

moy n'est pas initialisé, il pointe vers une case aléatoire...

En fait pour faire simple, tu utilise des pointeurs quand tu veux modifier des variables passées en paramètre à une fonction. Toi, tu te sert uniquement des valeurs pour retourner unenouvelle variable (la moyenne), donc pas besoin de pointeurs.

- Tu fais des castde int / float ..


Te donner la réponse, ce n'est pas t'aider mais bon ...

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
#include <stdio.h>
#include <stdlib.h>
float moyenne (float a, float b)
{
float moy;
moy=(a+b)/2;
return moy;
}
int main()
{
float moy1, moy2, moytot;
printf("saisissez la premiere moyenne\nmoy1=");
scanf("%f", &moy1);
printf("saisissez la deuxieme moyenne\nmoy2=");
scanf("%f", &moy2);
moytot=moyenne(moy1,moy2);
printf("la moyenne des deux notes est %f\n", moytot);
return 0 ;
}

Voilà, mais un conseil, reprend les pointeurs depuis le début !

Bonne soirée,
NeoKript

[NeoKript]

float moy1, moy2, moytot;

... float, tu as laissé int.

vu que tu as des int, il coupe la partie décimale lors de la division par 2 ainsi :

(5+2) / 2 = 3... et non 3.5
De même, lorsque tu récupères tes nombres avec scanf, tu met %f, mais c'est des int... du coup la variable ne contient pas du tout la bonne valeur.

[zolafencer]

ah mais oui qu'est ce que je suis bête!!!
désolée j'ai pas vu que j'ai laissé int et non float.
C'est bon merci

[YannPeniguel]

j'ai pas compris pourquoi pas besoin de pointeurs, quand est ce qu'on a besoin alors dans les fonctions?

Un pointeur est nécessaire dès lors que tu as besoin de manipuler une même variable dans plusieurs fonctions différentes.

[Sve@r]

Un pointeur est nécessaire dès lors que tu as besoin de manipuler une même variable dans plusieurs fonctions différentes.

On évitera donc d'aborder le sujet des globales

[YannPeniguel]

Je ne sais pas pour toi, mais moi je n'en utilise que pour les constantes.

[Sve@r]

Je ne sais pas pour toi, mais moi je n'en utilise que pour les constantes.

Ah, moi je passe plutôt par les macro pour ça. Non, j'ai beau réfléchir, je reste toujours à zéro globale...

[quetzacoatl]

Bonsoir,

Ici vous n'avez pas besoin des poiteurs car comme le dit Sve@r les valeurs passées en argument ne sont pas modifiées, vous travaillez avec mais vous ne les modifiez pas.
Par exemple si vous aviez voulu faire une fonction qui ajoute 1 à une variable a, la vous auriez eu besoin de pointeurs.

Pour votre problème, c'est dans le dernier printf() que vous avez fait une erreur:

printf("la moyenne des deux notes est \n%f", &moytot);

L'opérateur & désigne l'adresse, ainsi vous avez demander d'afficher l'adresse de la variable moytot.
Vous avez sûrement confondu avec scanf() mais si scanf() prend les adresses des variables entieres, c'est pour pouvoir en modifier la valeur comme ce que j'ai dit plus haut à propos de l'utilité des pointeurs dans les arguments d'une fonction

[zolafencer]

j'ai compris vos remarque, merci bcp
pour NeoKript, j'ai lu plusieurs tuto de pointeurs mais je comprnais toujours pas, je me suis dit alors que ce serait mieux si j'e faisais des exo et que lors des corrections je comprendrai à fur et à mesure les pointeurs. D'ailleurs vos remarques m'ont servis. Ils faut que je fasse d'avantage d'exo pour assimiler encore plus ce maudit chapitre.
Cependant, j'ai changé mon code mais je ne sais tjrs pourquoi ça ne marche pas encore (je veux dire il m'affiche un résultat bizarre) !!

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
#include <stdio.h>
#include <stdlib.h>
float moyenne (float a, float b)
{
    float moy;
    moy=(a+b)/2;
    return moy;
}
int main()
{
    int moy1, moy2, moytot;
    printf("saisissez la premiere moyenne\nmoy1=");
    scanf("%f", &moy1);
    printf("saisissez la deuxieme moyenne\nmoy2=");
    scanf("%f", &moy2);
    moytot=moyenne(moy1,moy2);
    printf("la moyenne est %f", moytot);
    return 0 ;
}