Skip to content

Project Finished! - #783

Closed
EternalWanderer7 wants to merge 4 commits into
the-csharp-academy:mainfrom
EternalWanderer7:main
Closed

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

Conversation

@EternalWanderer7

Copy link
Copy Markdown

I had a lot of fun revising my C# knowledge with this project. It showed me what I already knew and what I didn't know before. It also helped me learn how to use GitHub efficiently as well. Please review this critically so I can learn even more about C# and project development.

Copilot AI balanced review requested due to automatic review settings October 4, 2026 16:10

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Oct 4, 2026

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

@EternalWanderer7 Thanks for submitting! Before we can approve could you please fix this missing requirement?

Image
Image

Additionally a couple more improvements could be made:
🔍For better readability, clear the console more often or at least add some spacing as the UI is getting crammed.

🔍️ I highly recommend you learn about the DRY Principle (Don't Repeat Yourself) . There's a fair bit of repetition in the operations methods. You could have a single method for all operations, using variables for each operation.

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

@TheCSharpAcademy TheCSharpAcademy self-assigned this Oct 4, 2026
@EternalWanderer7

Copy link
Copy Markdown
Author

That should be good. Sorry it took so long; I had to study for midterms. Please review it when you can.

@TheCSharpAcademy

Copy link
Copy Markdown
Collaborator

@EternalWanderer7 Project approved! 😄✅ Thanks for making the changes and well done for completing the difficulty 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 shouldn't crash the app if menu validation fails. Put the menu in a loop and let the user try again. Also, Trim() and ToLowerCase() your input so I don't have to type exactly "Play" and "play" isn't invalid:

image

🔍️As you can see above you named your variable "a" and "b", which mean nothing. Make sure you name your properties properly to
clearly communicate their intent.

🔍For better readability clean the console more often, as it gets cluttered quickly.

🕵️‍♀️Also, have a look at our style guide since there are a few issues in your code. e.g. lots of unused usings, as per picture below. You can read our code conventions here🤓.

image

Overall good 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.

3 participants