Review - Githubissues

andrivg commented 5 months ago

Almennar athugasemdir

Spurning hvort sumar tilviksbreyturnar eiga að vera Final eða ekki. Stundum stinga sumir IDE uppá því, en munið að þetta er ákvörðun sem þið takið í kóðanum. Þið stjórnið IDE, ekki öfugt. Bara ábending, enginn frádráttur.
UML

Final breyturnar í BlackJack ekki teknar fram í UML. Ekki er sérstaklega tekið fram að það sé sleppt breytum fyrir lesanleika.
startGame er public, en sagt vera private í UML.
main aðferðin ekki í UML-inu.
PlayerInterface
Klasinn heitir PlayerInterface í kóðanum, en bara Player í UML-inu.
Það vantar getCards():List<Card> í UML.
Card
SUIT og VALUE í hástöfum í UML en ekki í kóða. Gott væri að hafa það í hástöfum báðum megin fyrst þetta eru fastar.
Vantar toString() fallið í UML.
Örvar

Ekki rétt ör á milli BlackJack klasa og Observer interface-inu. Notið samt rétta ör frá Player klasanum í Player interface.

ahp9 commented 5 months ago

Það er getCard(): List í UML-inu, það er önnur method í clasanum í UML

Wolfcoder13 commented 5 months ago

Það er ekki í interface-inu í UML-inu, en það er í interface-inu í kóðanum.