CSC207-2022F-UofT / course-project-monopolymen

Java Swing full recreation of Monopoly in the Clean Architecture
5 stars 3 forks source link

Card Deck Code Review #41

Closed SDunk1011 closed 1 year ago

SDunk1011 commented 1 year ago

For cardAction in PropertyRepairCard and AdvanceCard, to subtract player's money, there is a method called subtractMoney which you could use, because player may not have enough money to subtract. Besides the cardAction method in PropertyRepairCard is incomplete. you may complete it later. For the cardAction method in AdvanceCard, if collectGo is true, player may add money. For the cardAction method in GetOutOfJailCard, you may the method resetTurnInJail rather than enterJail. Also, you need to add Doc comments for your codes.