Шаблон "фабрика", но с параметрами объекта

Возьмите следующий классический шаблон "фабрика":

public interface IPizza
{
    decimal Price { get; }
}

public class HamAndMushroomPizza : IPizza
{
    decimal IPizza.Price
    {
        get
        {
            return 8.5m;
        }
    }
}
public abstract class PizzaFactory
{
    public abstract IPizza CreatePizza(ItalianPizzaFactory.PizzaType pizzaType);
}

public class ItalianPizzaFactory : PizzaFactory
{
    public enum PizzaType
    {
        HamMushroom,
        Deluxe,
        Hawaiian
    }

    public override IPizza CreatePizza(PizzaType pizzaType)
    {
        switch (pizzaType)
        {
            case PizzaType.HamMushroom:
                return new HamAndMushroomPizza();
            case PizzaType.Hawaiian:
                return new HawaiianPizza();
            default:
                throw new ArgumentException("The pizza type " + pizzaType + " is not recognized.");
        }
    }
}

Что, если один (или многие) Конкретных Пицц требует параметра, характерного для конкретной реализации в конструкции. Например, позволяет, говорят, что фабрика HamAndMushroom требует названного параметра, MushroomType и этот параметр были бы обязаны инстанцировать объекта?

21
задан Péter Török 28 July 2010 в 14:32
поделиться

6 ответов

Вам нужно будет добавить еще один метод CreatePizza () для этого фабричного класса. А это означало бы, что пользователи фабрики не смогли бы создавать такие типы пиццы, если бы они специально не использовали экземпляр класса HamAndMushroomPizzaFactory. Если у них просто есть ссылка на PizzaFactory, они могут вызывать только версию без параметров и не смогут создавать пиццу с ветчиной и грибами в целом.

0
ответ дан 29 November 2019 в 21:52
поделиться

Вы можете попробовать что-то вроде этого:

interface IPizza
{
}

class Pizza1 : IPizza
{
  public Pizza1(Pizza1Parameter p)
  {
  }
}

class Pizza2 : IPizza
{
  public Pizza2(Pizza2Parameter p)
  {
  }
}

interface IPizzaParameter
{
  object Type { get; set; }
}

class Pizza1Parameter : IPizzaParameter
{
  public object Type { get; set; }
}

class Pizza2Parameter : IPizzaParameter
{
  public object Type { get; set; }
}

static class PizzaFactory
{
  public enum PizzaType
  {
    Pizza1,
    Pizza2,
  }

  public static IPizza CreatePizza(PizzaType type, IPizzaParameter param)
  {
    switch (type)
    {
      case PizzaType.Pizza1:
        return new Pizza1(param as Pizza1Parameter);
      case PizzaType.Pizza2:
        return new Pizza2(param as Pizza2Parameter);
    }

    throw new ArgumentException();
  }
}

class Program
{
  static void Main()
  {
    var param1 = new Pizza1Parameter();
    var p1 = PizzaFactory.CreatePizza(PizzaFactory.PizzaType.Pizza1, param1);
  }
}

ИМХО концепция фабрики со специфическими параметрами реализации выглядит неправильно.

0
ответ дан 29 November 2019 в 21:52
поделиться

Вы можете добавить параметры к методам создания вашей фабрики. Однако, если количество параметров становится больше (для меня это было бы более 2-3), и особенно если некоторые или все эти параметры являются необязательными с разумными значениями по умолчанию, вы можете рассмотреть возможность преобразования фабрики в Builder вместо него.

Это может быть особенно подходящим для пиццы, у которой обычно одинаковая корочка, только с разными (комбинациями) начинок. Builder очень близко моделирует обычный способ заказа, например. «пицца с салями, помидорами, кукурузой и двойным сыром». OTOH для "предопределенных" пицц, возможно, вы захотите определить вспомогательные фабричные методы, например createMargaritaPizza или createHawaiiPizza , которые затем внутренне используют конструктор для создания пиццы с начинками, характерными для этого вида пиццы.

20
ответ дан 29 November 2019 в 21:52
поделиться

Вы можете использовать отражение:

using System.Reflection;

// ...

public override IPizza CreatePizza(PizzaType pizzaType, params object[] parameters) {
            return (IPizza)
                   Activator.CreateInstance(
                        Assembly
                             .GetExecutingAssembly()
                             .GetType(pizzaType.ToString()),
                        parameters);
        }
0
ответ дан 29 November 2019 в 21:52
поделиться

Во-первых, мне кажется странным, что абстрактный класс PizzaFactory содержит абстрактный общий метод CreatePizza , который принимает параметр более конкретного типа ItalianPizzaFactory.PizzaType .

Чтобы решить проблему, которую я только что упомянул, и проблему, изложенную в сообщении, я бы предложил следующий подход.

public struct PizzaDefinition
{
    public readonly string Tag; 
    public readonly string Name;
    public readonly string Description;
    public PizzaDefinition(string tag, string name, string description)
    {
        Tag = tag; Name = name; Description = description;
    }
}

public abstract class PizzaFactory
{
    public abstract IEnumerable<PizzaDefinition> GetMenu();
    public abstract IPizza CreatePizza(PizzaDefinition pizzaDefinition);
}


public class ItalianPizzaFactory : PizzaFactory
{
    public enum PizzaType
    {
        HamMushroom,
        Deluxe,
        Hawaiian
    }    

    public override IEnumerable<PizzaDefinition> GetMenu()
    {
        return new PizzaDefinition[] {
            new PizzaDefinition("hm:mushroom1,cheese3", "Ham&Mushroom 1", "blabla"),
            new PizzaDefinition("hm:mushroom2,cheese1", "Ham&Mushroom 2", "blabla"),
            new PizzaDefinition("dx", "Deluxe", "blabla"),
            new PizzaDefinition("Hawaian:shrimps,caramel", "Hawaian", "blabla")
        };
    }

    private PizzaType ParseTag(string tag, out object[] options){...}

    public override IPizza CreatePizza(PizzaDefinition pizzaDefinition)
    {
        object[] options;
        switch (ParseTag(pizzaDefinition.Tag, out options))
        {
            case PizzaType.HamMushroom:
                return new HamAndMushroomPizza(options);
            case PizzaType.Hawaiian:
                return new HawaiianPizza();
            default:
                throw new ArgumentException("The pizza" + pizzaDefinition.Name + " is not on the menu.");
        }
    }
}

Как видите, метод ParseTag () может иметь произвольную сложность, анализируя простой текст или зашифрованное значение. Или поле Tag может быть простым int, которое внутренне сопоставлено с некоторой таблицей рецептов пиццы, с целыми разными рецептами даже для слегка измененного содержания пиццы.

0
ответ дан 29 November 2019 в 21:52
поделиться

Вы можете передать новый параметр, например карту. И запросите свойства каждого конкретного конструктора. Тогда все методы будут иметь одинаковую сигнатуру. Однако в этом решении вызывающий конструктор должен знать конкретные свойства конкретного конструктора ... (Coupling)

1
ответ дан 29 November 2019 в 21:52
поделиться
Другие вопросы по тегам:

Похожие вопросы: