zaire90 (Normal User)
Rookie
 
Messaggi: 46
Iscritto: 16/10/2009
|
Non capisco perchè questo programma non mi gira bene...
Codice sorgente - presumibilmente C++ |
/* QUESTO PROGRAMMA ACQUISISCE DA TASTIERA UN NUMERO INTERO POSITIVO N, E CALCOLA E STAMPA SUL VIDEO LA SOMMA E LA MEDIA DEI PRIMI N ELEMENTI DELLA SUCCESSIONE DI NUMERI REALI a(n): a(n)=(n+3)/(n^3+5) n=1,2,3... */ #include <iostream> using namespace std; void presenta(); void leggiN(int); float somma(float,float); float media(float,float); void stampasommamedia(); int main() { int N;float n,SOMMA,MEDIA,x,y; presenta(); leggiN(N); SOMMA=somma(n,x); MEDIA=media(n,y); stampasommamedia(); system("PAUSE"); return 1; } void presenta() // Presentazione del programma { cout<<"Scrivi un numero intero positivo N e il programma calcolera'"; cout<<"\nla somma e la media dei primi N elementi della successione"; cout<<"\ndi numeri reali a(n):"; cout<<"\n\n a(n)=(n+3)/(n^3+5)"; return; } void leggiN(int N) // Legge il numero N { cout<<"\n\n\n N = ";cin>>N; while (N<=0) { cout<<"\nN deve essere maggiore di 0!\n\n"; cout<<" N = ";cin>>N; } return; } float somma(float n,float x) // Esegue la somma { float SOMMA=0;int N; for (n=1;n<=N;n++) { x=(n+3)/((n*n*n)+5); SOMMA=SOMMA+x; } return SOMMA; } float media(float n,float y) // Esegue la media { float MEDIA=0;int N; for (n=1;n<=N;n++) { y=(n+3)/((n*n*n)+5); MEDIA=MEDIA+y; } MEDIA=MEDIA/N; return MEDIA; } void stampasommamedia() // Stampa il risultato di somma e media { float SOMMA,MEDIA; cout<<"\n\nECCO LA SOMMA: SOMMA = "<<SOMMA; cout<<"\n\nECCO LA MEDIA: MEDIA = "<<MEDIA<<endl<<endl; return; }
|
...non capisco perchè!Mi aiutate?
|
|
|
theprogrammer (Normal User)
Guru^2
Messaggi: 2509
Iscritto: 28/01/2009
|
Non mi gira bene non significa nulla ...
|
|
|
zaire90 (Normal User)
Rookie
Messaggi: 46
Iscritto: 16/10/2009
|
"Non mi gira bene" significa che non da il risultato giusto. Se lo esegui vedrai che scrive come risultato cose impossibili come "SOMMA = 3.64709e-007" se scrivi N=3.
Questo significa...
|
|
|
theprogrammer (Normal User)
Guru^2
Messaggi: 2509
Iscritto: 28/01/2009
|
Postato originariamente da zaire90:
"Non mi gira bene" significa che non da il risultato giusto. |
Beh ... potevi dirlo.
Non gira bene potrebbe anche significare che si blocca, che hai una violazione di lettura o scrittura della memoria, e chissa' quante altre cose ...
Prima di eseguirlo, bisogna compilarlo correttamente ... e un compilatore decente ti indica che ci sono degli errori ..
Per cominciare, molte variabili non sono inizializzate (ad esempio N).
Ma quello che il compilatore non può evidenziare sono gli errori di "utilizzazione" delle variabili ...
La N, ad esempio, non è passata per riferimento alla funzione LeggiN (come invece dovrebbe essere) e le variabili non sono usate coerentemente ...
Ad esempio, la N all'interno della funzione somma è dichiarata localmente e quindi NON ha il valore di quella dichiarata nel main ...
Come nella funzione stampasommamedia in cui le variabili SOMMA e MEDIA sono dichiarate localmente e hanno (come di regola per il C/C++) valore casuale (quello che vedi in out), e nessun riferimento alle altre variabile dichiarate localmente nel main.
Sono problemi che riguardano il capitolo "visibilità delle variabili" in un qualsiasi libro di C/C++.
Ultima modifica effettuata da theprogrammer il 02/11/2009 alle 20:47 |
|
|
HeDo (Founder Member)
Guru^2
Messaggi: 2765
Iscritto: 21/09/2007
|
Quel programma è un disastro, sia dal punto di vista concettuale che stilistico:
1) "leggiN" oltre ad essere un nome orribile per una funzione non fa quello che dovrebbe fare, perchè il numero che legge da tastiera non lo inserisce in N. Il motivo è semplice: N non è passato per riferimento ma per valore. Per risolvere questo bisogna passare un puntatore. Per saperne di più studia il capitolo del libro sui puntatori.
2) I nomi delle variabili NON devono essere MAI scritti in maiuscolo, in quanto è una convenzione che si adotta per le costanti.
3) Per mandare a capo in una cout si usa la convenzione cout << endl; non cout << "\n";
4) Non mettere mai più istruzioni sulla stessa riga, deprime la lettura del codice.
5) Nella funzione somma la variabile N è privata e non inizializzata, quindi contiene un valore casuale preso dalla memoria di nessuna utilità pratica. Se devi riferirti alla N presa da "leggiN" devi passarla come parametro o dichiarare N globale.
6) In C++ non si scrive SOMMA = SOMMA + y, ma SOMMA += y
7) Sempre in "somma", la variabile y viene usata come una variabile locale, non come un parametro. In quanto il suo valore viene scritto ancora prima di essere letto.
8) In "somma" la variabile indice del for non è dichiarata
9) In "media" valgono le stesse cose dette in "somma"
10) La funzione "stampasommamedia" a parte il nome assolutamente orribile stampa due valori a caso, in quanto le variabili SOMMA e MEDIA non sono inizializzate, nè vengono prese da qualche parte. Se vuoi usarle in questo modo devi dichiararle globali o passarle come parametri dal main.
Se non capisci il perchè non ti gira una cosa di questo genere vuol dire che hai ancora molta strada da fare, torna a studiare.
|
|
|
zaire90 (Normal User)
Rookie
Messaggi: 46
Iscritto: 16/10/2009
|
Beh per quanto riguarda gli errori, lo compila senza segnalarmi nulla. Quindi avevo imaginato che fosse tutto giusto.
Però infatti anche io avevo pensato all'ipotesi del problema del passaggio per indirizzo, ma non sono riuscito a capire come risolverlo...devo mettere la "&" da qualche parte giusto?? Ma dove...come devo aggiustare il codice?
P.s. Purtroppo cado ancora in questo tipo di errore perchè non sono ancora molto pratico
|
|
|
HeDo (Founder Member)
Guru^2
Messaggi: 2765
Iscritto: 21/09/2007
|
|
|
|
theprogrammer (Normal User)
Guru^2
Messaggi: 2509
Iscritto: 28/01/2009
|
Postato originariamente da zaire90:
P.s. Purtroppo cado ancora in questo tipo di errore perchè non sono ancora molto pratico |
No ... cadi in questo tipo di errori perchè non hai studiato a sufficienza la parte "teorica" riguardante puntatori e reference sul tuo libro (mi auguro che tu segua un libro ...).
Ma - ancora prima dei puntatori - tu non hai chiare le BASI del linguaggio C (la visibilità delle variabili) e questo è grave. |
|
|
zaire90 (Normal User)
Rookie
Messaggi: 46
Iscritto: 16/10/2009
|
Mah...veramente no...non uso un libro, ma una dispensa corta corta. Poi non so nemmeno cosa siano (ancora) i puntatori...metto tra parentesi che non faccio informatica ma faccio matematica, quindi al corso di programmazione di matematica non mi spiegano per filo e per segno ogni cosa su come fare il programma perfetto (anzi!). Quindi non ho per niente chiaro la parte su puntatori,(visto che non so cosa siano) e tanto meno sulla visibilità delle variabili!
Ah...e inoltre, non dovrei nemmeno sforzarmi piu di tanto a scrivere programmi in funzioni, visto che non le avrei almeno fino al mese prossimo. Quindi siate clementi In ogni caso...mi potete correggere questo programma?
|
|
|
