[C++] Problema codice

[Nando1987]

salve a tutti, a skuola stiamo iniziando il c++ e avrei un problema kon questo codice ke si mi si kompila ma da kome output numeri strani(alti), in pratica è ke dati 3 numeri bisogna calcolare il minimo il max e la media...kon i sottoprogrammi... ekko il codice

//dati 3 numeri calcolare il minimo massimo e media

#include <iostream.h>

int a, b, c, x, y, z, scelta, r;

int MEDIA(int x, int y, int z)
{
int g;
int o;
o=x+y+z;
g=o/3;
return g;
}

int MAX(int x, int y, int z)
{
int h;
if(x>y && x>z)
{
h=x;
}
if(y>x && y>z)
{
h=y;
}
if(z>y && z>x)
{
h=z;
}
return h;
}

int MIN(int x, int y, int z)
{
int f;
if(x<y && x<z)
{
x=f;
}
if(y<x && y<z)
{
y=f;
}
if(z<x && z<y)
{
z=f;
}
return f;
}


void menu()
{
cout<<"1 = Calcola Il Massimo Tra i Tre Numeri"<<endl;
cout<<"2 = Calcola Il Minimo Tra i Tre Numeri"<<endl;
cout<<"3 = Calcola La Media Tra i Tre Numeri"<<endl;
}

main()
{
cout<<"Inserisci il Primo Numero: ";
cin>>a;

cout<<"Inserisci il Secondo Numero: ";
cin>>b;

cout<<"Inserisci il Terzo Numero: ";
cin>>c;

menu();

cout<<"Quale Calcolo Vuoi Fare?"<<endl;
cin>>scelta;

if(scelta==1)
{
r=MAX(x, y, z);
cout<<"Il Numero Massimo è: "<<r<<endl;
}
if(scelta==2)
{
r=MIN(x, y, z);
cout<<"Il Numero Minimo è: "<<r<<endl;
}
if(scelta==3)
{
r=MEDIA(x, y, z);
cout<<"La Media Fra i Tre Numeri è"<<r<<endl;
}
return 0;
}


dove sbaglio?

grazie mille

[Nando1987]

risolto! grazie lo stesso

[alka]

Originariamente inviato da Nando1987
risolto! grazie lo stesso

Potresti postare la risposta per i posteri?
(scusa la cacofonia...)

[Nando1987]

si scusa ^^

allora avevo sbagliato una stupidagine...sarà il gin ^^


quindi questo codice


if(scelta==1)
{
r=MAX(x, y, z);
cout<<"Il Numero Massimo è: "<<r<<endl;
}
if(scelta==2)
{
r=MIN(x, y, z);
cout<<"Il Numero Minimo è: "<<r<<endl;
}
if(scelta==3)
{
r=MEDIA(x, y, z);
cout<<"La Media Fra i Tre Numeri è"<<r<<endl;
}
return 0;
}

va sostituito kon questo

if(scelta==1)
{
r=MAX(a, b, c);
cout<<"Il Numero Massimo è: "<<r<<endl;
}
if(scelta==2)
{
r=MIN(a, b, c);
cout<<"Il Numero Minimo è: "<<r<<endl;
}
if(scelta==3)
{
r=MEDIA(a, b, c);
cout<<"La Media Fra i Tre Numeri è"<<r<<endl;
}
return 0;
}

al posto di x, y z va messo a b c ^^

ciauzz

[murder eyes]

cmq vada le tre funzioni fanno veramente schifo come stile di programmazione.

[Nando1987]

non mi interessa + di tanto, basta ke glie lo do al prof ^^


tu kome avresti fatto?

[murder eyes]

int MEDIA(int x, int y, int z)
{
return ((x+y+z)/3)
}

int MAX(int x, int y, int z)
{
int max;
if(x>y) max=x;
else max=y;
if(z>max) max=z;
return max;
}

int MIN(int x, int y, int z)
{
//analogo a sopra
}

E' un fatto di un minimo di stile: meno variabili usate, meno if...
Ciao

[Nando1987]

ah...ok infatti, grazie ne terrò conto per il prox codice