Skip to content

Chedilian Michaud's Math Game - #741

Open
Punkalo wants to merge 5 commits into
the-csharp-academy:mainfrom
Punkalo:main
Open

Chedilian Michaud's Math Game#741
Punkalo wants to merge 5 commits into
the-csharp-academy:mainfrom
Punkalo:main

Conversation

@Punkalo

@Punkalo Punkalo commented Aug 13, 2026

Copy link
Copy Markdown

No description provided.

@TheCSharpAcademy

Copy link
Copy Markdown
Collaborator

@Punkalo Thanks for submitting! This needs an issue in the dashboard before we can approve 😁.

@TheCSharpAcademy TheCSharpAcademy self-assigned this Aug 13, 2026

@Punkalo Punkalo left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Viewed all changes.

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

@Punkalo Thanks for submitting! There's a few requirements not being met. Please fix before we can approve:

Requirements Review
❌ At least five questions per game: The game ends based on lives rather than after a minimum of five questions. A player could lose all their lives before answering five questions.
⚠️ Addition: Easy mode currently generates addition questions only.
❌ Subtraction: Although Easy mode says it includes addition and subtraction, PerofmCalculationEasy() always performs x + y.
⚠️ Multiplication: Intended in Medium and Hard modes, but Medium mode currently never generates multiplication.
⚠️ Division: Division exists, but it doesn't follow the project's integer-division requirement. The project expects division questions to result in whole numbers rather than decimal answers.
❌ Operation menu: The player selects a difficulty rather than selecting Addition, Subtraction, Multiplication, or Division.
❌ Game history: There is currently no List storing previous game results and no option for the player to view their previous games.

Those core features need to be present before I'd approve the project. Let me know if you have questions. 😊

@Punkalo Punkalo left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Reviewed

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