Zadanie c++, analiza kodu

pytanie zadane 27 stycznia 2019 w C i C++ przez Jatolubie Nowicjusz (120 p.)

Witam wszystkich użytkowników, przychodzę do was z trochę dziwnym przypadkiem aczkolwiek wujek google nie potrafił znaleźć odpowiedzi na moje pytanie.

Jestem na etapie czytania książki Stephana Prata Jezyk C++ Wydanie VI, aktualnie jestem na zadaniu 4 z rozdziału 3.

Nie każdy takową książkę posiada więc przepiszę treść zadania:

Napisz program, który poprosi użytkownika o podanie liczby sekund (zmienna typu long albo long long, jeśli są dostępne) jako wartości całkowitej (użyj typu long) i pokaże odpowiednik przekazanej liczby w dniach, godzinach, minutach i sekundach. Użyj stałej symbolicznej, aby zapisać liczbę godzin w dniu, minut w godzinie i sekund w minucie.

Wyniki powinny wyglądać następująco:

Podaj liczbę sekund: 31600000

31600000 sekund = 365 dni, 17 godzin, 46 minut, 40 sekund

----------------------------------------------------

Napisałem program najlepiej jak mogłem, będąc szczerym czas poświęcony na nie to zapewne 40minut jak nie lepiej, rozumiem że dla niektórych z was jest to wręcz banalne ale nie chcę podążać błędną ścieżka już na początku, sam nie wiem czy mój tok myślenia jest zły ale bynajmniej dla mnie program nie wygląda jak powinien wyglądać.

Proszę o wytknięcie każdego błędu, oraz wytłumaczenie czy projekt jest napisany prawidłowo według treści zadania.

Dziękuję niżej załączam kod do programu:

#include "pch.h"
#include <iostream>
#include <cmath>

using namespace std;

int main()
{
    long long sekundy;
    cout << "Podaj liczbe sekund:";
    cin >> sekundy;

    long long nagodziny = sekundy / 60 / 60;

    long long dni = nagodziny / 24;

    long long godzin = nagodziny % 24;

    long long minut = sekundy % 3600 / 60;
    long long sekund = sekundy % 60;

    cout << sekundy << " sekund = " << dni << " dni, " << godzin << " godzin, " << minut << " minut, " << sekund << " sekund";

2 odpowiedzi

odpowiedź 27 stycznia 2019 przez m4sk1n Pasjonat (16,750 p.)

Nie użyłeś tych stałych symbolicznych. Poza tym, następnym razem gdy będziesz wstawiał kod na forum, użyj opcji z ostatniej ikony.

komentarz 28 stycznia 2019 przez Jatolubie Nowicjusz (120 p.)

Dzięki, zmieniłem kod i dodałem stałe symboliczne, następnym razem użyje ramki do kodu pozdrawiam.

odpowiedź 27 stycznia 2019 przez The Świstak-Hrustak Początkujący (400 p.)

Dla mnie wygląda ok. Jedyne co, to że w poleceniu była mowa o stałych tzn powinieneś zadeklarować na początku stałe np const int ILOSC_GODZIN_W_DNIU = 24; a następnie kazde wystąpienie liczby 24 zastąpić ta właśnie stałą. No i biblioteka cmath zdaje się być zbędna w tym przypadku.

komentarz 28 stycznia 2019 przez Jatolubie Nowicjusz (120 p.)

Szczerzę powiem że nie bardzo rozumiałem o co chodziło ze stałymi, własnie zaktualizowałem nimi kod i teraz jest okej, dzięki za pomoc.

cmath znalazł się tam podczas szukania błędu ;p.