Appel de procédure : variable non renvoyée

Appel de procédure : variable non renvoyée - C++ - Programmation

Marsh Posté le 27-11-2011 à 16:10:37    

Bonjour à tous ! je suis en 1ere année en IUT informatique et voila mon problème :
on est en train d'apprendre à se servir des sous programmes et des procédures.
je dois faire un sous programme qui calcule le minimum, le maximum et la moyenne des éléments d'un tableau, pour le moment pas de problème, mais je n'arrive pas à renvoyer mes résultats dans mon programme principal :( une idée ?
 
procédure :

Code :
  1. void minMaxMoy(int pNbelt,const float ptabVal[],float pMin, float pMax,float pMoy)
  2. {
  3.     int i;
  4.     float pSom;
  6.     //calcul du min et du max
  7.     pMin=ptabVal[0];
  8.     pMax=ptabVal[0];
  9.     for(i=1;i<pNbelt;i++)
  10.     {
  11.         if(ptabVal[i]<pMin)
  12.         {
  13.             pMin=ptabVal[i];
  14.         }
  15.         if(ptabVal[i]>pMax)
  16.         {
  17.             pMax=ptabVal[i];
  18.         }
  20.     }
  22.     //calcul de la somme puis de la moyenne
  23.     pSom=0;
  24.     for(i=0;i<pNbelt;i++)
  25.     {
  26.         pSom=pSom+ptabVal[i];
  27.     }
  28.     pMoy=pSom/pNbelt;
  29. }


 
 
programme principal :
 

Code :
  1. int main(void)
  2. {
  3.    float tabTemps[KNBCOURMAX] ; // stocke les temps saisis relatifs à chaque coureur
  4.    int nbCour ; //compteur, nombre de coureurs
  5.    float min; // meilleur temps
  6.    float max; // temps le plus mauvais
  7.    float moy; // moyenne des temps
  10. minMaxMoy(nbCour,tabTemps,min,max,moy);
  12. printf("\nLe meilleur temps est : %f\nLe plus mauvais temps est : %f\nLa moyenne des temps est : %f",min, max, moy);
  13. }
  15. return 0;


 
il y a peu être des trucs qui servent à rien ou en trop , j'ai sélectionné le bout de programme qui pose problème, le tableau a été rentré avant, le minimum, le maximum et la moyenne sont correctement calculé dans le sous programme c'est juste pour les renvoyer que j'ai un problème
 
merci d'avance pour votre aide !
 
 

Message cité 1 fois
Reply

Marsh Posté le 27-11-2011 à 16:10:37   

Reply

Marsh Posté le 28-11-2011 à 00:36:45    

Salut,

 

Utilise des pointeurs pour les valeurs retournées.

Reply

Marsh Posté le 28-11-2011 à 08:42:58    

Arg :s on l'a pas encore vus mais j'ai trouvé mon problème : il manquait les "&" après les floats dans la déclaration de la procédure :)


Message édité par Benh_31 le 28-11-2011 à 09:46:31
Reply

Marsh Posté le 28-11-2011 à 13:23:17    

Tu apprends la programmation ou l'alchimie ? [:404 not found:3]  
 
http://hfr-rehost.net/imgs.xkcd.com/comics/pointers.png

Reply

Marsh Posté le 28-11-2011 à 13:53:24    

j'ai pas compris la :s

Reply

Marsh Posté le 28-11-2011 à 19:02:02    

Renseigne-toi sur la signification des "&" que tu as ajouté ;)

Reply

Marsh Posté le 01-12-2011 à 22:01:39    

Moi aussi je suis en IUT informatique je vais essayer de te montrer un peu tes erreurs mais ne prend pas ça pour acquis  
 
 

Code :
  1. void minMaxMoy(int pNbelt,const float ptabVal[],float pMin, float pMax,float pMoy)
  2.     {
  3.         int i;
  4.         float pSom;
  5.         //calcul du min et du max
  6.         pMin=ptabVal[0];
  7.         pMax=ptabVal[0];
  8.         for(i=1;i<pNbelt;i++)
  9.         {
  10.             if(ptabVal[i]<pMin)
  11.             {
  12.                 pMin=ptabVal[i];
  13.             }
  14.             if(ptabVal[i]>pMax)
  15.             {
  16.                 pMax=ptabVal[i];
  17.             }
  18.         }
  19.         //calcul de la somme puis de la moyenne
  20.         pSom=0;
  21.         for(i=0;i<pNbelt;i++)
  22.         {
  23.             pSom=pSom+ptabVal[i];
  24.         }
  25.         pMoy=pSom/pNbelt;
  26.     }


 
Pour commencer petite partie de Cours : Une procédure qui renvoie un résultat s'appelle une "Fonction" soit relis bien la fonction MinMaxMoy que tu as fait tout Pour commencer le type de la fonction soit "Void" n'est pas bon car si tu veut renvoyer un resultat tu ne créer pas une fonction qui n'en renvoient pas Void permet est un type qui ne renvoient rien si je peut l'expliquer comme ca  
 
De plus relis le cours sur les fonctions a l'occasion , toute fonction retourne un résultat  
D'ou l'utilisation de la syntaxe suivante  

Code :
  1. Return "le resultat que tu veux retourner"


en l'occurence

Code :
  1. Return pSom ;

