-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
- Gra ciekawa, myślę że grywalna, kod stosunkowo dobrze zrefaktoryzowany na metody
- Brakuje pliku *.pro
- Jeśli Pan chce 50% i zal. to może zostają tak jak jest, ale sądzę (mam nadzieję), że ma Pan większe ambicje...
- Niestety w grze będzie trudno zastosować polimorfizm (ma Pan jednak automat wybierający ruch więc za to +), więc potencjał oceny określam na 80% po wprowadzeniu zmian:
a. bardzo razi mnie gdy w kodzie są metody:
bool mainLoopStep0();
bool mainLoopStep1();
bool mainLoopStep2();
bool mainLoopStep3();
bool mainLoopStep4();
bool playerLoopChoose0();
bool playerLoopChoose1();
bool playerLoopChoose2();
bool playerLoopChoose3();
bool playerLoopPlayer0();
bool playerLoopPlayer1();
bool playerLoopPlayer2();
bool playerLoopPlayer3();
to jest zawsze sygnał że coś jest nie tak... Proponuję dodać klasę gracza z metodami Step, Choose, każdy gracz powinien przechowywać swój zestaw kart, umożliwiać jego wyświetlanie, gracze powinni być przechowywani w klasie Game w pojedynczym kontenerze...
b. Proszę przygotować dokumentację w doxygenie opisując wszystkie pola i metody obiektów
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels