Przerobienie sekund na format GG:MM:SS

Question

Witam!

Otóż mam do zrealizowania takie zadanie:

A oto mój kod:

#include <iostream>
#include <string>
#include <algorithm>
#include <cstdlib>

using namespace std;

int main()
{
    string napis;
    getline(cin,napis);
    string szukaj=" ";
    size_t pozycja=napis.find(szukaj);

    int dlugosc=napis.length();

    string pierwsza=napis.substr(0,pozycja);
    string druga=napis.substr(pozycja+1,dlugosc-pozycja-1);

    int pier=atoi(pierwsza.c_str());
    int drug=atoi(druga.c_str());
    int czas=pier*drug;

    int gg=czas/3600;
    int mm=(czas-gg*3600)/60;
    int ss=czas-(gg*3600)-(mm*60);

    if (gg<10)
    {
        cout<<'0';
    }
    cout<<gg<<':';

    if (mm<10)
    {
        cout<<'0';
    }
    cout<<mm<<':';

    if (ss<10)
    {
        cout<<'0';
    }
    cout<<ss;

    return 0;
}

Problem w tym, że system sprawdzający poprawność kodu stwierdził, że program niepoprawnie liczy niektóre wartości. Proszę o pomoc w znalezieniu błędu, jeśli takowy istnieje.

Answer 1

Jeśli masz czas np.

3601 sekund

co oznaczałoby jedną godzinę i jedną sekundę, to robiąc

gg = 3601 / 3600

otrzymasz 1 i 1/3600. Czyli nie jedną godzinę, tylko godzinę z ułamkiem. To musisz naprawić.
Jednym z sensowniejszych rozwiązań byłoby podzielenie przez 3600, a także wyliczenie reszty z dzielenia przez 3600. Wtedy tę resztę można odjąć od godziny, aby otrzymać pełne godziny (żeby nie używać zaokrąglania), a następnie z tej reszty wyliczyć minuty. I dokładnie tak samo z sekundami.

Comments

Z tym, że z tego co mi wiadomo to wartość 1 i 1/3600 nie zapisuje się do formatu int. W int pozostaje tylko 1, chyba że się mylę.

---

Ah, no tak, zapomniałem, że to C++ i rzutowanie na typ.

Dobra, a nie możesz sobie odpalić tego programu sam i sprawdzić, czy dla wartości 3601 sekund dobrze wyświetla czas? Sprawdź też dla 7199

Poza tym właśnie rzutowanie na inta może powodować problemy. Mówię może, ale nie wiem. Ja po prostu nie ufam rzutowaniu liczb.

---

U mnie wszystko dobrze działa. Kwestia tego, że albo jest gdzieś błąd, albo to jest wina systemu. No ale innym osobom system zalicza to zadania.

---

A masz informację o tym, jakie wejścia nie przechodzą testu? Ułatwi to debugowanie.

A, moment. Wzór na czas to t = s / V, a ty tam coś mnożysz. Może o to chodzi?

---

Taką dostałem informację zwrotną:

---

Popraw wzór na czas tak jak napisałem chwilę wczesniej w komentarzu.

---

Tak, chyba chodziło o to, że t=s/v... Błąd kardynalny :D

---

Please yourself.

---

Kod przerobiony, wszystko działa poprawnie... Z tym, że teraz system zaliczył mi 60%...

Większość osób tak ma, a nie widzę problemu...

---

Rzutujesz czas na int?

int czas=pier*drug;

100 metrów przejechane z prędkością 30m/s to ile sekund? :D 3, czy 3,3? Chociaż raczej nie o to chodzi.

Swoją drogą duże wyniki Ci wychodzą względem obliczanych. Sprawdź, czy są gdzieś dostępne dane wejściowe, bo inaczej można gdybać. Powinny być dostępne.

---

Maksymalny czas, jaki może zostać osiągnięty przy takich zakresach danych to około 83 godziny. Co oznacza, że albo dane są skopane, albo coś się dzieje w pierwszych linijkach. Raczej to nie ma nic wspólnego z samym wyświetlaniem.

---

Dane wejściowe nie są dostępne, bo system sprawdzający wykonuje 10 prób i można by go było za pomocą if'ów obejść.

Answer 2

#include <iostream>

using namespace std;

void zamien( int sekundy )
{
	cout << "Godzin: " << sekundy / 3600 << endl;
	cout << "Minut: " << (sekundy % 3600) / 60 << endl;
	cout << "Sekund: " << (sekundy % 3600)%60 << endl;
}

int main()
{
	zamien(10000);
	
	return 0;
}