Skip to content

AidanLDevs Maths Game submission - #746

Open
AidanLDev wants to merge 4 commits into
the-csharp-academy:mainfrom
AidanLDev:main
Open

AidanLDevs Maths Game submission#746
AidanLDev wants to merge 4 commits into
the-csharp-academy:mainfrom
AidanLDev:main

Conversation

@AidanLDev

Copy link
Copy Markdown

Console-based Maths Game project for The C# Academy code review.

Presents the user a menu to start a new game or view game history
Each game consists of 5 randomly generated questions across +, -, *, /
Division questions are constructed so they always resolve to an integer (divisor/quotient both 1–10)
Tracks score per game and stores game history in-memory across multiple games in a session

@AidanLDev

Copy link
Copy Markdown
Author

I accidently submitted this to the Calculator project! Please ignore that one, I have linked it to the correct Maths Game project too 😅

@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.

@AidanLDev Thanks for submitting! At the moment your project doesn't fulfill the following requirement:

❌ Users should be presented with a menu to choose an operation

Your menu contains:

1: Play new game
2: View game history

But once a game starts, the operation is selected randomly.

Could you please fix that so we can approve? Once fixed just commit the code, no need for a new pull request. 😊

@AidanLDev

Copy link
Copy Markdown
Author

@AidanLDev Thanks for submitting! At the moment your project doesn't fulfill the following requirement:

❌ Users should be presented with a menu to choose an operation

Your menu contains:

1: Play new game
2: View game history

But once a game starts, the operation is selected randomly.

Could you please fix that so we can approve? Once fixed just commit the code, no need for a new pull request. 😊

Ah yes thanks for the feedback, I've started going through the OOP course and Spectre.Console seems pretty neat, will have to add that in to create a nice menu.

Previously the operator was chosen randomly inside playGame, so the
top-level menu never actually controlled which operation was played.
@AidanLDev

AidanLDev commented Aug 26, 2026

Copy link
Copy Markdown
Author

Just pushed up a couple of commits, should be okay now 🤞 @TheCSharpAcademy

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