Pętla czasu (DateTime)

Question

Witam mam problem z tym kodem. Wie ktoś jak go przerobić aby program się nie wykolejał?

private void bStartOdliczania_Click(object sender, EventArgs e)
        {
            tStart = DateTime.Now;
            do
            {
                tPunktKontrolny = DateTime.Now;
                lPokazanyCzas.Text = tPunktKontrolny.Hour.ToString() + ":" + tPunktKontrolny.Minute.ToString() + ":" + tPunktKontrolny.Second.ToString();
            } while (((tPunktKontrolny.Hour * 3600 + tPunktKontrolny.Minute * 60 + tPunktKontrolny.Second) - (tStart.Hour * 3600 + tStart.Minute * 60 + tStart.Second)) < 30);

        }

 

Comments

a na czym to wykolenienie polega? Może wklej tutaj treść błędu, na pewno bedzie to pomocne.

Użyte zmienne zakładam, że globalne? Bo nie widzę ich deklaracji.

---

...

---

@fisherek, Nie było błędu tylko program przestawał działać na czas działania tej pętli (sprawdzałem w menedżerze zadań). Tak te zmienne są globalne typu DataTime. Jestem pewien na 100% że błąd jest wywołany tym kodem co go tu wkleiłem na posta

---

@Jakub Michalik,
Mógłbyś mi nieco przybliżyć co ta pętla ma robić? I jak ten przycisk ma działać z punktu widzenia aplikacji?

Answer 1

tylko program przestawał działać

Tak, bo zatrzymałeś pętlę zdarzeń.

Tak długo jak ten kod się wykonuje, nic innego na tym wątku nie może się wykonać - w tym pętla zdarzań, więc okno nie ma jak odpowiadać na zdarzenia.

Musisz albo:

• uczynić tą funkcję asynchroniczną (async/await) z przerwami, tak, że jej kod będzie się wykonywać naprzemiennie z pętlą zdarzeń,
• zamiast ręcznej pętli, podpiąć swój kod do timera (a kliknięcie buttona tylko włączy timer)
• wykonywać to na wątku (ale to też trochę komplikacji dodaje)

Comments

Dzięki za wytłumaczenie! Chyba zrobię to timerem. Miłego dnia;

Answer 2

Podczas wykonywania pętli bardzo dużo razy zostaje pobrany czas DateTime.Now, co bardzo mocno obciąża system. Sprawdziłem na dotnetfiddle.net (tam co prawda jest jest chyba ograniczenie wykonania programu do 10s bo po tym czasie zawsze mi kończyło program), ale pokzało spore zużycie RAM. Gdy dodałem w pętli Thread.Sleep(1000) to program elegancko śmiga.

Comments

Gdy dodałem w pętli Thread.Sleep(1000) to program elegancko śmiga.

Nie naprawi "zacięcia" okna winforms, bo pętla zdarzeń wciąż się nie wykonuje.

Ale zgadzam się, że ogólnie dodanie jakiejś formy odstępu czasowego się przyda - tylko nie takiej ;)