Program źle dodaje liczby ujemne

Question

Przy liczbach dotatnich nie ma żadnych probleów i wszystko działa jak należy ale jak wpisuje ujemne to wyskakują strasznie duże liczby w odpowiedzi. Oto mój kod.

#include <iostream>



using namespace std;
int testy;
int ilosc;
int suma;

int odp[2] ;

int main()

{
    int s=0;// zmienna pomocnicza do przepisania sumy do tablicy
    cin>>testy;
    for(int i=0; i<testy; i++)
    {

        cin>>ilosc;


        int*tablica;
        tablica= new int[ilosc];
        for(int i=0; i<ilosc; i++)
        {

            cin>>tablica[i];


        }


        for (int x=0; x<=ilosc; x++)
        {
            suma+=tablica[x];
        }
        int s=0;
     
odp[s]=suma;
s++;


        delete [] tablica;

    }
    cout<< odp[0]<<endl;
    cout<<odp[1]<<endl;





    return 0;
}

Dodam jeszcze że program miał działać następująco- 

- poprać liczbę testów do wyknaia ( ile zostawów liczb będzie sumowanych)

- pobrać ( zmnienną w poszczególnych testach)  ilość liczb które będą sumowane 

- na koniec wypisać sumy po dwóch testach

Answer 1

Nie będe dalej analizował kodu bo już widzę że pętla w pętli ma tą sama zmienną czyli "i" . ;)

Answer 2

W drugiej petli z x-em: x<ilosc zamiast x<=ilosc. Wychodzisz poza tablice i do sumy dodaja sie smieci z pamieci.
Btw po co s++ jesli i tak zawsze s jest zerowane?

Comments

Słuszna uwaga. Wystawie s poza pętle żeby się zwiększałó po iteracjii.

---

Już to nawet zrobiłem bo s już jest poza pętlą tylko że togo starego nie skasowałem :)

Answer 3

Takie coś wystarczy?:

http://wklej.org/hash/4dba8f3cc2a/

Answer 4

Po wprowadzeniu poprawek kod wygląda tak :

#include <iostream>



using namespace std;
int testy;
int ilosc;
int suma;

int odp[2] ;

int main()

{
    int s=0;// zmienna pomocnicza do przepisania sumy do tablicy
    cin>>testy;
    for(int i=0; i<testy; i++)
    {

        cin>>ilosc;


        int*tablica;
        tablica= new int[ilosc];
        for(int z=0; z<ilosc; z++)
        {

            cin>>tablica[z];


        }


        for (int x=0; x<ilosc; x++)
        {
            suma+=tablica[x];
        }


odp[s]=suma;
s++;


        delete [] tablica;

    }
cout<<odp[0]<<endl;
cout<<odp[1]<<endl;




    return 0;
}

ale chyba ostatnia pętla nie działa poprawnie bo nie liczy drugiej sumy i uzyskuje dwie takie zame odpowiedzi na koniec