626 Obrzartuchy (latwy C++)

[krzysztof777]

Witam wszystkich.
Wlasnie napisalem nastepujacy kod ktory nie przechodzi przez sedziego, prosilbym o udzilenie pomocy i wskazanie w czym tkwi blad, gdyz na pc program wykonuje sie poprawie. Oto kod:

#include

using namespace std;

int obe, pac;
int nr;

int main()
{
cin>>nr;
int tab[nr];

for (int i=0; i<nr; i++)
{
    int sum=0;
    cin>>obe>>pac;

    for(int j=0; j<obe; j++)
    {
        cin>>tab[j];
        sum=86400/tab[j]+sum;
    }
    if(sum%pac==0)
        cout <<(int)sum/pac<< endl;
    else
        cout<<(int)sum/pac+1<< endl;
}
return 0;

}

[krzysztof777]

ID: mojego kodu ktory jest nieaceptowany przez sedziego to 27468434, a blad to SIGSEGOV (segmentation fault).

[pwojszepielak]

int tab[nr];

1. W taki sposób nie deklaruje się tablicy w C++.

2. tabela nie powinna mieć długość nr tylko … i powinna być dynamiczna.

3. Jeśli zaczynasz, przeczytaj koniecznie! <--kliknij w to proszę!

[tarpauwatratar]

Uzupełnię:

1. kod źle wklejony (czyli pkt 3 @pwojszepielak, ale powtórzę, żeby lepiej dotarło ) .
2. po co Ci ta tablica? Przecież for po liczbie testów wykona się prawidłowo bez niej…
3. … może potrzebujesz tablicy w innym miejscu i innego rozmiaru? Przyjrzyj się zakresom w pętlach i zastanów czy na pewno nie wybiegasz poza zakres tablicy (pisanie poza dostępną pamięcią powoduje Twojego segfaulta) (czyli pkt 2 @pwojszepielak)
4. wiem, że działa gdy zadeklarujesz tablicę w taki sposób, ale w perfekcyjnym kodzie liczy się nie tylko czy działa, ale też czy jest zgodne z pisanymi i umownymi standardami klepania kodu. Programistycznej moładzioży zaleca się podczas nauki C++ zapoznać z dynamicznym alokowaniem pamięci i to jest świetna okazja na pierwszy kontakt z tym tworem (czyli pkt 1 @pwojszepielak)