C++

Appel de procédure : variable non renvoyée

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Appel de procédure : variable non renvoyée

Benh_31

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 :

void minMaxMoy(int pNbelt,const float ptabVal[],float pMin, float pMax,float pMoy)
{
int i;
float pSom;
//calcul du min et du max
pMin=ptabVal[0];
pMax=ptabVal[0];
for(i=1;i<pNbelt;i++)
{
if(ptabVal[i]<pMin)
{
pMin=ptabVal[i];
}
if(ptabVal[i]>pMax)
{
pMax=ptabVal[i];
}
}
//calcul de la somme puis de la moyenne
pSom=0;
for(i=0;i<pNbelt;i++)
{
pSom=pSom+ptabVal[i];
}
pMoy=pSom/pNbelt;
}

programme principal :

Code :

int main(void)
{
float tabTemps[KNBCOURMAX] ; // stocke les temps saisis relatifs à chaque coureur
int nbCour ; //compteur, nombre de coureurs
float min; // meilleur temps
float max; // temps le plus mauvais
float moy; // moyenne des temps
minMaxMoy(nbCour,tabTemps,min,max,moy);
printf("\nLe meilleur temps est : %f\nLe plus mauvais temps est : %f\nLa moyenne des temps est : %f",min, max, moy);
}
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

Publicité

hoov

Salut,

Utilise des pointeurs pour les valeurs retournées.

Benh_31

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

404 Not Found

Tu apprends la programmation ou l'alchimie ? [:404 not found:3]

Benh_31

j'ai pas compris la :s

404 Not Found

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

flaschgordon

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 :

void minMaxMoy(int pNbelt,const float ptabVal[],float pMin, float pMax,float pMoy)
{
int i;
float pSom;
//calcul du min et du max
pMin=ptabVal[0];
pMax=ptabVal[0];
for(i=1;i<pNbelt;i++)
{
if(ptabVal[i]<pMin)
{
pMin=ptabVal[i];
}
if(ptabVal[i]>pMax)
{
pMax=ptabVal[i];
}
}
//calcul de la somme puis de la moyenne
pSom=0;
for(i=0;i<pNbelt;i++)
{
pSom=pSom+ptabVal[i];
}
pMoy=pSom/pNbelt;
}

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 :

Return "le resultat que tu veux retourner"

en l'occurence

Code :

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 :

Fonction Plus( int a)
{
a=a+1;
return a ;
}
Programme Principale
int Nb =10;
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

Terminapor

I'll see you rise.

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.

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 :

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

et dans le main

Code :

int main()
{
int val=10;
plus(&val); // le '&' permet d'envoyer l'adresse de la variable 'val' dans ce cas
}

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

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

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

gilou

Modérateur
Modzilla

Benh_31 a écrit :

Code :

void minMaxMoy(int pNbelt,const float ptabVal[],float pMin, float pMax,float pMoy)

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 :

void minMaxMoy(int pNbelt, const float ptabVal[], float& pMin, float& pMax, float& pMoy) {
    if (pNbelt < 1) {
        std::cerr << "ERREUR: Nb d'elements du tableau incorrect. Fin du programme!" << std::endl;
        exit(1);
    }
 
    pMoy = 0;
    pMin = pMax = ptabVal[0];
    for (int i = 0; i < pNbelt; ++i) {
        pMoy += ptabVal[i];
        if (ptabVal[i] < pMin)  { pMin = ptabVal[i]; }
        if (ptabVal[i] > pMax)  { pMax = ptabVal[i]; }
    }
    pMoy /= pNbelt;
}

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é! --

FORUM HardWare.fr

Programmation

C++

Appel de procédure : variable non renvoyée

Sujets relatifs
Variable tableau / Range("A1:A" & i) = montab - ne fonctionne pas	Procédure d'installation sur windows
[PL/SQL] Erreur création procédure	espaces dans valeurs variable bloque appel à la fonction ?
Récuperer tous les index d'un array dans 1 variable	procédure click
problème pour ajouter un nombre à une variable (heure UTC et CEST)	Lire dans un fichier puis associer une variable à chaque ligne
[VBS] associer le stout d'une commande à une variable
Plus de sujets relatifs à : Appel de procédure : variable non renvoyée

Page générée en 0.085 secondes