¿Es una buena práctica especificar explícitamente el tipo de ActionResult devuelto por un controlador en ASP.NET MVC

Question

He estado utilizando ASP.NET MVC por un tiempo y ahora parece que constantemente estoy devolviendo cosas que no sean ActionResult de mi Controladores Obviamente, devuelvo ViewResults pero también JSonResults y también un par de resultados personalizados que hemos creado en casa.

Me pregunto aunque, si, en lugar de declarar mis métodos de controlador como:

public ActionResult Index() 

debería empezar a declarar como

public ViewResult Index() 

o

public JsonResult Search() 

si Siempre sé que la acción de índice en mi controlador siempre será gire un ViewResult o la acción de Búsqueda en mi controlador siempre devolverá un JsonResult?

EDITAR: Solo para aclarar, estoy hablando específicamente de situaciones en las que siempre querré que se devuelva un tipo específico de ActionResult.

Answer 1

Voto sí por dos razones.

1. Está declarando explícitamente lo que espera que devuelva el método y el compilador detendrá cualquier intento de hacerlo. No hay necesidad de una prueba unitaria que haga un Assert (el resultado es ViewResult).

2. Debe arrojar el resultado al tipo esperado en sus pruebas al examinar cualquier propiedad exclusiva de ese tipo de resultado (por ejemplo, verificar la propiedad Url del RedirectResult). Simplemente declarando la variable de prueba como var se elimina cualquier fragilidad provocada por el cambio de tipos.

Answer 2

Lo dejaría como el genérico ActionResult. Si lo convierte en el resultado específico y lo cambia más tarde, algunas de las pruebas de su unidad, si no todas, deberán reescribirse para adaptarse al cambio. Esto hará que las pruebas de tu unidad sean más frágiles de lo que deberían ser.

EDIT: adicionalmente, al dejarlo como un ActionResult, se permite la capacidad de devolver varios resultados diferentes según su lógica de acción. Por ejemplo, el flujo normal de su método puede devolver un resultado de redireccionamiento, pero puede tener rutas de error que devuelven un resultado de revisión o HttpUnauthorizedResult. Si escribe su método con más fuerza de la necesaria originalmente, puede terminar teniendo que refactorizarlo innecesariamente y sus pruebas a medida que agrega resultados alternativos más adelante.

La conclusión es que no veo ninguna ventaja real y, al menos un par, desventajas.

Answer 3

Al declarar un tipo de devolución más específico, está ganando un poco más de comprobación del tipo de compilador que ahora no tiene que cubrir en pruebas unitarias.

Sin embargo, usted se vincularía a ese tipo, y tendría que revertir si eso cambia. Un ejemplo común es si debe redireccionar al usuario a otra parte en algunas condiciones especiales devolviendo un resultado de redireccionamiento.