Ciekawy problem, zegar liczący do 24h i potem ma się wyłączyć, prosty program

Question

Witam serdecznie wszystkich forumowiczów,

otóż napotkałem się z problemem, nie wiem dlaczego ta pętla nie trybi tak jak powinna. Wie ktoś może gdzie tu jest błąd? 

#include <iomanip>
#include <iostream>
#include <windows.h>
#include <cstdlib>

using namespace std;

int hrs=0;
int mins=0;
int sec=0;

void ShowClock() {
    system("cls");
    cout << "           "<< " CLOCK \n";
    cout << hrs << " Hours  ";
    cout << mins << " Minutes  ";
    cout << sec << " Seconds ";
}

void Clock() {
    for (int i=hrs; i<=24; i++) {
        ShowClock();
        Sleep(10);
        sec++;

        if (sec == 60) {
            mins++;
            if (mins == 60) {
                hrs++;
                mins = 0;
            } sec = 0;
        }
    }
}

int main() {
   Clock();

   return 0;
}

powinien program się skończyć, jeśli dojdzie zmienna hrs do 24'rech, a jednak zatrzymuje się, jeśli sekundy dojdą do 24. Z góry dzięki za pomoc ;)

Answer 1

Program nie bazuje na sekundach tzn nie kończy się na 24 sekundach, tylko 24 iteracjach 

int i=hrs; i<=24; i++

) tylko pętla przechodzi 24 razy  od 0 - > 24  bo nie zważając na operacje i dane po każdym przejściu masz i++ :) a i <= 24... stąd przejdzie od hrs=0 do i = 24 czyli 24 razy 
Każde rozpoczęcie pętli nie przywraca ci I do 0, i nie staruje iteracji od hrs.  Iteracja startuje raz  potem nie modyfikujesz zmiennej i, wiec ona jest podyktowana ilością operacji (i<=24) 

Możesz użyć pętli while hrs <24 np :) lub 

Comments

Kurcze coś w tym jest, ale może przez alkohol nie potrafię zrozumieć, trochę zabalowałem dziś, dam Ci naj i jutro to zbadam, bo wydaje mi się to proste i w miare rozumiem co napisałeś, ale ciężko mi wydedukować co zmienić, by to naprawić. W razie problemów jutro się odezwę, ale i tak dzięki ;)

---

Tak na marginesie ...
od i=0 do i<=24 masz 25 inkrementacji od 0, 1 ..., 24.  

#include <iostream>

int main()
{
    int i;
    
    std::cout << "i<=" << std::endl;
    for (i=0; i<=24; i++) {
        std::cout << i << " ";
        if (i == 9) std::cout << " ";
    }
        
    std::cout << std::endl;    
        
    for (i=0; i<=24; i++)
        std::cout << i+1 << " ";
       
    std::cout << std::endl << std::endl;

    std::cout << "i<" << std::endl;    
    for (i=0; i<24; i++) {
        std::cout << i << " ";
        if (i == 9) std::cout << " ";
    }
        
    std::cout << std::endl;    
        
    for (i=0; i<24; i++)
        std::cout << i+1 << " "; 

    return 0;
}

---

jeszcze nie wytrzeźwiałem do końca, ale tam i tak jest i=hrs, nie i=0 (nie bierz tego za złe mi, jeszcze napisałem nie mile, choć mam nadzieję, że to tak nie zabrzmiało)

---

dajcie mi czas do jutra, aż wytrzeźwieje, bo trochę ciężko dziś mi się myśli xd

---

Napisałem  

więc odniosłem się w ogóle do idei używania i< czy i<=.  

Answer 2

Według definicji sleep, przyjmuje wartość w sekundach, czyli "tik" w Twoim programie powinien być

sleep(1);
sec++;

żeby inkrementować zmienną sec co 1 sekundę.  

jeżeli inkrementujesz inne zmienne w oparciu o wartość zmiennej sec w tym konkretnym przypadku to

for (int i=hrs; i<=24; i++)

nie ma sensu

powinna zawierać np. dla 1h - 3600 sec. lub dla 24h - 86400 sec

pętla poniżej działa przez godzinę.  

#include <iostream>
#include <iomanip>
#include <windows.h>
 
using namespace std;

int main()
{
    int hrs=0, min=0, sec=0;
    
    for (int i=0; i<3600; i++) {
        system("cls");
    
        sec++;

        if (sec == 60) {
            sec = 0;
            min++;

            if (min == 60)
            {
                min = 0;
                hrs++;

                if (hrs == 24) hrs = 0;
            }
        }
    
        cout << setfill('0') << setw(2) << hrs << ":"
             << setfill('0') << setw(2) << min << ":"
             << setfill('0') << setw(2) << sec;
    
        sleep(1);
    }
    
    return 0;
}

albo pętla while  

#include <iostream>
#include <iomanip>
#include <windows.h>
 
using namespace std;

int main()
{
    int hrs=0, min=0, sec=0;
    
    while (true) {
        system("cls");
    
        sec++;

        if (sec == 60) {
            sec = 0;
            min++;

            if (min == 60)
            {
                min = 0;
                hrs++;

                if (hrs == 24) hrs = 0;
            }
        }
    
        cout << setfill('0') << setw(2) << hrs << ":"
             << setfill('0') << setw(2) << min << ":"
             << setfill('0') << setw(2) << sec;
    
        sleep(1);
        
        if (hrs == 1) break; // po 1h wypad ;-)
    }
    
    return 0;
}

using System;
using System.Threading;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            int hrs = 0, min = 0, sec = 0;
            Console.CursorVisible = false;

            while (true)
            {
                sec++;

                if (sec == 60)
                {
                    sec = 0;
                    min++;

                    if (min == 60)
                    {
                        min = 0;
                        hrs++;

                        if (hrs == 24) hrs = 0;
                    }
                }

                Console.Write("{0:00}:{1:00}:{2:00}", hrs, min, sec);                

                Thread.Sleep(1000); // c# Sleep(Int32 milliseconds)
                Console.Clear();

                if (hrs == 1) break;
            }
        }
    }
}

a tu bez użycia <windows.h>  

#include <iostream>
//#include <windows.h>
#include <curses.h>
#include <thread>
#include <chrono>
 
using namespace std;

int main()
{
    int hrs=0, min=0, sec=0;
    initscr();
    
    while (true) {
        //system("cls");
        clear();
    
        sec++;

        if (sec == 60) {
            sec = 0;
            min++;

            if (min == 60)
            {
                min = 0;
                hrs++;

                if (hrs == 24) hrs = 0;
            }
        }
    
        printw("%02d:%02d:%02d", hrs, min, sec);
        refresh();
    
        //sleep(1);
        this_thread::sleep_for (chrono::seconds(1));
        
        if (hrs == 1) break; // po 1h wypad ;-)
    }
    
    endwin();
    return 0;
}