Jak mogę uskutecznić program?

Question

Cześć,

Jestem nowy i dopiero zaczynam programowanie w C++, dzisiaj napisałem prosty program symulujący działanie sygnalizacji świetlnej. Chciałbym, żebyście zerkneli i powiedzieli co mogę w nim ulepszyć?

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


using namespace std;

int petla;

int main()
{

while (petla<=1000)
{



    HANDLE kolor;
    kolor = GetStdHandle( STD_OUTPUT_HANDLE );

    for (int i=15; i>=0; i--)
    {
        SetConsoleTextAttribute( kolor, 2 );
        Sleep(1000);
        system("cls");
        cout<<"Sygnalizacja swietlna"<<endl;
        cout<<"ZIELONE "<<i<<endl;
    }
    system("cls");

    for (int i=7; i>=0; i--)
    {
        SetConsoleTextAttribute( kolor, 6 );
        Sleep(1000);
        system("cls");
        cout<<"Sygnalizacja swietlna"<<endl;
        cout<<"ZOLTE "<<i<<endl;
    }
    system("cls");

    for (int i=10; i>=0; i--)
    {
        SetConsoleTextAttribute( kolor, 4 );
        Sleep(1000);
        system("cls");
        cout<<"Sygnalizacja swietlna"<<endl;
        cout<<"CZERWONE "<<i<<endl;
    }


}



}

 

Answer 1

Mógłbyś pozbyć się zmiennej globalnej, w tak prostym programie raczej niewiele więcej da się zrobić (jakby nie patrzeć: prawie nic tu nie ma).

Answer 2

Wszystkie trzy pętle `for` różnią się tylko trzema elementami więc można by zamiast tego zrobić jedną funkcję, która przyjmuje te trzy elementy jako argumenty.

Answer 3

Dorzucając swoje 3 grosze, myslę, że można by było wrzucić całą zmiane do jednej pętli i ustawić ją tak, aby w zależności od if-ów generowała dany kolor, a jeszcze bardziej czytelnie wywoływałaby daną funkcję odpowiedzialną za dany kolor :)