search

sandeeplayam / Jump-In-Board-Game

Sysc 3110 project
2 stars 0 forks source link

Milestone 3 Feedback #4

Open RJBansal opened 4 years ago

RJBansal commented 4 years ago

Alot of errors when hitting Hint and many repainting issues

Design is overall good, however you need to clean it up

Action performed in Controller class is very messy. Split this up

Should be a unified undo, not one for each class

brougy commented 4 years ago

@RJBansal We put the hint button as a known issue since it was buggy, but Babak said it's ok if the button doesn't work properly.

We only added the undo to the fox/rabbit to implement the solver. We wanted to check if an object's possible move is an undo(therefore backtracking)by checking that object's last move rather than undoing the last move made in the board, which could be a different fox/rabbit.

Thanks for the rest of the feedback, we will get on top of it!

We were also wondering what did we lose marks on, in MileStone 2? Thanks again