[C++] Calcolo delle medie minime e massime

[Hilal]

Chiedo cortesemente se qualcuno riesce a trovarmi l'errore in questo programma , perché il compilatore non me lo lascia eseguire.
Il programma chiede di trovare la media delle minime e delle massime di un mese.
Sto veramente impazzendo nel provare a trovarlo ma nulla , grazie mille a chi mi aiuterà.
Hilal






#include <iostream>
using namespace std;
int ChiediDimensione();
int MediaMax(int vettore[],int d);
int MediaMin(int vettore[],int d);
int ChiediDimensione()
{
int d;
do{
cout<<"Digita i giorni composti dal mese che si sta trattando."<<endl;
cin>>d;
}while(d<28 || d>31);
return d;
}
int MediaMax(int vettore[],int d)
{ int MediaMaax=0;
for(int i=0;i<d;i++)
{cout<<"Digita le massime del giorno che stiamo trattando:"<< i <<endl;
cin>>vettore[i];
MediaMaax=vettore[i]+MediaMaax;}
return MediaMaax;
}
int MediaMin(int vettore[],int d)
{
int MediaMiin=0;
for(int i=0;i<d;i++)
{
cout<<"Digita la minima del giorno che stiamo trattando:"<< i <<endl;
cin>>vettore[i];
MediaMiin=vettore[i]+MediaMiin;
}
return MediaMiin;
}
int main()
{
int n,vettore[n];
int MediaMaax=0,MediaMiin=0;
n=ChiediDimensione();
MediaMax(vettore,n);
MediaMin(vettore,n);
cout<<"La media delle massime e':"<<MediaMaax/n<<endl;
cout<<"La media delle minime e':"<<MediaMiin/n<<endl;
return 0;
}

[shodan]

A parte l'identazione che non c'è, ma che pensi di fare con:

int n,vettore[n];

?
Di dare una dimensione a vettore prima di chiederla?

[Hilal]

A parte l'identazione che non c'è, ma che pensi di fare con:

int n,vettore[n];

?
Di dare una dimensione a vettore prima di chiederla?

Ho inserito due variabili alle quali do un valore nelle due righe sotto.
La dimensione del vettore avevo gia stabilita nell'array di sopra, in ChiediDimensione

[shodan]

Non funziona così: prima devi definire la dimensione, poi allocare il vettore. Quel che hai scritto in quelle righe non va bene dato che n non è inizializzata.

Corretto è:

    int n, *vettore;
        int MediaMaax=0,MediaMiin=0;
    n=ChiediDimensione();
    vettore = new int[n];
    ...

    delete[] vettore; // prima di return 0

[Hilal]

Ti ringrazio molto!! Adesso eliminando le 3 righe prima di ChiediDimensione funziona...#include <iostream>using namespace std;
int ChiediDimensione()
{
int d;
do{
cout<<"Digita i giorni composti dal mese che si sta trattando."<<endl;
cin>>d;
}while(d<28 || d>31);
return d;
}
int MediaMax(int vettore[],int d)
{ int MediaMaax=0;
for(int i=0;i<d;i++)
{cout<<"Digita le massime del giorno che stiamo trattando:"<< i <<endl;
cin>>vettore[i];
MediaMaax=vettore[i]+MediaMaax;}
return MediaMaax;
}
int MediaMin(int vettore[],int d)
{
int MediaMiin=0;
for(int i=0;i<d;i++)
{
cout<<"Digita la minima del giorno che stiamo trattando:"<< i <<endl;
cin>>vettore[i];
MediaMiin=vettore[i]+MediaMiin;
}
return MediaMiin;
}
int main()
{
int n,vettore[n],max,min;
n=ChiediDimensione();
max=MediaMax(vettore,n);
min=MediaMin(vettore,n);
cout<<"La media delle massime e':"<<max/n<<endl;
cout<<"La media delle minime e':"<<min/n<<endl;
return 0;
}

[shodan]

E' rimasto uguale a prima, ma vabbè...

[Hilal]

E' rimasto uguale a prima, ma vabbè...

Non è uguale a prima , non ho fatto il tuo cambiamento che a quanto pare non è necessario visto che funziona ...

[oregon]

Non è uguale a prima , non ho fatto il tuo cambiamento che a quanto pare non è necessario visto che funziona ...

Mi sembra inutile (e un po' presuntuoso) chiedere a chi ne sa più di te e poi non seguire i consigli.

Devi scrivere

int n,max,min;
n=ChiediDimensione();
int vettore[n];

e non come hai scritto tu, anche se ti sembra che funzioni e che vada bene ... Non va assolutamente ...

[Hilal]

Mi sembra inutile (e un po' presuntuoso) chiedere a chi ne sa più di te e poi non seguire i consigli.

Devi scrivere

int n,max,min;
n=ChiediDimensione();
int vettore[n];

e non come hai scritto tu, anche se ti sembra che funzioni e che vada bene ... Non va assolutamente ...

Se c'è una cosa che non sono è proprio essere presuntuoso!! E anzi mi sembra presuntuoso da lei dire che sicuramente ne sa più di me ... Il programma funziona benissimo , non capisco perché da me dovrebbe andare e da lei no ....

[oregon]

Se c'è una cosa che non sono è proprio essere presuntuoso!! E anzi mi sembra presuntuoso da lei dire che sicuramente ne sa più di me ... Il programma funziona benissimo , non capisco perché da me dovrebbe andare e da lei no ....

Non mi riferivo a me stesso ma a chi aveva già risposto la prima volta e confermato la seconda e all'affermazione "non ho fatto il tuo cambiamento che a quanto pare non è necessario "

Comunque, se il programma funziona e ritieni non necessari i cambiamenti proposti, tutto ok, inutile andare oltre. Penso però che al prossimo thread del tipo " avrei bisogno di una mano" avrai poche risposte ...