Factory создать общие классы

Question

У меня есть абстрактный класс Validator:

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

И у меня есть несколько конкретных реализаций. Одним из них является AccountValidator:

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

Еще бы LoginValidator:

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

Теперь я хочу, чтобы создать фабрику, чтобы вернуть экземпляр реализации валидатора. Что-то вроде:

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

я тогда хотел бы сделать называть это как

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

Однако это не нравится возвращение нового AccountCreateValidator() линию, или тот факт, я объявляю myValidator, как Валидатор, а не Validator<SomeType>. Любая помощь будет оценена по достоинству.

Answer 1

Кажется, что вы используете фабрику для преобразования аргумента enum в конкретную реализацию проверки. Но я бы предположил, что, хотя вызывающий не знает или не заботится о конкретном типе валидатора, он, по-видимому, знает тип, который он хочет проверить. Это должно означать, что разумно сделать метод GetValidator общий метод:

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

Затем вызывающий код будет выглядеть следующим образом: Validator<IAccount> validator = ValidatorFactory.GetValidator<IAccount>(ValidationType.AccountCreate)

Answer 2

Если вы хотите, чтобы он использовался, как вы сказали, без указания общего параметра, вы можете объявить не общий интерфейс и сделать его абстрактным классом Validator. Непроверенные но что-то вдоль этих линий:

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(); 
     } 
    } 
} 

этот код:

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

Должно работать нормально.

Answer 3

Обычно я бы иметь завод, который принял Type как чтобы фабрика создавала уникальный производный тип. Но поскольку у вас есть несколько валидаторов, которые принимают один и тот же объект для проверки (в этом случае IAccount), я думаю, вам нужно будет предоставить общий параметр на вашем заводе, а также какой тип валидатора для создания, например:

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... 
     } 
    } 
} 

Я попытался назвать:

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

и это теперь возвращает AccountCreateValidator приведение к правильному Validator<IAccount> типа.

Это не идеальный вариант, так как теперь вам нужно знать, какой валидатор вы хотите, и какой ввод он принимает, но вы должны, надеюсь, получить ошибку компиляции, если передаете неправильный тип ввода из-за явного добавления я добавил, например. ValidatorFactory.GetValidator<IUser>(ValidationType.AccountCreate) не работает.

EDIT: Из-за комментария, говорящего, что это не скомпилируется, я отредактировал приведенный выше фрагмент кода, чтобы сделать литье как new AccountCreateValidator() as Validator<T>. Я думал, что это может быть брошено в любом случае, но, видимо, нет (не знаю, почему, хотя). Я подтвердил, что это работает в LINQPad, и я могу получить результат от валидатора.

Я также считаю, что мой предыдущий комментарий о возможной ошибке компиляции, если вы передаете неправильные общие типы, больше не стоит, поскольку литье с использованием ключевого слова as просто вернет null, если он не смог это сделать.