Использовать список общих интерфейсов

Question

Это сообщение основано в моем original post со вчерашнего дня. Но я не был достаточно точен относительно моих потребностей, поэтому я попробую снова здесь.

Пожалуйста, смотрите мой текущий код:

public interface IPosterGenerator<T> 
{ 
    IQueryable<T> GetPosters(); 
} 

public class PetPosterGenerator : IPosterGenerator<PetPoster> 
{ 
    IQueryable<PetPoster> GetPosters() 
    { 
     return busLogic.GetPetPosters(); 
    } 
} 

public class FlowerPosterGenerator : IPosterGenerator<FlowerPoster> 
{ 
    IQueryable<FlowerPoster> GetPosters() 
    { 
     return busLogic.GetFlowerPosters(); 
    } 
} 

public class PinBoard 
{ 
    protected List<IPosterGenerator> PosterGenerators { get; set; } // 1. compiler error 

    public PinBoard(List<IPosterGenerator> generators) // 2. compiler error 
    { 
     this.PosterGenerators = generators; 
    } 

    public List<BasePoster> GetPosters() 
    { 
     var posters = new List<BasePoster>(); 

     foreach (var generator in PosterGenerators) 
     { 
      posters.Add(generator.GetPosters()); 
     } 

     return posters; 
    } 
} 

Моя цель состоит в том, чтобы создать «Pinboard», который может возвращать список плакатов. Каждый плакат может быть другого типа (например, плакат для животных, плакат с цветами и т. Д.). Каждый плакат имеет совершенно другой вид, содержание и так далее. Но все они унаследованы от класса BasePoster.

В общем, у меня будет около 100 различных типов плакатов. Конкретный экземпляр PinBoard может легко содержать 1000 плакатов и более разных плакатов (в разном порядке).

Чтобы заполнить плакаты определенного PinBoard, мне нужно передать PinBoard список определенных генераторов плакатов (количество и тип генераторов будут меняться в зависимости от контекста). Будет один плакат-генератор для каждого типа плаката (например, PetPosterGenerator, который генерирует коллекцию плакатов для домашних животных).

Я думал, что было бы неплохо, если бы все мои генераторы плакатов могли использовать один и тот же (тип безопасного) интерфейса. Вот почему я представил интерфейс IPosterGenerator.

Моя проблема заключается в том, что код не компилируется. Есть две ошибок компилятора с тем же сообщением об ошибке: Использование универсального типа «myApp.IPosterGenerator» требуют 1 аргументов типа

Я не удивлен сообщением об ошибке, потому что я не определения любого типа в этих местах. Было бы замечательно, если бы я мог сделать что-то подобное в строке первой ошибки:

protected List<IPosterGenerator<T>> PosterGenerators { get; set; } 

Но когда я делаю это, компилятор не может найти тип или пространство имен Т.

Теперь я вид потерянного. Возможно, использование универсального интерфейса IPosterGenerator в конце концов не такая хорошая идея. Но я уверен, что в какой-то момент моего приложения мне нужно узнать или получить доступ к конкретному типу плаката, который представляет собой определенный IPosterGenerator.

Как вы, ребята, подходите к этому? Большое вам спасибо за вашу поддержку заранее.

Answer 1

Там нет волшебного решения. Если вы хотите использовать generics, , вам все равно нужно использовать их. Я имею в виду, что вы не можете их избежать.

В другой стороны, вы можете воспользоваться ковариации для того, чтобы позволить IPosterGenerator<T>T параметр принимает BasePoster:

// Check the "out" keyword in the generic parameter! 
// "out" makes the T parameter covariant. 
public interface IPosterGenerator<out T> where T : BasePoster 
{ 
    IQueryable<T> GetPosters(); 
} 

Теперь вы можете сделать это:

public class PinBoard 
{ 
    protected List<IPosterGenerator<BasePoster>> PosterGenerators { get; set; } 

    public PinBoard(List<IPosterGenerator<BasePoster>> generators) 
    { 
     this.PosterGenerators = generators; 
    } 

    public List<BasePoster> GetPosters() 
    { 
     var posters = new List<BasePoster>(); 

     foreach (var generator in PosterGenerators) 
     { 
      posters.Add(generator.GetPosters()); 
     } 

     return posters; 
    } 
} 

Так легко ! :)

Answer 2

Обычно вы определяете не общий интерфейс, а затем наследуете его общий интерфейс. Как так:

public interface IPosterGenerator 
{ 
    IQueryable GetPosters(); 
} 

public interface IPosterGenerator<T> : 
    IPosterGenerator 
{ 
    new IQueryable<T> GetPosters(); 
} 

Для реализации этого бы интерфейс вы могли бы сделать что-то вроде

public class PetPosterGenerator : 
    IPosterGenerator<PetPoster> 
{ 
    IQueryable<PetPoster> GetPosters() 
    { 
     return busLogic.GetPetPosters(); 
    } 

    // Explicit interface implementation on non-generic method 
    IQueryable IPosterGenerator.GetPosters() 
    { 
     // Invokes generic version 
     return this.GetPosters(); 
    } 
} 

Answer 3

Можно с обобщениями:

public abstract class BasePoster { } 
public class PetPoster : BasePoster { } 
public class FlowerPoster : BasePoster { } 

// NOTE the out keyword here. 
public interface IPosterGenerator<out T> where T : BasePoster 
{ 
    IQueryable<T> GetPosters(); 
} 

public class PetPosterGenerator : IPosterGenerator<PetPoster> 
{ 
    public IQueryable<PetPoster> GetPosters() 
    { 
     return Enumerable.Range(0, 5).Select(i => 
     { 
      return new PetPoster(); 
     }).AsQueryable(); 
    } 
} 

public class FlowerPosterGenerator : IPosterGenerator<FlowerPoster> 
{ 
    public IQueryable<FlowerPoster> GetPosters() 
    { 
     return Enumerable.Range(0, 5).Select(i => 
     { 
      return new FlowerPoster(); 
     }).AsQueryable(); 
    } 
} 

public class PinBoard 
{ 
    protected List<IPosterGenerator<BasePoster>> PosterGenerators 
    { 
     get; 
     private set; // fixes compiler warning #1 
    } 

    public PinBoard(List<IPosterGenerator<BasePoster>> generators) // specify the generic type, fixes compiler warning #2 
    { 
     this.PosterGenerators = generators; 
    } 

    public List<BasePoster> GetPosters() 
    { 
     var posters = new List<BasePoster>(); 

     foreach (var generator in PosterGenerators) 
     { 
      posters.AddRange(generator.GetPosters()); // call AddRange not Add 
     } 

     return posters; 
    } 
} 

Тогда это работает:

var generators = new List<IPosterGenerator<BasePoster>>(); 
generators.Add(new FlowerPosterGenerator()); 
generators.Add(new PetPosterGenerator()); 

var pinBoard = new PinBoard(generators);