Nie używaj zmiennych globalnych
http://www.learncpp.com/cpp-tutorial/4-2a-why-global-variables-are-evil/
Oddzielaj od siebie poszczególne bloki. Np. zamiast tak:
cout<<"6.Zapisz gre"<<endl;
cout<<"7.Wyjscie"<<endl;
cout<<"Wybor:";
wybor2=getch();
if((wybor2>='8')||(wybor2=='0'))
{
cout<<endl<<"Nie ma takiej opcji w menu!"<<endl;
system("pause");
}
Rób tak:
cout<<"6.Zapisz gre"<<endl;
cout<<"7.Wyjscie"<<endl;
cout<<"Wybor:";
wybor2=getch();
if((wybor2>='8')||(wybor2=='0'))
{
cout<<endl<<"Nie ma takiej opcji w menu!"<<endl;
system("pause");
}
Rozdzielaj kod na funkcje. Unikniesz dzięki temu też tego:
}//Petla
}//Budowa Koniec
}///Wioska
}///Nowa gra
}///Menu Glowne
}///Int main
Zamiast tego:
int poziom_drw,poziom_tar,poziom_kop,poziom_hut,poziom_kuz;
użyłbym struktur.
Cały kod rozdzieliłbym na kilka plików .cpp
}///Wioska
}///Nowa gra
}///Menu Glowne
}///Int main
Komentarz zaczyna się od dwóch slash'y.