Obżartuchy (spoj) - crash

Question

Witam. Przedstawiony poniżej program crashuje mi (nie wywala mi przy tym żadnych błędów przy kompilacji) gdy wpisuje czas zjedzenia 1 ciastka do CzasJedz.

Treść zadania: http://pl.spoj.com/problems/GLUTTON/

#include <iostream>

using namespace std;

int main()
{
    int ile, IleObzart, IleCiasPudel, *CzasJedz, *IleCiasDoba, s = 0;
    cin >> ile;
    for (int i = 0; i < ile; i++)
    {
        cin >> IleObzart >> IleCiasPudel;

        CzasJedz = new int [IleObzart];
        IleCiasDoba = new int [IleObzart];

        for (int q = 0; q < IleObzart; q++ )
        {
            cin >> CzasJedz[q];
            for (int k = 0; k < IleObzart; k++)
            {
                IleCiasDoba [k] = 86400/CzasJedz [k];
                s+= IleCiasDoba [k];
            }
            if (s%IleCiasPudel==0)
                cout << s/IleCiasPudel << endl;
            else cout << s/IleCiasPudel+1 << endl;
        }
    }
    return 0;
}

 

Answer 1

U mnie wszystko działa, nic nie crashuje

Comments

Mi nadal crashuje. To, jakie dane podaję do programu nie ma znaczenia. Gdy przesyłam kod do SPOJ'a to wyskakuje mi błąd wykonania (SIGFPE)

Answer 2

a po co Ci ta pętla z ineksem k ? ?, tutaj masz błąd

przecież lecisz po ludkach w petli q

Answer 3

Wszystkie zmienne które przez które dzielisz zamień na flout.

edit: Dobra nieważne, nie zauważyłem że występują tylko liczby całkowite.

Answer 4

Dzięki za pomoc. Finalny, działający kod (punkcik na SPOJ!)

#include <iostream>

using namespace std;

int main()
{
    int ile, IleObzart, IleCiasPudel, *CzasJedz, *IleCiasDoba, s = 0;
    cin >> ile;
    for (int i = 0; i < ile; i++)
    {
        if (s > 0) s=0;
        cin >> IleObzart >> IleCiasPudel;

        CzasJedz = new int [IleObzart];
        IleCiasDoba = new int [IleObzart];

        for (int q = 0; q < IleObzart; q++ )
        {
            cin >> CzasJedz[q];
            IleCiasDoba [q] = 86400/CzasJedz [q];
            s+= IleCiasDoba [q];
        }
        if (s%IleCiasPudel==0)
        cout << s/IleCiasPudel << endl;
        else cout << s/IleCiasPudel+1 << endl;
    }
    return 0;
}

 

Answer 5

http://forum.pasja-informatyki.pl/90416/spoj-zasady-umieszczania-postow?show=90416#q90416

Answer 6

Pierwsze, co się rzuca w oczy, to zastosowanie new[] bez delete[]... Zero zwalnianej pamięci. Po drugie, w pętli k dzielisz przez komórki CzasJedz[k], które nie zostały jeszcze zapełnione. Potem masz ifa z modulo, czyli dzieleniem - istnieje ryzyko dzielenia przez zero.