Symfony - tworzenie obiektu, który ma 3 relacje.

Question

Chciałbym dowiedzieć się czy sposób, w jaki tworze tego typu rzeczy jest poprawny, czy powinienem robić to inaczej.

Otóż, mam encje w bazie, która posiada relacje z trzema innymi encjami. Teraz chcąc usunąć tą encję z bazy, musze najpierw pobrać ją z repo, potem przekazać do entity Managera aby ją usunął.

Problem, w tym, że aby ją usunąłem musze najpierw zbudować trzy składowe obiekty.

 

            $kind = $kindService->makeKindObject($kindId);
            $type = $typeService->makeTypeObject($typeId);
            $property = $propertyService->makePropertyObject($propertyid);

                $ar = $availabilityRepository->findOneBy(
                    [
                        'advertKind' => $kind,
                        'advertType' => $type,
                        'property' => $property,
                    ]
                );
                $propertyLinkService->deleteLink($ar);

Jest to poprawne rozwiązanie?

Answer 1

Używaj prawidłowej nomenklatury. Wyciągając obiekty z bazy lepiej używać słów find, search, get. Ty ich nie tworzysz. Tworzy je orm na niższym poziomie.

Generalnie wygląda ok. Jeśli te 3 relacje tworzą kryteria po których chcesz szukać tego obiektu. Zawsze można ładnie użyć QueryBuildera w repo i wtedy nie trzeba angażować tych 3 serwisów.

No i wypada sprawdzić czy na pewno coś znalazłeś. 

Comments

Bardzo dziękuję za tą analizę :)