Проблема с генераторами С# - создание типичного типа с параметрами в конструкторе

Ответ 1

Ответ Jared по-прежнему остается хорошим способом - вам просто нужно сделать конструктор с помощью Func<PageData, T> и занести его позже:

public class HomepageCarousel<T> : List<T> where T: IHomepageCarouselItem
{
    private readonly Func<PageData, T> factory;

    public HomepageCarousel(Func<PageData, T> factory)
    {
        this.factory = factory;
    }

    private List<T> GetInitialCarouselData()
    {
       List<T> carouselItems = new List<T>();

       if (jewellerHomepages != null)
       {
            foreach (PageData pageData in jewellerHomepages)
            {
                T homepageMgmtCarouselItem = factory(pageData);
                carouselItems.Add(homepageMgmtCarouselItem);
            }
       }
       return carouselItems;
    }

Затем вы просто передаете функцию в конструктор, где вы создаете новый экземпляр HomepageCarousel<T>.

(Я бы рекомендовал композицию вместо наследования, btw... получение из List<T> почти всегда неправильным способом.)

Ответ 2

Рассматривали ли вы использование Activator (это еще один вариант).

T homepageMgmtCarouselItem = Activator.CreateInstance(typeof(T), pageData) as T;

Ответ 3

Просто добавьте к другим ответам:

То, что вы здесь делаете, в основном называется проекцией. У вас есть List одного типа и вы хотите проецировать каждый элемент (с помощью делегата) на другой тип элемента.

Итак, общая последовательность операций на самом деле (с использованием LINQ):

// get the initial list
List<PageData> pageDataList = GetJewellerHomepages();

// project each item using a delegate
List<IHomepageCarouselItem> carouselList =
       pageDataList.Select(t => new ConcreteCarousel(t));

Или, если вы используете .Net 2.0, вы можете написать вспомогательный класс, например:

public class Project
{
    public static IEnumerable<Tdest> From<Tsource, Tdest>
        (IEnumerable<Tsource> source, Func<Tsource, Tdest> projection)
    {
        foreach (Tsource item in source)
            yield return projection(item);
    }
}

а затем используйте его как:

// get the initial list
List<PageData> pageDataList = GetJewellerHomepages();

// project each item using a delegate
List<IHomepageCarouselItem> carouselList =
       Project.From(pageDataList, 
           delegate (PageData t) { return new ConcreteCarousel(t); });

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

[Изменить] Подумайте о следующем:

Я считаю, что теперь у вашего класса есть такой конструктор:

public class HomepageCarousel<T> : List<T>
    where T: IHomepageCarouselItem, new()
{
    private readonly List<PageData> jewellerHomepages;
    public class HomepageCarousel(List<PageData> jewellerHomepages)
    {
        this.jewellerHomepages = jewellerHomepages;
        this.AddRange(GetInitialCarouselData());
    }

    // ...
}

Я предполагаю, что это так, потому что вы обращаетесь к полю jewellerHomepages в своем методе (поэтому, я думаю, вы храните его в ctor).

С этим подходом существует несколько проблем.

• У вас есть ссылка на jewellerHomepages, которая не имеет значения. Ваш список - это список IHomepageCarouselItems, так что пользователи могут просто вызвать метод Clear() и заполнить его тем, что им нравится. Затем вы получите ссылку на то, что вы не используете.

• Вы можете исправить это, просто удалив поле:

  public class HomepageCarousel(List<PageData> jewellerHomepages)
  {
      // do not save the reference to jewellerHomepages
      this.AddRange(GetInitialCarouselData(jewellerHomepages));
  }
  

  Но что произойдет, если вы поймете, что можете захотеть инициализировать его с помощью другого класса, отличного от PageData? Прямо сейчас вы создаете такой список:

  HomepageCarousel<ConcreteCarousel> list =
       new HomepageCarousel<ConcreteCarousel>(listOfPageData);
  

  Вы оставляете себе возможность каким-либо образом создать экземпляр? Даже если вы добавите новый constuctor, ваш метод GetInitialCarouselData все еще слишком специфичен, чтобы использовать только PageData в качестве источника.

Заключение: не используйте конкретный тип в своем конструкторе, если это не нужно. Создайте фактические элементы списка (конкретные экземпляры) в другом месте.

Ответ 4

Это С# и CLR гандикап, вы не можете передать аргумент новому T(), простому.

Если вы исходите из фона С++, это используется НЕ-сломанно и TRIVIAL. PLUS вам даже не требуется интерфейс/ограничение. Ломать повсюду, и без этого функционального взлома factory 3.0 вы вынуждены выполнять двухпроходную инициализацию. Управляемое богохульство!

Сначала создайте новый T(), а затем установите свойство или передайте синтаксис экзотического инициализатора или, как и все предлагаемые, используйте функциональное обходное решение Pony. Все yucky, но что компилятор и представление времени для "generics" для вас.

Ответ 5

Существует другое решение, довольно грязное.

Сделать IHomepageCarouselItem методом "Construct", который принимает параметр pageData как параметр и возвращает IHomepageCarouselItem.

Затем сделайте следующее:

   T factoryDummy = new T();
   List<T> carouselItems = new List<T>();

   if (jewellerHomepages != null)
   {
        foreach (PageData pageData in jewellerHomepages)
        {
            T homepageMgmtCarouselItem = (T)factoryDummy.Construct(pageData);
            carouselItems.Add(homepageMgmtCarouselItem);
        }
   }
   return carouselItems;

Ответ 6

Вероятно, я поеду за предложением Тони "jon" пони Skeet, но есть другой способ сделать это. Поэтому в основном для развлечения здесь существует другое решение (у которого есть недостаток во время выполнения во время выполнения, если вы забудете реализовать необходимый метод, но не в том, чтобы не использовать метод factory, то компилятор волшебным образом подключит вас.

public class HomepageCarousel<T> : List<T> where T: IHomepageCarouselItem
{

    private List<T> GetInitialCarouselData()
    {
       List<T> carouselItems = new List<T>();

       if (jewellerHomepages != null)
       {
            foreach (PageData pageData in jewellerHomepages)
            {
                T homepageMgmtCarouselItem = null;
                homepageMgmtCarouselItem = homepageMgmtCarouselItem.create(pageData);
                carouselItems.Add(homepageMgmtCarouselItem);
            }
       }
       return carouselItems;
    }
}

public static class Factory
{
   someT create(this someT, PageData pageData)
   {
      //implement one for each needed type
   }

   object create(this IHomepageCarouselItem obj, PageData pageData)
   {
      //needed to silence the compiler
      throw new NotImplementedException();
   }
}

просто для того, чтобы повторить мое "отказ от ответственности", это очень важно, чтобы служить напоминанием о том, что могут быть довольно разные подходы к решению одной и той же проблемы, которые у всех у них есть, и сильные стороны. Один из обратных путей этого подхода состоит в том, что он частично черная магия;)

T homepageMgmtCarouselItem = null;
homepageMgmtCarouselItem = homepageMgmtCarouselItem.create(pageData);

но вы избегаете конструктора perculiar, принимающего аргумент делегата. (но я обычно придерживаюсь такого подхода, если только я не использовал механизм инъекции зависимостей для предоставления класса factory для меня. Наверное, это тот вид среды, в которой я работаю в свое свободное время; p)

Ответ 7

Почему бы вам просто не поставить статический метод "конструктора" на интерфейс? Немного хаки, я знаю, но ты должен делать то, что должен делать...