SPOJ Czy Kod...

Question

Witam, jestem lajkiem w programowaniu ale robię już któryś kod z kolei nad którymi się głowie po dwa dni jak nie więcej. Mimo, że wynik(wyjście) wychodzi poprawnie i tak mi nie zalicza zadania.. np nad tym.. który właśnie dokończyłem (niby teraz proste ale sie musialem nagłowić ;d) -  tu zadanie : http://pl.spoj.com/problems/SUMA/
Nie wiem czy to jakoś inaczej trzeba pisać te kody pod spoja czy wydaje mi się że mój kod już jest ok (mimo, że daje dobre wyjscie) zobaczcie jeśli możecie :
 

#include <iostream>
#include <cstdlib>
using namespace std;
int  liczba[200];
int suma[200];
int ile;
int main()
{
    cout<<"Podaj ilosc liczb: "; cin>>ile;
    cout<<"Podawaj liczby z przedzialu od -100 do 100 "<<endl;;

    for(int i=0; i<ile; i++)
    {
            cin>>liczba[i];
            cout<<endl;
            if((liczba[i]>-100) && (liczba[i]<100))     {suma[i]= suma[i-1]+liczba[i];}
            else if (liczba[i]<-100)                    {cout<<"Za mala liczba!"; i--;}
            else                                        {cout<<"Za duza liczba! Powtorz liczbe"; i--;}

    }
    system("cls");
    for(int i=0; i<ile; i++)
    cout<<suma[i]<<endl;

    return 0;
}

 

Answer 1

Jedyne czego potrzebujesz to pętla while(cin>>liczba) i oczywiście reszta instrukcji.

Answer 2

Nie dawaj ozdobników w kodzie do sprawdzajki ....
system("cls"), couty które nie zawierają odpowiedzi itp.

Answer 3

Teraz zajmę się samym wejściem. Cytując zadanie: "Na wejście programu podana zostanie pewna nieokreślona".
Twój program pobiera ilość liczb jakie zostaną wprowadzone, a to już jest źle.

---

Nie musisz sprawdzać czy liczby mieszczą się w danym zakresie, to jest tylko informacja żebyś wiedział na jakich typach danych najlepiej pracować.

---

Sposób zapisu danych do tablicy suma też zostawia wiele do życzenia. 
W pierwszym przejściu pętli program wykonuje mniej więcej coś takiego:
 

suma[0] = suma[-1]+liczba[0];

Indeksem tablicy nie może być liczba ujemna.

Answer 4

#include <iostream>
using namespace std;
int main()
{
	int liczba, suma=0;
	while (cin >> liczba) {
		suma+= liczba;
		cout << suma<< endl;
	}
	return 0;
}

Ten kod jest poprawny. Tylko nie wrzucaj go na chama, tylko przeanalizuj i zrozum jak działa.

Answer 5

Czyli trochę przekombinowałem.. No Twój kod wydaję się zrozumiały i sensown ale totalnie nie wiedziałem i nie wpadło mi do głowy, że można cin wpisać w while.. Na początku też zrobiłem podobny i prosty kod z for ale mi go nie przyjmowało. Dzięki za pomoc w razie czego będę pytał pewnie nie raz jak już nie będę wiedział co robić.

Comments

Ważne że jak sam napisałeś, starałeś się to zrobić przez kilka dni.