Skip to content

Comments

Habit Tracker project#396

Open
natholder wants to merge 13 commits intoTheCSharpAcademy:masterfrom
natholder:master
Open

Habit Tracker project#396
natholder wants to merge 13 commits intoTheCSharpAcademy:masterfrom
natholder:master

Conversation

@natholder
Copy link

habit tracker done wahoo!

@andymartinez1 andymartinez1 self-assigned this Feb 20, 2026
Copy link
Collaborator

@andymartinez1 andymartinez1 left a comment

Choose a reason for hiding this comment

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

Hi @natholder. Great job with this project! Everything is looking good except for one issue:
When entering a value in the main menu that isn't 1-5, it closes the program. Instead of the default case exiting the program, you should have case "5" exit and the default to warn the user that the input is not valid. Please fix this issue so I can approve the project.

Other feedback


⚠️ For DateTime format:

I would suggest using (YYYY-MM-DD) since this is the international standard for representing dates and times (ISO 8601).


✅ The README was well thought out and included our requirements:

  • Project description
  • How to build and run the project
  • Thought process behind the project (What was hard? What was easy? What have you learned?)

⚠️ You should validate the entry first and then ask for new values only if the ID exists. As it is now, it asks for values before even checking if the ID is in the db Image

Other than that, everything is looking good. No unused imports or inconsistent code/formatting. Once you make that quick change to the main menu options, I will approve the project.

Best regards,
@andymartinez1

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