Jakiś czas temu napisałem grę jako że cały czas się uczę proszę o konstruktywną krytykę mojego dzieła
Zasady: chodzimy biedronką "O" zbieramy mszyce "*" i jednocześnie uciekamy przez krwiożerczymi modliszkami "&"
Sterowanie ruch o jedno pole "wsad" zabiera 0.2 życia aby odnowić zbieramy mszyce
ruch o trzy pola "WSAD" można użyć shift lub caps lock zabiera nam 3 punkty życia
:)
umieszczam kod i proszę o wytykanie błędów :)
0
3
- Polskie nazwy
- u mnie jeden z plikow wyswietla regu+ey (jeszcze dziwne e jest po plusie)
jak sie programuje, to programuje sie w jezyku angielskim.
- niejednolistosc. Klasy/pliki raz piszesz z duzej raz z malej
- Magic numbers
- static
- Swiat to tylko x,y? No to sredni to ten swiat
- rozmiary swiatu powinno dawac sie w konstruktorze
- Main powinien robic najwyzej 5 instrukcji, jezeli masz wiecej to robisz cos zle (chodzi o architekture)
- losowe entery w losowych miejscach
- System.Threading.Thread.Sleep(60); po co na co?
- Duze funkcje, funkcje powinny robic JEDNA rzecz. Tak musisz stworzych architekture zeby kazda funkcja robila jedna rzecz (idealny przypadek)
- Wspomnialem o formatowaniu?
- wyglad.wyswietl(ref nr1, ref mod1, ref mod2, ref msz1, ref msz2, ref msz3, ref nowy ,ref anonim);
- nie trzeba tyle parametrow na pewno
- Lamiesz zasady DRY i KISS
- Teraz stworz 1000 mszyc a nie trzy
- Mszyca i swiat wygladaja indentycznie, tylko nazwa zmieniona...
- wspomnialem o formatowaniu kodu? Chyba tak...
0
Dzięki spróbuje dostosować kod do twych uwag
jak by ktoś jeszcze coś miał jeszcze do zarzucenia niech pisze śmiało im więcej tym lepiej :)
ps
System.Threading.Thread.Sleep(60); użyłem po to aby modliszki mi za szybko nie zmieniały pozycji i dały użytkownikowi czas na reakcje masz jakiś lepszy pomysł na rozwiązanie tego ?
jak bym zrobił większą mapę nie musiał bym tego używać bo samo rysowanie by dało wystarczające opóźnienie ale taka mapka mi pasowała i dla tego tego użyłem :)
Zarejestruj się i dołącz do największej społeczności programistów w Polsce.
Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.