[HELP] ciclo FOR in C++

[folk3n]

ciao a tutti, sottolineo da subito il fatto che sono un principiante alle prime armi con c++ ed è prorpio x questo che ho un problema da esporvi

devo scrivere un programmino che faccia questo:

"Redigere un programma che legga da input numeri interi non negativi fino a quando viene letto un numero negativo. A questo punto visualizzare la somma dei numeri non negativi."

io l'ho risolto così:

#include<iostream>
using namespace std;

int main(){
int n,x,y,risultato=0;
cout<<"inserire un numero intero (>0):";
cin>>n;
for(x=n;n>0;n--)
cout<<"inserire un numero intero:";
cin>>y;
risultato += y;
cout<<risultato;
system("pause");
return 0;
}

il problema è che non fa quello che deve fare!!!

ho anche la soluzione xò è con il WHILE invece che con il FOR ma io purtroppo ho bisogno di usare il FOR.
vi metto anche quella così capite meglio cosa deve fare il programma
eccola:

/*soluzione corretta:
#include <iostream>

using namespace std;

int main(){
int n, x, somma=0;
cout << "\nIntrodurre un numero intero (>0): ";
cin >> n;
while (n--)
{
cout << "Dammi un numero intero: ";
cin >> x;
somma += x;
}
cout << "La somma dei numeri introdotti e' " << somma << endl;
return 0;
}


potete aiutarmi x favore...grazie mille in anticipo

[zerocool_pe]

TE LO FACCIO DA CAPO CHE IL TUO NON VA PER NIENTE
#include <iostream>
#include <stdlib.h> //metticela sempre qst libreria
using namespace std;

int main()
{
int i,x,risultato;
risultato=0;
for (i=0;x>0;i++)
{
cout << "Inserisci un numero positivo" << endl;
cin >> x;
if (x>0)
{
risultato=risultato+x;
}
else
{
cout << "Hai inserito un numero negativo!!!" << endl;
}
}
cout << "Hai inserito " << i-1 << " numeri positivi" << endl;
cout << "La somma dei numeri positivi inseriti e: " << risultato << endl;
system ("PAUSE");
return 0;
}

[shinya]

Originariamente inviato da zerocool_pe

#include <stdlib.h> //metticela sempre qst libreria

Meglio includere <cstdlib>. E' più C++oso

[fgr]

Originariamente inviato da zerocool_pe

using namespace std;

cattiva tecnica C++: meglio non specificare spazi di nomi.
Usare:
using std::cout;
using std::cin;
ecc.

Originariamente inviato da zerocool_pe

system ("PAUSE");

questo non e' portabile

[folk3n]

ok grazie x l'aiuto 1a sola cosa, è possibile farlo con il FOR decrementato invece che incrementato(come stavo provando a fare io nel mio tentativo )?

ps grazie dei consigli fgr, 1a cosa, non capisco cosa significhi portabile e cosa c sia di sbagliato in system("pause");

[ka0s]

Originariamente inviato da fgr

cattiva tecnica C++: meglio non specificare spazi di nomi.
Usare:
using std::cout;
using std::cin;
ecc.

Perchè la consideri una cattiva tecnica?
Dov'è lo svantaggio nell'usare il namespace std?

[fgr]

Originariamente inviato da folk3n

ps grazie dei consigli fgr, 1a cosa, non capisco cosa significhi portabile e cosa c sia di sbagliato in system("pause");

per codice *portabile* si intende che lo stesso codice dovrebbe essere ricompilato su altri sistemi e/o piattaforme senza modifiche (magari cosi fosse).

system("pause") funziona su MS-DOS, Windows e basta.

[fgr]

Originariamente inviato da ka0s

Perchè la consideri una cattiva tecnica?
Dov'è lo svantaggio nell'usare il namespace std?

causa conflitti dei nomi nell'eventuale uso di altre librerie. Se queste possiedono nomi simili di funzioni e parametri.

Eventuali approfondimenti su questo e su moltri altri aspetti del C++ si trovano sul sito http://www.accu.org

[zerocool_pe]

Per la cronaca...se levi system pause non si blocca la finestra e non puoi vedere l'output prodotto...secondo..inserisci sempre namespace std; altrimenti quando usi stringhe di caratteri non vengono accettati gli spazi. Poi il programma come l'ho fatto è molto grezzo.puo essere ottimizzato e di molto ma non ho tempo di pensarci...devo stud per l'esame di informatica...cmq se avete altre soluzioni da postare sono felice di confrontarle con la mia...

[kk3z]

Originariamente inviato da zerocool_pe

Per la cronaca...se levi system pause non si blocca la finestra e non puoi vedere l'output prodotto...

La stessa cosa ci può fare con getchar() senza aver problemi con la portabilità.

Originariamente inviato da zerocool_pe

secondo..inserisci sempre namespace std; altrimenti quando usi stringhe di caratteri non vengono accettati gli spazi.

Eh?

PS: Ma zitti... ma dove stiamo

[ka0s]

Originariamente inviato da fgr

causa conflitti dei nomi nell'eventuale uso di altre librerie. Se queste possiedono nomi simili di funzioni e parametri.

Eventuali approfondimenti su questo e su moltri altri aspetti del C++ si trovano sul sito http://www.accu.org

ok grazie!

[fgr]

Originariamente inviato da zerocool_pe

Per la cronaca...se levi system pause non si blocca la finestra e non puoi vedere l'output prodotto...

ma quale cronaca /sapietone/: se avvii il programma da una finestra DOS vedi l'output del tuo programma!!!

Originariamente inviato da zerocool_pe

secondo..inserisci sempre namespace std; altrimenti quando usi stringhe di caratteri non vengono accettati gli spazi. Poi il programma come l'ho fatto è molto grezzo.puo essere ottimizzato e di molto ma non ho tempo di pensarci...devo stud per l'esame di informatica...cmq se avete altre soluzioni da postare sono felice di confrontarle con la mia...

su quale manuale l'hai letto? i guru del linguaggio raccomandano di non usare `using namespace std'! in ogni caso per le stringhe e` sufficiente `using std::string''.

p.s. sei libero di scriverer codice secondo i tuoi «standard»!!!

p.s.2 vedi di non essere scortese, nessuno ti ha chiamato in causa. Punto e taci! che modi sono?

[fgr]

Originariamente inviato da kk3z

PS: Ma zitti... ma dove stiamo

gia', noi siamo gentili, da dove sbuca fuori?

[fgr]

Originariamente inviato da ka0s

ok grazie!

su Usenet ho trovato qualcosa in italiano:
http://groups.google.it/group/it.com...0752121e342ddb
http://groups.google.it/group/it.com...5b15e9f4d047a3

[folk3n]

grazie a tutti, mi avete dato degli ottimi suggerimenti, sia sul mio problema nello specifico, sia in generale per quanto riguarda la programmazione in c++!!!
grazie d nuovo