626. Obżartuchy [GLUTTON]

[narbej]

Gratuluję!.

Może jeszcze parę uwag?

1. Unikaj zmiennych globalnych - zmienna ile prób.
2. Nie żałuj spacji - zobacz np mój ps 3
3. Jeszcze bardziej wymowne nazwy zmiennych
4. Zamisat for (.. tutaj można spokojnie while, np while(prob--){ ..
5. Zamiast: int doba=86400 lepiej const int doba = 24*60*60 // = 86400 obliczone w czasie kompilacji
6. Zamiast ostatniego bloku if ... else, można cout << ceil ((double) suma/ciastka) ......
   • + include cmath
     lub cout << (suma + ciastka - 1)/ciastka << ......
7. PS w C tak, w C++ w funkcji main() już nie jest konieczna instrukcja return 0.

[lukasz1303]

Dzięki za rady,na pewno z nich skorzystam i jeszcze raz dzięki za pomoc.

[ccoder]

Witajcie. Jestem aż tak zły, że muszę wam o tym napisać jaki głupi błąd zrobiłem przez co ciągle mi wyrzucało błąd kompilacji . Spędziłem dzisiaj pół dnia nad tym zadaniem , bo po napisaniu go raz i wyrzucaniu błędu skasowałem wszystko i od nowa napisałem no i bez zmian , ale doszedłem do tego , że nazwałem zmienną z czasem - “time” i za każdym razem błąd kompilacji przez taką pierdołę…

[narbej]

Kompilator zawsze wypisuje komunikat błędu a przy odpowiednim ustawieniu, także ostrzeżenia, więc wystarczy tylko przeczytać i od razu wiedzieć gdzie tkwi problem czy błąd. Tak jest, gdy kompilujesz u siebie lub na ideone. Gdy wysyłasz od razu na spoj’u, wystarczy “kliknąć” w napis błąd kompilacji przy swoim zgłoszeniu i jak wyżej.

[tarpauwatratar]

3600s -> 1 ciastko / h --> 24 ciastka / dobę. Resztę błędów w swoich obliczeniach znajdziesz sam.

[mariusz193]

przygotuj sobie test w pliku:
1
10000 1
1 … 1

oczywiście tych jedynek ma być 10000 a potem sprawdź wynik

[mddaniel]

Policzyłem na kartce, że pudełek wtedy wychodzi 864 000 000 000. Int zamieniłem na long long, a float na long double. Zadziałało! Dziękuję

[mariusz193]

to teraz usuń kod rozwiązania z forum

[puddignerg]

Mam problem z zaakceptowaniem przez sędziego mojego kodu. Informuje on o błędzie wykonania (SIGSEGV) . Proszę o pomoc i wskazówki jak mogę poprawić kod. Wyniki wychodzą prawidłowe.

#include <iostream>

using namespace std;

int n;

int main()
{
    int t; int suma=0; long m;
    cin>>t;

    for (int j=1; j<=t; j++){
        int tab[n-1]; const int x=24*60*60;
        cin>>n>>m;

        for (int i=0; i<n; i++){
            cin>>tab[i];
            suma+=x/tab[i];}

    double box = suma/m;
    if (box / m >0.1)
      {cout<<box+1<<endl;}
    else
      {cout<<box<<endl;}
    }
}

[mariusz193]

Mam problem z zaakceptowaniem przez sędziego mojego kodu. Informuje on o błędzie wykonania (SIGSEGV) . Proszę o pomoc i wskazówki jak mogę poprawić kod. Wyniki wychodzą prawidłowe.

SIGSEV najczęściej powodowany jest próbą zapisu/odczytu tablicy poza zakresem istniejących elementów (ogólnie jest to próba dostępu do pamięci poza dozwolonym obszarem dla programu)

zaś wyniki wcale nie wychodzą prawidłowe, sprawdź dla danych:

1
1 1
3600

[marcinmk]

Witam, w C++ programuję od niedawna, od jakiegoś czasu siedzę nad problem, nie wiem gdzie jest błąd, dla przykładowych danych wychodzą mi wyniki 9 i 6032237. Ktoś, coś? https://ideone.com/MFhYdl

[tarpauwatratar]

Program nie liczy (wszystkiego) dobrze w takiej postaci, co zresztą sugeruje błąd. Po kliknięciu w link opisu błędu dostaniesz pełen opis segfaulta. Wprowadzenie odpowiedniej poprawki w kodzie (wystarczy jedno zero) daje AC.

Być może pomoże Ci lektura treści zadania, a ściślej zakresów liczb.

Po AC usuń kod.

[persikk]

#include <iostream>

using namespace std;
int ile,grubasy,zjedzone,paczka,zjedzone2,ciastka;


int t[1000];
int pudla;


int main()
{

cin>>ile;
for(int i=1; i<=ile; i++)
{

cin>>grubasy>>ciastka;

for (int i=1; i<=grubasy; i++)
{


cin >>t[i];

zjedzone=86400/t[i];

zjedzone2=zjedzone2+zjedzone;

}

pudla=zjedzone2/ciastka;

   if (pudla*ciastka<zjedzone2) {pudla=pudla+1;}

zjedzone2=0;

cout<<pudla<<endl;

}

    return 0;
}

Powie mi ktoś co tutaj jest nie tak ? Cały czas błąd wykonania ;/ Program liczy wszystko dobrze

[tarpauwatratar]

Jakiej pomocy oczekujesz, skoro Twój kod nie działa nawet dla przykładowych testów? https://ideone.com/MSR70J

[jeremiasz0707]

Generalnie działa podaję dokładną liczbę dla pojedynczego obżartucha poniżej zaokrągla tą liczbę zsumuje też ilości obżartuchów dla danej próby.
Nie wiem dokładnie co jest nie tak, ale wydaje mi się, że błąd leży w tym, że zaokrągla po każdym obżartuchu, a powinien po każdej próbie, ale nie wiem jak tego dokonać. Już tydzień edytuje ten program i nie mogę nic wymyślić.
Bardzo proszę o pomoc.

[tarpauwatratar]

Generalnie nie działa. Wyjście, jakie masz mieć to 8 i 2, a nie cuda typu 0.00735467.

Domyślam się, że wyświetlasz jakieś obliczenia pomocnicze, ale nie oczekuj, że ktoś będzie wczytywał się w kod programu, który nie wyświetla tego co powinien i będzie analizował za co odpowiadają kolejne zmienne i jak wyniki pomocnicze mają się do ostatecznego.

Okomentuj albo usuń linie kodu, które wyświetlają cokolwiek innego niż określono w specyfikacji outputu.

EDIT
Rzuciłem okiem na wybrane fragmenty kodu i szczerze mówiąc kiepsko to widzę…

1. Nie używaj float do obliczeń zmiennoprzecinkowych, o ile nie jest to absolutnie konieczne
2. float obzartuchow[l]; - nie tak alokuje się tablice dynamiczne
3. Na SPOJu nie trzeba tablicować wyników
4. W tym zadaniu w ogóle nie potrzeba tablic
5. Nazwy zmiennych powinny być samoobjaśniające się. y, l oraz o zdecydowanie takimi nie są
6. Dlaczego tak mieszasz z zakresami ważności? Czy wiesz do czego służą klamry w cpp?
7. Po co Ci iomanip?
8. Dlaczego masz nieprawidłową indentację?

EDIT2

Już tydzień edytuje ten program i nie mogę nic wymyślić.

Dostosuj się do wymienionych powyżej punktów oraz uwag i jedziemy z koksem

[jeremiasz0707]

Wielkie dzięki za pomoc.
Tablice stworzyłem, bo nie wiedziałem jak zrobić aby program zapisywał wynik dla każdego kolejnego obżartucha.
Generalnie poczytam o wszystkim co mówiłeś i ruszam do boju
Jeszcze raz wielkie dzięki za pomoc

[jeremiasz0707]

Witam robię zadanie o obżartuchach:

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

Od miesiąca uczę się programować w c++ jednak utknąłem na tym zadaniu, od tygodnia je poprawiam i nie wiem co jest nie tak…
Bardzo proszę o wyrozumiałość.
Tutaj są moje wypociny :

http://wklej.org/id/3362965/

Dziękuje za wszelką pomoc i pozdrawiam

[j4rooo]

Silnik Discuss’a znów popsuł chronologię postów przy łączeniu wątków

[tarpauwatratar]

- Bo zaburza naturalną kolejność czytania.
- Dlaczego?
- Topposting.
- Co jest najbardziej irytujące na grupach? 

~ http://roflcopter.pl/2616.html