fabryczne do tworzenia klas generycznych

Question

Mam abstrakcyjne klasy o nazwie Validator:

public abstract class Validator<T> where T : IValidatable 
{ 
    public abstract bool Validate(T input); 
} 

i mam kilka konkretnych wdrożeń. Jednym z nich jest AccountValidator:

public class AccountCreateValidator : Validator<IAccount> 
{ 
    public override bool Validate(IAccount input) 
    { 
     //some validation 
    } 
} 

Innym byłoby LoginValidator:

public class LoginValidator : Validator<IAccount> 
{ 
    public override bool Validate(IAccount input) 
    { 
     //some different validation 
    } 
} 

teraz chcę utworzyć fabrykę do zwrotu instancji implementacji walidatora. Coś jak:

public static class ValidatorFactory 
{ 
    public static Validator GetValidator(ValidationType validationType) 
    { 
     switch (validationType) 
     { 
      case ValidationType.AccountCreate: 
       return new AccountCreateValidator(); 
     } 
    } 
} 

bym wtedy lubią robić nazwać jak

Validator myValidator = ValidatorFactory.GetValidator(ValidationType.AccountCreate); 

Jednak to nie podoba się nowy AccountCreateValidator powrotną() linię, albo fakt ja deklarując myValidator jako Validator, a nie Validator<SomeType>. Każda pomoc będzie doceniona.

Answer 1

Wygląda na to, że używasz fabryki, aby przetłumaczyć argument enum do konkretnej implementacji sprawdzania poprawności. Ale wyobrażam sobie, że chociaż osoba dzwoniąca nie zna lub nie dba o konkretny typ walidatora, prawdopodobnie zna typ, który chce zatwierdzić. To powinno oznaczać, że jest to rozsądne, aby uczynić metoda GetValidator metoda rodzajowa:

public static Validator<TypeToValidate> GetValidator<TypeToValidate>(ValidationType validationType) where TypeToValidate : IValidatable 

następnie wywołanie kodu wyglądałby następująco: Validator<IAccount> validator = ValidatorFactory.GetValidator<IAccount>(ValidationType.AccountCreate)

Answer 2

jeśli chcesz, aby był używany tak, jak powiedziałeś, bez określania ogólnego parametru, możesz zadeklarować nie ogólny interfejs i sprawić, że klasa abstrakcyjna Validator zaimplementuje to. Niesprawdzone ale coś wzdłuż tych linii:

public interface IValidator 
{ 
    bool Validate(object input); 
} 

public abstract class Validator<T> : IValidator where T : IValidatable 
{ 
    public abstract bool Validate(T input); 

    public bool Validate (object input) 
    { 
     return Validate ((T) input); 
    } 
} 

public static class ValidatorFactory 
{ 
    public static IValidator GetValidator(ValidationType validationType) 
    { 
     switch (validationType) 
     { 
      case ValidationType.AccountCreate: 
       return new AccountCreateValidator(); 
     } 
    } 
} 

następnie ten kod:

IValidator myValidator = ValidatorFactory.GetValidator(ValidationType.AccountCreate); 

Powinna działać OK.

Answer 3

Normalnie musiałbym fabrykę który przyjął Type jako parametr tak, aby fabryka utworzyła unikalny typ wyprowadzony. Ale ponieważ masz wiele walidatorów, które akceptują ten sam rodzaj obiektu do sprawdzenia poprawności (w tym przypadku IAccount), myślę, że będziesz musiał podać ogólny parametr fabryki, jak również rodzaj walidatora do stworzenia, na przykład:

public static class ValidatorFactory 
{ 
    public static Validator<T> GetValidator<T>(ValidationType validationType) 
     where T : IValidatable 
    { 
     switch (validationType) 
     { 
      case ValidationType.AccountCreate: 
       return new AccountCreateValidator() as Validator<T>; 
        // etc... 
     } 
    } 
} 

próbowałem nazywając:

var value = ValidatorFactory.GetValidator<IAccount>(ValidationType.AccountCreate) 

a ten zwraca się teraz o AccountCreateValidator obsady do prawidłowego Validator<IAccount> typu.

Nie jest to idealne rozwiązanie, ponieważ musisz teraz wiedzieć, jaki to poprawny weryfikator i jakie dane wejściowe akceptuje, ale powinieneś mieć nadzieję, że wystąpi błąd kompilacji w przypadku podania błędnego typu wejścia z powodu wyraźnej obsady, którą dodałem, np. ValidatorFactory.GetValidator<IUser>(ValidationType.AccountCreate) powinien się nie powieść.

EDIT: z powodu komentarza mówiąc to nie skompilować, mam edytowany powyższy fragment kodu do zrobienia do obsady jako new AccountCreateValidator() as Validator<T>. Myślałem, że może być rzucany w obie strony, ale najwyraźniej nie (nie wiem dlaczego). Sprawdziłem to w LINQPad i mogę uzyskać wynik z walidatora.

Wierzę również, że mój poprzedni komentarz na temat błędu kompilacji mógł zostać przekazany, jeśli przekazałeś nieprawidłowe typy generyczne, które nie jest już używane jako rzutowanie, używając słowa kluczowego as, po prostu zwróciliby null, gdyby nie było to możliwe.