Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TimeExtensions.cs #213

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

CoderAmbar
Copy link

Added null checks and exception handling to prevent runtime errors. Improved conditional logic in the IsBetween method. Added XML documentation comments for better code readability and maintainability. Improved string parsing in ToTimeOfDay with robust error handling.

Added null checks and exception handling to prevent runtime errors.
Improved conditional logic in the IsBetween method.
Added XML documentation comments for better code readability and maintainability.
Improved string parsing in ToTimeOfDay with robust error handling.
@joaomatossilva
Copy link
Owner

What if instead Time is changed to a Struct? I would prefer to not be able to be null rather than propagating null checks everywhere

@CoderAmbar
Copy link
Author

Ohh yess you're right we can do that, and also it would be really grateful if you give me credit as well

@joaomatossilva
Copy link
Owner

your username will forever be recorded on the commit history of this repo and joining the Contributers list

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