Skip to content

Math Game - ready for review - #760

Closed
anathemaq wants to merge 1 commit into
the-csharp-academy:mainfrom
anathemaq:main
Closed

Math Game - ready for review#760
anathemaq wants to merge 1 commit into
the-csharp-academy:mainfrom
anathemaq:main

Conversation

@anathemaq

Copy link
Copy Markdown

Completed Math Game project for review.

Implemented:

  • Difficulty levels
  • Game timer
  • Random game mode
  • Unified game logic following DRY principle

@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

@anathemaq . Project approved! 😄✅ Nice work on the Math Game. The core functionality is implemented cleanly, and you’ve also gone beyond the basics with difficulty levels, a timer, a single method for all operations and a random-game option.

🥋And congratulations! You got your Green belt!! 🫒🫒 The Coding Tracker 👏

👩‍🏫Feedback

🔍️Your code is neat and well organized. The only thing I would say is that GetNumbers() creates a new Random instance every time a question is generated. Consider keeping a single Random instance and reusing it.

🧐Challenges Review

✅ Difficulty levels — Easy, Medium, and Hard are implemented, with different number ranges depending on the operation.
✅ Timer — Stopwatch measures the duration of each game, and the duration is saved in the game history.
✅ Random Game — The R option randomly selects one of the four operations.
➖ Configurable number of questions — Games currently always contain five questions. This would be a good next challenge: ask the player how many questions they want and pass that value into PlayGame().

🗺️ Suggested next step

I noticed that you already complete our OOP course. I'd highly recommend taking our LINQ Fundamentals course next. LINQ is one of the most important tools in modern C# and becoming comfortable with it will make your code much cleaner and more expressive.


💁‍♂️If you have any questions, the best way to get help is in our Discord, in the #get-help-with-projects channel.


☕If you like our roadmap, please consider buying us a coffee. We appreciate your help 🙂

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