È consigliabile specificare esplicitamente il tipo di ActionResult restituito da un controller in ASP.NET MVC

Question

Sto usando ASP.NET MVC da un po 'di tempo e sembra trovarmi costantemente a restituire cose diverse da ActionResult dal mio controllori. Ovviamente restituisco ViewResults ma anche JSonResults e anche un paio di risultati personalizzati che abbiamo costruito in casa.

Mi chiedo tho, se, invece di dichiarare i miei metodi di controller come:

public ActionResult Index() 

dovrei iniziare dichiarandoli come

public ViewResult Index() 

o

public JsonResult Search() 

se So sempre che l'azione Index sul mio controller sarà sempre re girare un ViewResult o l'azione di ricerca sul mio controller restituirà sempre un JsonResult?

EDIT: Giusto per chiarire, sto parlando specificamente di situazioni in cui voglio sempre un tipo specifico di ActionResult da restituire.

Answer 1

Io voto si per due motivi.

1. Si dichiara esplicitamente che cosa si aspetta che il metodo restituisca e il compilatore prenderà qualsiasi tentativo di fare altrimenti. Non c'è bisogno di un test unitario che faccia un Assert (il risultato è ViewResult).

2. È necessario trasmettere il risultato al tipo previsto nei test quando si esaminano le proprietà univoche del tipo di risultato (ad esempio, controllo della proprietà Url di RedirectResult). Semplicemente dichiarando la variabile di prova come var si rimuove qualsiasi infrazione dovuta alla modifica dei tipi.

Answer 2

Vorrei lasciarlo come ActionResult generico. Se rendi il risultato specifico e lo cambi in seguito, alcuni, se non tutti, i test delle tue unità dovranno essere riscritti per adattarsi alla modifica. Ciò renderà le tue unità test più fragili di quanto debbano essere.

EDIT: inoltre, lasciandolo come ActionResult, ti concedi la possibilità di restituire risultati diversi in base alla tua logica di azione. Ad esempio, il normale flusso del tuo metodo potrebbe restituire un RedirectResult, ma potresti avere percorsi di errore che restituiscono un ViewResult o HttpUnauthorizedResult. Se si digita il metodo più fortemente del necessario in origine, si può finire per dover refactoring inutilmente e i test come si aggiungono risultati alternativi in ​​seguito.

La linea di fondo è che non vedo alcun vantaggio reale e, almeno una coppia, svantaggi.

Answer 3

Dichiarando un tipo di restituzione più specifico si ottiene un po 'più di controllo del tipo di compilatore che ora non è necessario coprire nei test di unità.

Si vincolerebbe te stesso, tuttavia, a quel tipo e dovresti tornare se questo cambia. Un esempio comune è se devi reindirizzare l'utente altrove in alcune condizioni speciali restituendo un RedirectResult.