Program zwraca dziwne wartosci - C++

Question

Witam. Na wstępie chciałbym dodać iż jestem początkującym programistą, o ile można mnie już tak nazywać :) 

Wczoraj ukończyłem odcinek Pana Mirosława na temat polskiego SPOJA. Stronka przypadła mi do gustu i kilka problemów już na niej ukończyłem. Niestety jednak znalazłem się w kompletnej czarnej dziurze odnośnie zachowania mojego programu i zwracania bardzo dziwnych liczb w problemie http://pl.spoj.com/problems/SUMA/

Po wpisaniu liczb całkowitych, np. takich jakie są przykładowe w tym problemie na SPOJU, to otrzymuję w konsoli nie wiadomo skąd wzięte liczby. Czasem jest to 40010, czasem kilka milionów. Analizowałem kod na spokojnie, ale nie potrafię sobie z tym poradzić. Poniżej zamieszam kod źródłowy: 

 

#include <iostream>
#include <windows.h>

using namespace std;


int main()
{
    int ilosc;
    int tab[10000];
    int a;

    cin>>ilosc;

    for (int i=0; i<ilosc; i++)
    {
        cin>>tab[i];
        a+=tab[i];
        cout<<a<<endl;
    }



    return 0;
}

 

Z góry dziękuję za wszystkie odpowiedzi i pozdrawiam. 

Answer 1

Musisz zainicjować zmienną. W C++ tworzona zmienna nie ma automatycznie wartości 0. Dlatego:

int a = 0;

Comments

1. zainicjaLIZOWAĆ zmienną
W informatce przyjęła się ta forma: http://sjp.pwn.pl/poradnia/haslo/inicjalizowac;9919.html

2. Zmienna globalna oraz statyczna jest wstępnie inicjalizowana zerem.

---

No bo inicjacja to może być najwyżej seksualna. Może być też rytuał inicjacji, jak w masonerii. 

---

Illuminati https://i.ytimg.com/vi/Erbh3YRsxy4/maxresdefault.jpg

---

Przepraszam, obiecuję poprawę.

---

To na prawdę działa o.O. Nie sądziłem że to będzie aż tak banalne. :) Dziękuję za pomoc, jednakże mam jeszcze jedno pytanie. Nie zawsze przy inicjalizacji zmiennej nadawałem jej wartość 0, a programy działały w miarę sprawnie, bez podobnych problemów. Jaki może być tego powód? 

---

Tworzysz zmienną, która zostaje umieszczona gdzieś w nieużywanej pamięci. W tej pamięci mogą znajdować się jakieś śmieci, które nie zostały wyczyszczone, ale też może być pusta. Jeżeli pusta, to a logicznie będzie miało wartość 0. Jeżeli nie to wartość a jest uzależniona od tego, co tam siedziało wcześniej w pamięci.

---

Jeżeli zdefiniujesz zmienne poza funkcją main() i jakąkolwiek inną funkcją, to wtedy zmienna jest inicjalizowana zerem.

Twój program działałby, gdybyś umieścił zmienne tak:

#include <iostream>
// #include <windows.h> a na co to tutaj? 0_0
 
using namespace std;
 
int a;
 
int main()
{
    int ilosc;
    int tab[10000];
 
    cin>>ilosc;
 
    for (int i=0; i<ilosc; i++)
    {
        cin>>tab[i];
        a+=tab[i];
        cout<<a<<endl;
    }
 
 
 
    return 0;
}

Nie należy jednak tak robić, bo zmienne globalne, to zło :-)

---

A ja właśnie nie mogę pojąć dlaczego? Wiem jedynie że wtedy taka zmienna ma już zaklepaną nazwę i żadna inna nie może jej użyć

---

To z nawą to też istotna sprawa, ale nie najważniejsza. Zmienna globalna to całkowite zaprzeczenie programowania Obiektowo Orientowanego. To jest niedopuszczalne, że jakieś funkcje korzystałyby z tej samej zmiennej, z czego każda funkcja może ją swobodnie modyfikować. Mało tego! Konwencje pisania klas w C++ mówi, że publiczne powinny być tylko i wyłącznie metody (funkcje składowe). Tylko.

To jeszcze nie wszystko. Czasami mamy potrzebę zdefiniować jakąś globalną stałą dla programu. Jakieś ważna wartość, z której korzystają inne funkcje, np.const int doba = 24;
To konwencja mówi, żeby unikać nawet takich zapisów i zamiast nich stosować funkcje:

int doba()
{
	return 24;
}

Więc jak widzisz, od globalizacji naprawdę wszyscy starają się uciekać. Największy zarzut przeciwko używaniu globalnych obiektów jest w programowaniu Obiektowo Orientowanym (OO). Cała ta technika polega na tym, że istnieją klasy, które porozumiewają się miedzy sobą i TYLKO ze sobą nie dając nawet takiej możliwości innym.

Są oczywiście niezbędne wyjątki jak chociażby strumienie "cout" i "cin". Są to obiekty zdefiniowane w zakresie globalnym, ale w tym przypadku ma to oczywiście sens :-)