commit#701
Conversation
|
@hexpriv Thanks for the PR. I can't see this project in the dashboard could you please check? 😊 |
Sorry, I was waiting for the code analysis to complete - which it never did. Have updated my profile with my GitHub username and submitted a link the pull request in the dashboard. |
There was a problem hiding this comment.
@hexpriv Thanks for submitting, I found a few issues when having a first look:
🔍The main issue is that users are not presented with a menu to choose the operation. The menu only offers Start, Random mode, Scoreboard, and Exit, while the Math Game requirement expects users to choose between the operations.
🔍Game history is also not implemented as required. The project asks for previous games to be recorded in a List, but the current implementation stores only scores in an int[] highScores = new int[10]. That loses useful history details such as game type, date/time, questions, or final score per game, and it is not a List.
🔍There is a bug with currentScore: it is initialized once at the top, incremented during games, but never reset in EndGame(). This means the next game starts with the previous game’s score instead of zero.
Could you please recheck the requirements and make the changes so we can review? 😊
2 participants
No description provided.