Skip to content

Added solution for Car Crash problem - #5

Open
Poyam-ITT wants to merge 4 commits into
mainfrom
Assignment5
Open

Added solution for Car Crash problem#5
Poyam-ITT wants to merge 4 commits into
mainfrom
Assignment5

Conversation

@Poyam-ITT

Copy link
Copy Markdown
Owner

We have to consider a car crash scenario and implement exception handling.
Note: We have to consider all the maximum exceptions we could have with adequate error handling

Comment thread CarOperationHandler.cs Outdated
Comment thread Car.cs Outdated
Comment thread Car.cs Outdated
@Poyam-ITT
Poyam-ITT requested a review from mukulpalol16 April 2, 2024 14:43

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

all seems good to me

@Poyam-ITT
Poyam-ITT requested a review from Naman7877 April 3, 2024 03:18
Comment thread Car.cs
Comment thread CarOperationHandler.cs
Comment thread Car.cs Outdated
Comment thread CarOperationHandler.cs Outdated
Comment thread ExceptionHandler.cs Outdated
@@ -0,0 +1,81 @@
namespace CarApplication
{
public class ExceptionHandler

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.

ExceptionHandler is more like a generalized name. Rename it to something more specific.

@Poyam-ITT Poyam-ITT Apr 3, 2024

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Resolved

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.

7 participants