626. Obżartuchy [GLUTTON]

Hej
Zrobiłem testy i wyszły, sam też nie widzę błędu w tym zadaniu. Jednak kompilator mówi mi, że jest błąd. Mógłbym wiedzieć o co chodzi? Z góry dzięki!

#include
#include

using namespace std;

int n,a[1000000],b; // zestaw testowy
int zaproszeni;
int pudelko, czas[1000000];
float c,lpud;

int main()
{
cout << "Podaj liczbê zestawów testowych: " << endl;
cin >> n;
for (int i = 0; i < n; i++)
{

cout << "Podaj liczbê zaproszonych ob¿artuchów: " << endl;
cin >> zaproszeni;
cout << "Podaj liczbê ciastek w pude³ku: " << endl;
cin >> pudelko;


for (int i=1; i<=zaproszeni; i++)
{
    cout << "Podaj czas jedzenia " << i << "ego ob¿artucha" << endl;
    cin >> czas[i-1];
}
for (int i=1; i<=zaproszeni; i++)
{
    cout << "Czas jedzenia " << i << "ego ob¿artucha to: " << czas[i-1] << endl;

}
 for (int i=1; i<=zaproszeni; i++)
{
    a[i-1]=86400/czas[i-1];
    cout << "Liczba ciastek " << i << "ego ob¿artucha to: " << a[i-1] << endl;
 b=a[i-1]/pudelko;
 c=c+a[i-1];
}

lpud = (c/pudelko);
cout << c << endl;
if ((int)lpud - lpud == 0)
cout << lpud << endl;
if ((int)lpud - lpud != 0)
cout << ceil(lpud) << endl;
c=0;
}

return 0;

}

Jak mogły Ci wyjść testy jeśli twierdzisz, że kod nie przeszedł kompilacji?
Twój kod jest bardzo zawikłany i niepotrzebnie skomplikowany. W tym zadaniu można sobie poradzić bez użycia tablic a Ty wprowadzasz dwie tablice. Po co rozpoczynasz licznik w pętli od i=1 a później dajesz indeks tablicy i-1? Jeśli licznik będzie startował od 0 to indeks tablicy i licznik będą zgodne (to nie jest błąd ale niepotrzebne zamieszanie). Lepiej nie używaj polskich znaków bo widzisz jak to wychodzi. Zresztą po co to wypisujesz? Zwróć uwagę w treści zadania co masz prawo (i obowiązek) wypisać. Wypisanie czegoś więcej jest błędem.

Dzięki za odpowiedź!
Zdaję sobie sprawę, że kod mógłby być prostszy - jest to mój praktycznie samodzielny pierwszy program (biorę udział w kursie C++ na stronie pana Mirosława Zelenta) i to jest takie pierwsze “poważniejsze” zadanie domowe. A kod skopiowałem prosto z Codebloxa i chodzi o to, że tam wprowadzałem przykładowe dane, przeliczałem i działało

Przemyśl co ma być na wyjściu programu (jest to w przykładzie) a co jest u Ciebie. Należy to potraktować DOSŁOWNIE

Innymi słowy wrzuć kod do Ideone, wrzuć mu przykładowe dane wejściowe do standardowego wejścia i uruchom program. Wyjdzie tak samo jak powinno?

Funkcję sum wywołuj tylko jeden raz i usuń parametr c, który jest zbędny. W treści zadania nie ma nic o sprawdzaniu poprawności wejścia. Usuń wszystkie takie konstrukcje:

 if ((1 <= quantityMonsters) && (quantityCookiesInBox <= 10000) && (1 <= quantityCookiesInBox) && (quantityCookiesInBox <= 1000000000))

Tyle powinno wystarczyć do AC. Możesz też zrobić to zadanie bez tablic i typów zmiennoprzecinkowych

BTW. W C++ raczej nie powinno się używać funkcji exit. Możesz zakończyć działanie funkcji main używając return.

Bardzo dziękuje, zadziałało. Wydawało mi się, że jeśli jest napisane jaki przedział liczbowy jest dla danej zmiennej to siłą rzeczy trzeba zrobić sprawdzanie poprawności.
Jeśli chodzi o to że mógłbym to zdanie zrobić bez tablic to, tak wiem, Tzn. później wpadłem na taki pomysł, ale pozostałem przy tablicy, bo chciałem trochę poćwiczyć.
Co do returna, tak. zmieniłem, ponieważ gdy za pierwszym razem wkleiłem kod z exitem to wyświetlił się komunikat o braku kompilacji. Zapomniałem o tym wspomnieć.
Jeszcze raz dziękuje za pomoc.

