[SFML/C++]Snake-ocena kodu

Question

Witam, chciałbym pokazać wam prostego Snake. Jest to moja pierwsza gra, więc proszę o ocenę kodu. 

Link do Githuba:https://github.com/Curriosityy/Snake

Link do gry:https://drive.google.com/folderview?id=0B18n3GJaCdzyOWVDQVliY2JBZ1k&usp=sharing

Answer 1

• czemu wszystko jest publiczne?
• GameInfo powinno być namespacem.

• apple Apple;

  czemu takie odwrotne nazewnictwo? Czemu to w ogóle jest w klasie, jak jest używane tylko w jednym miejscu? To samo z "snakePart part;". To samo z "counter".

• std::vector<apple>::const_iterator appleIterator;

  Po co to? To samo z "snakePartIterator;"

• Jeśli destruktor jest pusty, to nie ma sensu go definiować,

• propablyDirection 

  Dlaczego "propably"? (i literówka)

• snake::changeDirection(int a)

  Skoro masz enum z kierunkiem, to możesz go tu użyć. No i "a" to słaba nazwa na zmienną.

• "snakeSize" jest niepotrzebny, bo snakePartArray zna swój rozmiar.

• GameState *gameState;

  Mogło być referencją.

Comments

1	std::vector<apple>::const_iterator appleIterator;

było mi potrzebne żeby użyć erase(), na jednym z tutoriali widziałem takie rozwiązanie do usuwania danych ze środka vektora.

Puste destruktory sam Visual Studio mi tworzył to ich nie usuwałem, bo może się przydadzą ;d.

Nie wiedziałem jak inaczej nazwać ten enum.

---

było mi potrzebne żeby użyć erase(),

OK, ale czemu go zdefiniowałeś w klasie, a nie w funkcji?

Nie wiedziałem jak inaczej nazwać ten enum.

...Direction? Skąd w ogóle to "probably"?

Answer 2

Hmm, dziwnie znajomy kod (a raczej jego szkielet i konstrukcja).

Kilka uwag:

• nazewnictwo klas: konwencja zakłada że nazwy klas rozpoczynają się z dużej litery, a więc: SnakePart, Game, Engine, ... .
• brak konsekwencji w nazewnictwie: jak wspomniałem powyżej większość klas ma nazwy rozpoczynające się z małych liter (podobnie nazwy plików), ale część rozpoczyna się z dużych, a więc: GameInfo, GameState; dotyczy się to także nazewnictwa obiektów: czasami rozpoczynają się małych liter, czasami z wielkich
• puste konstrukcje (niby masz obsługę błędu, ale  w if'ie w linii 8 game.cpp nic nie robisz w razie błędu)
• linia 17 game.cpp powinna być przeniesiona poza główną pętlę, limit fps wystarczy ustawić tylko raz

To tyle jeżeli chodzi o rzeczy, które jakoś rzuciły mi się w oczy (przydałaby się dokładna analiza)

Comments

co do szkieletu to inspiracją był mi twój tutorial

http://szymonsiarkiewicz.pl/obsolete/piszemy-gre-w-sfmlu/piszemy-gre-w-sfmlu-lekcja-1/#more-92

---

Tia, a ja raczej już nie polecam z niego korzystać