[masterix]

#php #symfony #symfony3 proszę o #codereview mojej próby stworzenia projektu w oparciu o Hexagonal Architecture i jednocześnie zrozumienia tej koncepcji. W przypadku encji poszedłem na skróty, ale z tego co oglądałem na konferencjach, to zdarzają się takie decyzje, które ułatwiają pracę (np Kris Wallsmith). Moja encja jest jednocześnie tą z domeny oraz z bazy danych. Jeśli z kodu wynika Wam, że czegoś nie rozumiem lub źle rozumiem to też proszę o wyjaśnienie :)
Kod: https://github.com/masterix/wiosna

[krejdd]

@masterix: daję 1% szans, że ktoś w tagu #php ogarnia architekturę heksagonalną :D.

[masterix]

@krejdd: A Ty ogarniasz :) Jeśli tak to proszę o zerknięcie w kod. Ew jaki tag jeszcze proponujesz?

[krejdd]

@masterix: patrzyłem w kod - nie ogarniam. Poczytałem jakieś pół godziny o implementacjach tej architektury w PHP i to za mało czasu, żeby ogarnąć co i jak. Tym bardziej, że nigdy nie pisałem niczego w oparciu o DDD.

[MacDada]

@masterix: Jestem w połowie code review. Generalnie jak jakąś uwagę dałem w jednym miejscu, to już w innych tego typu jej nie daję.

Ogólnie jest nieźle ;)

[MacDada]

daję 1% szans, że ktoś w tagu #php ogarnia architekturę heksagonalną :D

@krejdd: Pokaż swój kod to się pośmiejemy.

[Negass]

@masterix: Brak PHPDoc i type hintingu kuje w oczy. Wstrzykuj zależności do kontrolera, nie wyciągaj ich z kontenera.

[Mortale]

@masterix: nie nazwałbym tego co robisz architekturą heksagonalną. Brak wstrzykiwania zależności by móc nimi żonglować (a nam tym polega ta architektura, wyciągasz bloczek "wolny MySQL" i wkładasz za niego "szybki Elasticsearch"), beznadziejnie zaimplementowany CQRS (to nawet nie jest CQRS bo nie ma literki Q, nie ma literki C, pewnie nie ma również R i S, chyba, że zmienna $commandBus jest tylko z nazwy), brak separacji zmiennych (modyfikator publiczny, kiedy można dać

[MacDada]

@masterix: Dobra, poszły uwagi przy commitach, więc dyskusja o szczegółach tam. Jednej ważnej nie zapisałem: gdzie są testy? ;-)

[Mortale]

@MacDada:

przekazywanie encji do persist jest deprecated – będzie wywalone w nowym doctrine

rozwiniesz? nie śledzę zmian w Doctrine na bieżąco, jakbyś mógł to podeślij jakieś źródło.

[MacDada]

@Mortale: https://github.com/doctrine/common/issues/714

[masterix]

@Mortale: tak jak pisałem, nie jest to takie czyste HA. Chciałem wziąć część elementów, ale widocznie rozminąłem się z niektórymi rzeczami. Nie znalazłem przykładu, który by mnie bardziej nakierował jak to spiąć. Każdy przykład robił to inaczej, często były to tylko fragmenty, a nie pełne aplikacje. Dzięki za uwagi, przeanalizuję.

[MacDada]

@Mortale: I jeszcze to: https://github.com/doctrine/doctrine2/issues/6118

[masterix]

@Mortale: Widziałeś/czytałeś może książkę "Domain-Driven Design in PHP". Tam jest implementacja jeszcze prostsza wg mnie, nazwana również heksagonalną. Choć fakt, że tam prosto podmieniali np MySQL na Redis. Żeby to samo osiągnąć powinienem z EntryRepository zejść jeszcze niżej, już na poziom frameworka i tam konkretną implementację stworzyć - dobrze to rozumiem?

[masterix]

@MacDada: ok, czekam :)

[MacDada]

ok, czekam :)

@masterix: Już skończyłem ;)