开发者

How to return errors to view from controller not tied to a specific model property

开发者 https://www.devze.com 2023-01-13 12:18 出处:网络
Curious what the best practice is for returning errors to a view from a controller where the error isn\'t really a validation error, but more like \"user not found\" or \"service timeout\" type of err

Curious what the best practice is for returning errors to a view from a controller where the error isn't really a validation error, but more like "user not found" or "service timeout" type of errors. (in ASP.NET MVC2 framework)

I've been adding them to the ModelState's model errors, but that doesn't seem appropriate. (although easy to implement and maintain)

Example - A user tries to log in, and their credentials do not match a known 开发者_JAVA百科user.


I believe you need to be slightly more clear with regards to what youre calling an error.

Exceptions (timed out, database error, etc.) should be handled using specific exception handlers (not least because you often don't want to show the error message to the end user).

In this scenario, look at overloading the OnException() method on the controller. (or having all your controllers inherit from a common ControllerBase which has that method overriden to avoid repetition)

You may also want to check the value of filterContext.HttpContext.IsCustomErrorEnabled when handling exceptions to determine whether to expose detailed exception information or to obfuscate it.

Remember that exceptions should be for exceptional circumstances - "normal" operation should never result in an exception

The other type of error you're mentioning is more like "Unable to process request due to business rules / invalid input" (or similar) in which case, adding error messages to the ViewModel seems appropriate. They may not be direct validation errors but they are most likely as a result of the user input


I've just researched this, and I think that for network-related errors and so on, which are bound to happen, it's OK to use TempData (but don't put the Exception inside but the exception message only, for security reasons).

http://forums.asp.net/p/1460169/3353779.aspx

0

精彩评论

暂无评论...
验证码 换一张
取 消