[Vengard]

Czy można połączyć wzorzec dekorator z (chyba)strategią? Projekt związany z pizzerią.
Pizza ma listę składników, które są obiektami implementującymi interfejs Addition, i chciałbym, abym mógł dekorować pizzę każdym składnikiem, czyli coś w stylu:
public class Ham implements Addition extends AdditionDecorator
Ma to sens?
#java #programowanie

[kefaise]

@Vengard: nie lepiej, żeby pizza zawierała składniki jako listę obiektów typu Addition?

[kefaise]

@Vengard: nie lepiej, żeby pizza zawierała składniki jako listę obiektów typu Addition?

[Vengard]

@kefaise: tak jest zrobione, źle napisałem xD
private List listAddition;

[P.....s]

@Vengard:

private List listAddition;

No to już masz zrobione, że każdym składnikiem można to o co chodzi?

[Vengard]

@Philopolemus_Fronius:

Pizza pizza = new CheesePizza(list);
Pizza pizza = new Ham(pizza);

I z tego powinna wyjść pizza serowa z dodatkową szynką. I zastanawiam się, czy takie rozwiązanie jest dobre czy może coś za bardzo przekombinowuję.

[P.....s]

@Vengard:

Ten fragment kodu jest nieprawidłowy, redefiniujesz zmienne. Zmienna o nazwie pizza jest już utworzona.

[Vengard]

@Philopolemus_Fronius: Czemu nieprawidłowy? W taki sposób właśnie używa się wzorca dekorator. Pamiętaj, że AdditionDecorator dziedziczy od klasy Pizza.

[P.....s]

@Vengard:

Czemu nieprawidłowy?

Powiedziałem

[Vengard]

@Philopolemus_Fronius: aa, to tak, znowu źle zapisałem xD
powinno być:
Pizza pizza = new CheesePizza(); //bez tego list, już nie wazne to
pizza = new Ham(pizza);

[Kizio]

zapis pizza = new Ham(pizza) jest okropnie mylący. Przekombinowujesz. lepiej by było
pizza.add(new Ham()). Skoro chcesz w ten sposób dodawać składniki to może jakiś Builder?

[nilphilus]

@Vengard: ja tam bym sugerowl zrobienie tego jako FluentPizza ;-) chyba builder by pasowal. Pizza z metodami WithHam ktore dodaje ham do skladnikow i zwraca this.

[P.....s]

@Vengard:

Teraz jest błąd semantyczny.

Pizza pizza = new CheesePizza(); <- tworzysz zmienną typu pizza przechowującą obiekt typu CheesePizza. To jest ok
pizza = new Ham(pizza); <- nadpisujesz zawartość, więc CheesePizza jest usuwana a w jej miejsce wstawiany

[P.....s]

@nilphilus: Ja bym to zrobił tak jak pisze @Kizio czyli jakiegoś buildera zastosował i np połączył z fabryką jak masz już jakieś predefiniowane pizze i byłoby coś w rodzaju

Pizza myPizza = PizzaFactory.createPizza(CHEESE_PIZZA).withToppings(new Ham(), new Mushrooms(), new Pepperoni());

[P.....s]

@Philopolemus_Fronius: A, sorki, nie zauważyłem, że tam w parametrze przekazujesz obiekt typu pizza. W sumie nie baredzo bo to sens bo czemu szynka jest robiona na podstawie pizzy i czemu szynka jest zapisywana do pizzy

[P.....s]

@Vengard:

public class Ham extends AdditionDecorator implements Addition

Jeżeli masz to tak zrobione to nie możesz przypisać obiektu typu Ham do zmiennej typu Pizza

[Vengard]

Ogólnie przeczytałem książkę o wzorcach projektowych, i teraz staram się ją przerobić lepiej tworząc projekt, i używając ich w niej.
@Philopolemus_Fronius: klasa Ham miałaby raczej wygladać tak:
//AdditionDecorator dziedziczy z pizzy
public class Ham extends AdditionDecorator implements Addition {
Pizza pizza;
public Ham(Pizza pizza) { this.pizza =

[OverRide]

@Vengard: Ojjj o----ł bym taką z szynką czy salami (ʘ‿ʘ)

[P.....s]

@Vengard: Źle, nie możesz przypisać szynki do pizzy. ZObacz sobie jakiś przykład np. https://www.journaldev.com/1540/decorator-design-pattern-in-java-example

Możesz zrobić np tak.

Pizza pizza = new CheesePizza(new HamPizza())