Obzartuchy spoj- problem z kodem

Question

Mam pytanko, a raczej prośbę o jakąś wskazówke. Dlaczego program w 1. części testu ze spoja dokonuje poprawnych obliczeń a w drugiej się wysypuje.

#include <iostream>
#include <cmath>

using namespace std;

int main()
{
    short ile;
    cout << "Ile testow: " ;
    cin>> ile;
    for (int i=1;i<=ile;i++)
    {
        double suma;
        int obzartuchy, pudelkociastek;
        cout<< "Ile obzartuchow na zlocie: "; cin>>obzartuchy;
        cout<< "Ile ciastek w pudelku: "; cin>>pudelkociastek;
        int czasy[obzartuchy];
        for(int j=1;j<=obzartuchy;j++)
        {
            cout<< "Podaj czas "<<j<<". obzartucha:";
            cin>>czasy[j];
            double x;
            x=floor(86400/czasy[j]);
            cout<< "Tyle ciastek na dobe: "<<x<<endl;

            double y;
            y=x/pudelkociastek;
            cout<< "Tyle pudelek na dobe: "<<y<<endl;


            suma+=y;


        }
        cout<<ceil (suma)<<endl;


    }

    return 0;
}

*wrzucając na spoja wyrzucam te couta i zbędne obliczenia ;)

Answer 1

short ile;

czemu short? integer bezpieczniejszy. Nie powiedzieli jak dużo testów może być.

int czasy[obzartuchy];

tak nie wolno. Długość tablicy musi być znana w momencie kompilacji. Działa tylko na niektórych kompilatorach. Napraw używając wektora lub dynamicznej alokacji.

int czasy[obzartuchy];
for(int j=1;j<=obzartuchy;j++)

wchodzisz poza zakres, tablica musi być na obzartuchy+1 jesli pętla startuje od 1.

double suma;
suma+=y;

użycie niezainicjowanej zmiennej, w której jest jakaś losowa wartość. Zainiciuj zerem.

Comments

Dzięki za pomoc, udało mi się to zrobić tak by spoj to zaakceptował. Przesyłam kod, jeśli dalej jest tam coś nie tak, to prosze o wskazówke ;D

 

#include <iostream>
#include <cmath>

using namespace std;



int main()
{
    int ile;
    cin>> ile;
    for (int i=1;i<=ile;i++)
    {
        double suma=0,x,y;
        int obzartuchy, pudelkociastek;
        cin>>obzartuchy;
        cin>>pudelkociastek;
        int*tab;
        tab=new int [obzartuchy];


        for(int j=0;j<obzartuchy;j++)
        {
            cin>>tab[j];

            x=86400/tab[j];
            y=x/pudelkociastek;
            suma+=y;
        }
        cout<<ceil(suma)<<endl;
        delete[]tab;


    }

    return 0;
}