kfoxIsProgrammer / Sysc3110_Risk

This is used for our Sysc 3110 Project. The game of risk
0 stars 1 forks source link

RiskModel is doing the job of the RiskView and RiskController #31

Closed MohamedZalat closed 3 years ago

MohamedZalat commented 3 years ago

So many methods such as menuBack and menuConfirm are present in RiskModel and it should not be the responsibility of RiskModel, Moreover, the methods are full of many cases. This is because they should have been done by a separate controller each, instead of the model itself.