1) Postaraj się powywalać jak najwięcej kodu z main do osobnych funkcji poniżej - zwłaszcza z bloków if i else.
2) Formatuj wcięcia - nie wiem czy masz taki układ czy ci się rozsypało przy wklejaniu na forum, jednak ilość tabów/spacji w lewym marginesie powinna ułatwiać czytanie kodu, nie utrudniać.
Źle:
if ((x=="c")||(x=="C"))
{
cout<<"Poprawna odpowiedz, przechodzisz do nastepnej rundy!"<<endl;
Sleep(1000);
cout<<"Nastepne pytanie... dziedzina wiedzy: Historia..."<<endl;
Sleep(1000);
cout<<"Stanislaw Antoni Poniatowski objal tron Polski, jednoczesnie bedac ostatnim krolem w wolnej elekcji w:"<<endl;
Sleep(1000);
cout<<"A. 1764r."<<endl;
Sleep(1000);
cout<<"B. 1777r."<<endl;
Sleep(1000);
cout<<"C. 1853r."<<endl;
Sleep(1000);
cout<<"D. 1790r."<<endl;
Sleep(500);
cout<<"Twoja odpowiedz: ";
cin>>x;
if ((x=="a")||(x=="A"))
{
cout<<"Swietnie! nastepna runda!"<<endl;
}
else
{
cout<<"Bledna odpowiedz, sprobuj jeszcze raz!";
}
}
else
{
cout<<"Niepoprawna odpowiedz, dziekuje za zagranie! The game was created by Tomek.";
}
Lepiej:
if ((x=="c")||(x=="C"))
{
cout<<"Poprawna odpowiedz, przechodzisz do nastepnej rundy!"<<endl;
Sleep(1000);
cout<<"Nastepne pytanie... dziedzina wiedzy: Historia..."<<endl;
Sleep(1000);
cout<<"Stanislaw Antoni Poniatowski objal tron Polski, jednoczesnie bedac ostatnim krolem w wolnej elekcji w:"<<endl;
Sleep(1000);
cout<<"A. 1764r."<<endl;
Sleep(1000);
cout<<"B. 1777r."<<endl;
Sleep(1000);
cout<<"C. 1853r."<<endl;
Sleep(1000);
cout<<"D. 1790r."<<endl;
Sleep(500);
cout<<"Twoja odpowiedz: ";
cin>>x;
if ((x=="a")||(x=="A"))
{
cout<<"Swietnie! nastepna runda!"<<endl;
}
else
{
cout<<"Bledna odpowiedz, sprobuj jeszcze raz!";
}
}
else
{
cout<<"Niepoprawna odpowiedz, dziekuje za zagranie! The game was created by Tomek.";
}
3) Poza drobnymi wyjątkami (jak np. iterator w pętli for) unikaj nazw zmiennych typu "x", "dupa", "zmienna". Lepiej ją nazwać np. "userAnswer" lub wybrać inną nazwę zgodną z twoją konwencją, ale zawierającą informację czym jest zmienna.