Skip to content

Purdyz MathGame - #788

Closed
purdyz wants to merge 10 commits into
the-csharp-academy:mainfrom
purdyz:main
Closed

purdyz wants to merge 10 commits into
the-csharp-academy:mainfrom
purdyz:main

Conversation

@purdyz

@purdyz purdyz commented Oct 18, 2026

Copy link
Copy Markdown

My first project PR 😮

@github-actions

Copy link
Copy Markdown

🚀 Project Submitted!

Thanks for submitting your project! I will review it soon.

📋 Please check the dashboard to make sure the project is marked as Pending.

⚠️ If it's not marked as pending, submit it manually using the URL for this PR.

@TheCSharpAcademy

Copy link
Copy Markdown
Collaborator

@purdyz Project approved! 😄✅ And well done for completing the challenges! I recommend you always do that, as it pushes you and makes you learn faster. Your project is clean, tidy, concise and to the point. And you're using only one method for all operations, great work!

🥋And congratulations! You got your Green belt!! 🟩🟩

Feedback
🔍️You can simplify boolean expressions: Instead of using operation == "Random" ? true : false, use operation == "Random" directly to improve readability and reduce unnecessary code.

🔍️Currently, the application displays "Press any key to return to main menu" even when the player chooses to exit. Consider displaying this message only when the game is still running.

🔍You can improve input validation: Instead of defaulting to Random mode or Easy difficulty when invalid input is entered, consider prompting the player again until they provide a valid choice. This gives users more control and improves the overall experience.

Overall great work!✋🏻Looking forward to seeing your next projects!

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