Pętle typu while(true) w programowaniu

Question

Czy używanie nieskończonych pętli while (true), for (;;) to zła praktyka programistyczna? Pytam bo zastanawiam się nad przepisaniem kodu C++ w taki sposób by uniknąć niepotrzebnych deklaracji zmiennych i zrobić po prostu returna w odpowiednim ifie.

Comments

Nie bardzo widzę związek jednego z drugim.

Takie dziwne while czasami występują w makrach (#define) by w każdym kontekscie takie makro działało jako jedna instrukcja.

---

@whiteman808, na niejednej produkcji widziałem XD 

---

@whiteman808, Kiedy nieskończona pętla jest dobrym rozwiązaniem?                                       

Główna pętla programu lub serwera

Jeśli masz program działający w trybie ciągłym (np. serwer, system obsługi zdarzeń), pętla nieskończona to naturalne rozwiązanie:

while (true) { handleRequests(); }

Przykłady: pętle gier, serwery HTTP, systemy czasu rzeczywistego.

Pętle obsługujące zdarzenia (event loop)

while (true) {

    Event event = getNextEvent();

    if (event == QUIT) break;

    processEvent(event);

}

Przykłady: systemy operacyjne, GUI, game loop.

Makra i metaprogramowanie

W niektórych bibliotekach (np. w makrach preprocesora #define) nieskończone pętle są używane, aby zachować jednolitą strukturę kodu.

Kiedy unikać nieskończonych pętli?

Brak warunku wyjścia

Jeśli nie masz jasnego sposobu na przerwanie pętli, łatwo stworzyć zawieszenie programu (np. while (true) {} bez break).

Niepotrzebna alokacja CPU

Problem: Pusta pętla będzie zjadać 100% CPU:

while (true) {}  // Źle!
Rozwiązanie: Użycie sleep lub mechanizmu obsługi zdarzeń:

while (true) { std::this_thread::sleep_for(std::chrono::milliseconds(100)); }

Zastępowanie warunkowych pętli (while lub for)

Lepsze podejście:

while (condition) {
    doSomething();
}
Złe podejście:

while (true) {
    doSomething();
    if (!condition) break;
}

Czy warto zamieniać pętlę na return w if?

Jeśli Twoja pętla ma jeden warunek kończący, można użyć return, ale zależy od kontekstu.

Przykład: while (true) vs. return

Podejście z while (true):

void run() {

    while (true) {

        int x;

        std::cin >> x;

        if (x == 0) break;

        std::cout << "Liczba: " << x << "\n";

    }

}

Podejście z return:

void run() {

    while (true) {

        int x;

        std::cin >> x;

        if (x == 0) return;

        std::cout << "Liczba: " << x << "\n";

    }

}

W tym przypadku return upraszcza kod, bo nie musimy robić break, ale nie zawsze jest to najlepsze rozwiązanie.

 Nieskończona pętla jest OK, jeśli wiesz, jak ją przerwać (break, return, sleep).

Używaj while (true) tylko tam, gdzie to logiczne (pętle zdarzeń, serwery, gry).

 Zamiast while (true), lepiej używać while (condition), jeśli warunek jest znany wcześniej.

 Jeśli chcesz uprościć kod, return zamiast break może być dobrym wyborem, ale nie zawsze.

---

Takie pętle są też często stosowane w sterownikach różnych urządzeń, czyli w systemach embedded - program sterujący telewizorem lub pralką się nie kończy. Najwyżej się wyłacza zasilanie.

W wielu zacytowanych kodach - przede wszystkim należy właściwie wybrać rodzaj pętli. Skoro pętla ma się wykonać co najmniej raz, zamiast while () {}, bardziej stosowna będzie pętla do {} while (). Takie przerywanie pętli w środku (bądź co bądź naruszenie zasad programowania strukturalnego) należy zostawić na sytuacje awaryjne i wyjątkowe - raczej nie powinna to być normalna ścieżka wykonania programu. W C++ pętle można też przecież przerwać wyjątkiem - standardowym lub własnym.

Answer 1

Nie, są jak najbardziej dobre jeśli to najprostszy sposób zapisania logiki którą chcesz osiągnąć.

w taki sposób by uniknąć niepotrzebnych deklaracji zmiennych

To znaczy? W jaki sposób są niepotrzebne?

Comments

Zrobić while(true) zamiast sprawdzania bool success w warunku pętli

---

Jeśli masz na myśli zastąpienie

while (good) {
   ...
   if (costam)
        good = false;
}

przez

while (true) {
   ...
   if (costam)
        break;
}

To tak, to jest jak najbardziej OK.