Skip to content

Add MathGame console app - #761

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

Add MathGame console app#761
paddymcn wants to merge 1 commit into
the-csharp-academy:mainfrom
paddymcn:main

Conversation

@paddymcn

Copy link
Copy Markdown

No description provided.

@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

@paddymcn Well done! Project approved. This Math Game meets the core expectations and is easy to follow. You’ve implemented all four operations, score tracking, input validation, game history, and a continuous menu loop.😊

🥋You still need to submit your Freecodecamp certificate before you start earning belts. 🤔

👩‍🏫Feedback

I highly recommend you learn about the DRY Principle (Don't Repeat Yourself). There's a fair bit of repetition in your menu as per picture below. Each operation contains almost the exact same game loop, history update, and score reset. That should ideally be moved into one reusable method such as PlayGame(string gameType):
image

🧐Challenges Review

I don’t see extra challenges such as difficulty levels, a timer, random game mode, or allowing the user to choose the number of questions. Those are optional, so they aren’t required for approval.

A good next challenge would be adding difficulty levels, because your current AskQuestion() method could easily use different number ranges depending on the selected difficulty.


📚I noticed that you haven't completed any of our courses yet. I suggest you try our Intro to Object Oriented Programming course next. This will close gaps in your knowledge and give you more confidence about OOP


👪Oh and if possible join our Discord Community and add your username to your profile so you can be part of the community as well.

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