Skip to content

Code Review Request#654

Open
leolin2810 wants to merge 7 commits intoTheCSharpAcademy:mainfrom
leolin2810:main
Open

Code Review Request#654
leolin2810 wants to merge 7 commits intoTheCSharpAcademy:mainfrom
leolin2810:main

Conversation

@leolin2810
Copy link

No description provided.

@TheCSharpAcademy TheCSharpAcademy self-assigned this Feb 21, 2026
Copy link
Owner

@TheCSharpAcademy TheCSharpAcademy left a comment

Choose a reason for hiding this comment

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

@leolin2810 Thanks for submitting! The project looks good except that score is broken. I got all answers right but my score was 0:


Image


Since we're here: A couple of things you could improve (not mandatory, but would be good).

🔍Scores at the moment show a block of "current settings". It would be better if it was a one liner per game: Date - Game Type, Score, etc.

🔍It's better to create one instance of Random and use it everywhere instead of creating one per operation. I'll let you research why.

🔍For tracking the time of a game I suggest you look into the Stopwatch class, provided by the .NET class library, instead of:

Image

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