@dy0mber847

Else statement should also be omitted because we are using return inside if, no need to use else in this case

@times747

please continue this whats wrong series. please🙏

@TELUCODER

The best video on YouTube channel till now keep posting these sir

@ishaz11

I like this series. Please keep it up

@prateektomar4874

I appreciate the series!!

@oremichenjesumeichen7801

This series is great 👍

@being_aslam_tiger

Please Make a design pattern playlist. Like gang of four design patterns C#.

@pilotboba

I guess my first question would be, if you don't want to allow a string, why do you use nullable string?

I also don't see the routing definition for this. Also, I don't know if this is MVC5 or Core/Modern dotnet. So, I'm not sure how this would be called. Would it be:

/Home/Index?userid=   

or

/Home/Index assuming routing is used like /Home/Index/{userid}  then if no user id is in the URL it won't even be routed to this endpoint.

You can also use route constraints to eliminate the need to do the is empty check. Although, like I said, it looks like this is not using attribute routing.

Also, wouldn't use ==null, but rather do pattern matching. Granted you used the IsNullOrEmpty which covers the null check.

Finally, it's redundant to pass the view name if it matches the action name.

@cavila0217

Thanks for the tip!

@ajaymanav2010

It would be good to create some good content..these are the basics that you have coverd in your videos and i believe every developer is aware of it ..let's contribute with some real and practical interview scenarios that would help