(je n'en suis pas sur car ton programme est un peu compliqué a lire)
 
Pour continuer et j’espère ne pas te dire de bêtise  
tu déclare dans ta fonction  
int i ;
puis par la suite dans ton for tu redeclare i a 1
il n'est peut être donc pas utile de déclarer int i et éviter ainsi de surcharger ton programme  
 
Autre information il me semble que hoov a voulut faire passez c'est que si tu fait un passage de paramètre par valeur tu ne pourra pas changer les valeur de ton programme principale imaginons  

Code :
  1. Fonction Plus( int a)
  2. {
  3. a=a+1;
  4. return a ;
  5. }
  6. Programme  Principale
  7. int Nb =10;
  8. Plus(Nb) ;


Si tu fait cela Nb restera a 10 car le passage par valeur ne peut modifier la valeur declarer dans le programme principe
 c'est encore du cours , pour cela il faut donc utiliser un passage de paramétre soit par référence soit par adresse ( pointeur) après si vous ne l'avez pas fait effectivement c'est pas simple.
 
J'espere t'avoir aider  
 
PS : c'est du C++ que j'ai mis n'ayant pas encore trop bucher sur du C peut etre que des choses différent  
 
Voila bien sur je peut me tromper je suis étudiant tout comme toi donc soit indulgent :)
au plaisir  

Reply

Marsh Posté le 02-12-2011 à 23:35:18    

si c'est du CPP, fais passer tes résultats max / min / moy par reference avec le '&' dans le prototype, là tu fais passer par copie.
Sinon, en C, je dirais pointeurs

 

edit : Houlà, avait pas vu l'espèce de petit déterrage de topic résolu. :o

 

Bon sinon, t'es pas forcément obligé d'utiliser un return pour renvoyer une valeur. si tu passes par pointeur, tu modifie direct la valeur dans la case mémoire, pour reprendre ton exemple :

Code :
  1. void plus(int *a)
  2. {
  3. // On accède à la valeur à l'adresse 'a' via le * et on incrémente de 1
  4. (*a)+=1;
  5. }


et dans le main

Code :
  1. int main()
  2. {
  3. int val=10;
  4. plus(&val); // le '&' permet d'envoyer l'adresse de la variable 'val' dans ce cas
  5. }
 

Mais pour du C++, faut faire passer par reference, c'est plus sûr. :o


Message édité par Terminapor le 02-12-2011 à 23:39:01

---------------
Perhaps you don't deserve to breathe
Reply

Marsh Posté le 03-12-2011 à 00:34:07    

Benh_31 a écrit :

Bonjour à tous ! je suis en 1ere année en IUT informatique et voila mon problème :
on est en train d'apprendre à se servir des sous programmes et des procédures.
je dois faire un sous programme qui calcule le minimum, le maximum et la moyenne des éléments d'un tableau, pour le moment pas de problème, mais je n'arrive pas à renvoyer mes résultats dans mon programme principal :( une idée ?
 
procédure :

Code :
  1. void minMaxMoy(int pNbelt,const float ptabVal[],float pMin, float pMax,float pMoy)


 
il y a peu être des trucs qui servent à rien ou en trop , j'ai sélectionné le bout de programme qui pose problème, le tableau a été rentré avant, le minimum, le maximum et la moyenne sont correctement calculé dans le sous programme c'est juste pour les renvoyer que j'ai un problème
 
merci d'avance pour votre aide !
 
 

C'est pas mal ce que tu as fait. A l'oubli des & devant les paramètres près.
Tu pouvais tout faire en une seule boucle.
Attention néanmoins à tester les cas limites des paramètres en entrée, ici, il faut pNbelt > 0, sinon on va au devant de problèmes.
 

Code :
  1. void minMaxMoy(int pNbelt, const float ptabVal[], float& pMin, float& pMax, float& pMoy) {
  2.     if (pNbelt < 1) {
  3.         std::cerr << "ERREUR: Nb d'elements du tableau incorrect. Fin du programme!" << std::endl;
  4.         exit(1);
  5.     }
  6.  
  7.     pMoy = 0;
  8.     pMin = pMax = ptabVal[0];
  9.     for (int i = 0; i < pNbelt; ++i) {
  10.         pMoy += ptabVal[i];
  11.         if (ptabVal[i] < pMin)  { pMin = ptabVal[i]; }
  12.         if (ptabVal[i] > pMax)  { pMax = ptabVal[i]; }
  13.     }
  14.     pMoy /= pNbelt;
  15. }


Dernier point, n’hésites pas à utiliser des espaces, ça rend le code bien plus lisible (if (ptabVal[i] < pMin) plutôt que if(ptabVal[i]<pMin) par exemple).
 
Remarque: Bien que tu n'aies pas vu les pointeurs, il yen a un caché ici:
void minMaxMoy(int pNbelt, const float ptabVal[], float& pMin, float& pMax, float& pMoy)  
signifie en fait
void minMaxMoy(int pNbelt, const float *ptabVal, float& pMin, float& pMax, float& pMoy)  
Malgré ce que l'écriture laisse croire, quand on a un paramètre avec [], ce n'est pas une table que l'on passe, mais en fait un pointeur.
 
A+,


Message édité par gilou le 03-12-2011 à 01:35:55

---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed