IValidatableObject vs Single Responsibility

Question

IValidatableObject vs Single Responsibility

#1 da (6 voti)
#2 da (3 voti)
#3 da (2 voti)

11

Mi piace il punto di estensibilità di MVC, che consente ai modelli di visualizzazione di implementare IValidatableObject e di aggiungere la convalida personalizzata.

Cerco di mantenere i miei controllori snelli, visto che questo codice è l'unica logica di convalida:

if (!ModelState.IsValid)
    return View(loginViewModel);

Ad esempio, un modello di vista di accesso implementa IValidatableObject, ottiene l'oggetto ILoginValidator tramite l'iniezione del costruttore:

public interface ILoginValidator
{
    bool UserExists(string email);
    bool IsLoginValid(string userName, string password);
}

Sembra che Ninject, iniettando istanze nei modelli di vista non sia davvero una pratica comune, potrebbe anche essere un anti-pattern?

È un buon approccio? C'è uno migliore?

dependency-injection architecture validation asp.net-mvc single-responsibility

posta Boris Yankov 15.06.2012 - 14:13

fonte

3 risposte

Leggi altre domande sui tag dependency-injection architecture validation asp.net-mvc single-responsibility

Qual è la differenza dai modelli di sviluppo push e pull? Separazione di ASP.NET IdentityUser dalle altre mie entità

score 6 · Answer 1

Personalmente, per me il tuo design sembra pulito.

IValidatableObject significa che il modello di visualizzazione fornirà alcune convalide che non possono essere fornite da semplici attributi - iniettando il vero validatore che chiamerà servizi / database / qualsiasi cosa mantenga pulito il tuo progetto e ti garantisca non violare il principio di responsabilità unica - Visualizza i modelli sono responsabili, in sostanza, per il trasferimento di dati e la convalida dei dati che vengono trasferiti (sia tramite attributi o IValidatableObject o entrambi).

score 3 · Answer 2

Avere un oggetto dedicato per la convalida garantisce che si rispetti effettivamente SRP - che era già il caso in ogni caso dal momento che è una tipica responsabilità di un modello di vista per convalidare i suoi dati.

Per quanto riguarda l'iniezione di istanze in modelli di vista, non riesco a vedere nulla di sbagliato in questo. Non c'è praticamente alcun limite a ciò che può essere iniettato in cosa.

score 2 · Answer 3

Invece di iniettare ILoginValidator nel tuo costruttore VM, puoi usare ValidationContext (che è l'argomento di IValidatableObject.Validate ()) per ottenere il tuo Validator.

public IEnumerable<ValidationResult> Validate(ValidationContext vc)
{

var loginValidator = (ILoginValidator)vc.GetService(typeof(ILoginValidator));
return loginValidator.Validate();

}