Skip to content

Project is ready#721

Open
Solomonlol wants to merge 2 commits into
the-csharp-academy:mainfrom
Solomonlol:main
Open

Project is ready#721
Solomonlol wants to merge 2 commits into
the-csharp-academy:mainfrom
Solomonlol:main

Conversation

@Solomonlol

Copy link
Copy Markdown

No description provided.

@TheCSharpAcademy TheCSharpAcademy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Solomonlol Thanks for submitting! This project needs a bit of work before we can approve. Could you please fix the issues below?

🔍The main menu runs inside an infinite loop, but there is no option to exit the application. Please add an explicit exit option so the user does not have to close the console window manually.

🔍The application does not currently store or display previous games. A completed game should normally record information such as: Date played, Operation, Score

🔍Correct and incorrect answers are displayed, but the application does not keep a total score. Consider incrementing a score for every correct answer and showing the result when the game ends.

🔍The user can enter zero or a negative number of questions. In that situation, the game ends immediately. Validate that the number is greater than zero before starting the game.

🔍When the player enters text instead of a number, the question counter is still incremented. It would provide a better experience to keep asking until a valid integer is entered.

@Solomonlol

Copy link
Copy Markdown
Author

Okay, thanks for checking. I'll fix everything and try again :)

@Solomonlol Solomonlol marked this pull request as draft July 15, 2026 09:34
@Solomonlol Solomonlol marked this pull request as ready for review July 15, 2026 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants