Program przestał działać - codeblocks

Question

#include <iostream>

using namespace std;

int main()
{
    int t, l1, l2;
    int wyniki[t];
    char znak;
    cin >> t;
    for (int i=0; i<t; i++)
    {
        cin >> znak;
        cin >> l1 >> l2;
        if (znak=='+')
        {
            wyniki[i]=l1+l2;
        }
        else if (znak=='-')
        {
            wyniki[i]=l1-l2;
        }
        else if (znak=='*')
        {
            wyniki[i]=l1*l2;
        }
        else if (znak=='/')
        {
            wyniki[i]=l1/l2;
        }
        else if (znak=='%')
        {
            wyniki[i]=l1%l2;
        }
    }
    for (int j=0; j<t; j++)
    {
        cout << wyniki[j] << endl;
    }
    return 0;
}

Po próbie uruchomienia wyświetla się komunikat -"Program przestał działać".  Koledze wszystko działa.

Wie ktoś gdzie jest błąd ?

zadanie:

http://pl.spoj.com/problems/CALC/

 

 

Answer 1

int t, l1, l2;
int wyniki[t]

Nie widzisz tu błędu? Deklarujesz tablice. O jakim rozmiarze? Nie wiadomo. Może zero, może dwa miliardy. (tak, deklaracja tablicy powinna być po inpucie do t) Btw. taka deklaracja tablicy (rozmiar będący zmienną) jest nie do końca okej - tzn. kompilator inny niż GCC ci na to nie pozwoli.

Comments

Dzięki już działa. Nie wiedziałem że na początku trzeba podać wartość zmiennej.

---

Inaczej przyjmuje taką wartość jaka akurat ułoży się z zer i jedynek w miejscu gdzie się pojawiła.

---

Btw. taka deklaracja tablicy (rozmiar będący zmienną) jest nie do końca okej - tzn. kompilator inny niż GCC ci na to nie pozwoli.

a jak inaczej można to zrobić aby tablica miała rozmiar który wpisze do zmiennej ? 

---

Niestety tylko alokacja dynamiczna :/ (albo użyć std::vector co ofc też sprowadza się do dynamicznej alokacji)

(nie mówię, że tak jak teraz robisz jest źle; standard tego zabrania, ale GCC troche nagina... no to czemu nie używać - ważne żebyś wiedział)