Przedział liczbowy to informacja względem której możesz wybrać typ zmiennej, lub algorytm itp. Istnieją zadania w których masz sprawdzać poprawność danych, ale wtedy jest to dokładnie opisane.
Błąd kompilacji mógł wystąpić, ponieważ nie dołączyłeś nagłówka < cstdlib > (tam znajduje się std::exit).

Ok, będę pamiętał
Hmmm… dziwne, w VS nie mialem cstdlib a i tak exit dział.
Nie mniej, jeszcze raz BARDZO dziękuje za pomoc!

Hej, mam problem z tym zadaniem.
Wiem, że kod móglby być łatwiej napisany, prosze o pomoc
https://pastebin.com/uxS9qFZH14
Pzdr

Popatrzyłem na Twój kod i się przeraziłem. 83 linie kodu, gdy można to zrobić w mniej niż 30. Nie zrozum mnie źle,nie krytykuję go. Tworzenie klas ma tutaj pewnie jakieś walory edukacyjne, ale bardzo wydłuża kod i obniża, moim zdaniem, czytelność programu. Sorry, nie byłem w stanie zmusić się, by go przeczytać. Ale możliwe, że znajdziesz innych chętnych.

Hej, męczę się z obżartuchami już trochę i nie umiem rozkminić, dlaczego moja odpowiedź jest niepoprawna, ktoś z Was będzie tak dobry coś mi zasugerować?

Dziękuję, i proszę o wybaczenie, jeśli coś przeoczyłem tu na forum.

2 rzeczy mi sie rzuciły w oczy:

1. Pomyśl o tym jak liczysz opakowania i czy na pewno dobrze to robisz.
2. Przy wypisywaniu przydałoby się coś jeszcze
   Po tych zmianach mam AC z twoim kodem. Powodzenia

Mam problem po dla podanych zestawow dziala a zielnej rameczki nie dostaje ;). PS. wulgaryzmy wiem zaraz przestawie wyglada to tak

#include

using namespace std;

long long int test_nr,ilosc_obazar,ciacha_wpudle,time_one_cookie;
long long int max_seconds=86400,all_cookies=0;

// liczy ile dany obzartuch opierdolil ciach
int opierdolone_ciacha(int seconds)
{
return max_seconds/seconds;
}

// How many boxes we need
int all_boxes(int cookies, int cookies_one_box)
{
if (cookies%cookies_one_box==0)
{
return cookies/cookies_one_box;
}
else
{
return (cookies/cookies_one_box)+1;
}
}

int main()
{
cin>>test_nr;

// main loop
for (int i = 0;i< test_nr; i++)
{
    cin>>ilosc_obazar>>ciacha_wpudle;
    for (int i=0; i<ilosc_obazar; i++)
    {
        // wczytac czas jedzenia 1 ciacha dla kazdego obzartucha
        cin>>time_one_cookie;
        // policzyc ile opierdoli
        all_cookies+=opierdolone_ciacha(time_one_cookie);

    }
    //cout<<"Lacznie ciach: "<<all_cookies<<endl;
    cout<<all_boxes(all_cookies,ciacha_wpudle);
    all_cookies=0;
}



return 0;

}

Kod jest zbyt wulgarnie wstawiony, żeby w ogóle na niego patrzeć.

#include

using namespace std;

int main()
{

int ilosc_testow,ilosc_obazar,ciacha_wpudle,time_one_cookie;
int max_seconds=86400,all_cookies=0;

cin>>ilosc_testow;

for (int i = 0;i< ilosc_testow; i++)
{
    cin>>ilosc_obazar>>ciacha_wpudle;
    for (int i=0; i<ilosc_obazar; i++)
    {

        cin>>time_one_cookie;
        all_cookies+=max_seconds/time_one_cookie;

    }

    if (all_cookies%ciacha_wpudle==0)
    {
        cout<<all_cookies/ciacha_wpudle;
    }
    else
    {
        cout<<(all_cookies/ciacha_wpudle)+1;
    }


    all_cookies=0;
}





return 0;

}

Proszę Bardzo z góry dzięki za odpowiedź. od razu jedna pytanie czy obojętne jest dla sędziego czy użyję dodatkowych funkcji czy też nie ?

Liczy się tylko poprawna odpowiedź (i sensowny czas wykonywania). Nadal wstawiłeś tak samo kod.