Symfony - wydzielenie operacji na Entity (logika biznesowa) z kontrolera

0

Witam,

na większości stronach internetowych z fragmentami kodu źródłowego, Entity jest załączony do kontrolera, np.

<?php
namespace AppBundle\Controller;

use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Request;

use AppBundle\Entity\Dzierzawca;
 

Część operacji Read (select) i Remove (delete) umieszcza się w EntityRepository, czyli do innego pliku, natomiast dołączanie Entity do kontrolera powoduje że wszystkie operacje związane z Create i Update umieszczane są w kontrolerze... (może się mylę)

Z jakiego wzorca projektowego powinienem skorzystać, żeby operacje na encjach były w osobnym (osobnych) plikach? Jak "przenieść" logikę biznesową (czy też CU - create update) poza kontrolery? Dodam, że poza "grubym" (nieczytelnym) kontrolerem problem pojawia się, gdy muszę pisać podobne funkcje do różnych kontrolerów...

Przykładowy fragment kontrolera (który chciałbym umieścić w osobnym pliku):

        public function addNewAdres($idDzierzawcy,$ulica, $mieszkanie, $kodPocztowy, $miasto)
        {
    	$emt = $this->getDoctrine()->getEntityManager();
    	$dzierzawca = $emt->getRepository('AppBundle:dzierzawca')->find($idDzierzawcy);
            $newAdres = new Adres();
            $newAdres->ulica = $ulica;
            $newAdres->mieszkanie = $mieszkanie;
            $newAdres->kodPocztowy = $kodPocztowy;
            $newAdres->miasto = $miasto;
            $newAdres->dzierzawca = $dzierzawca;
     
            $em = $this->getDoctrine()->getManager();        
            $em->persist($newAdres);
            $em->flush();
            
            return $newAdres->getIdAdresu();
        }

... mam nadzieję, że opisałem problem czytelnie... z góry dziękuję za wszelkie wskazówki...

1

A nie możesz tego wydzielić do jakichś service'ów ? http://symfony.com/doc/current/book/service_container.html
BTW. masz polskie i angielskie nazwy zmiennych - nie rób tak.

0

Dzięki - możliwe, że to mnie ukierunkuje - widzę dużo materiałów w sieci... tylko rozgryzę i może zamknę temat.

Taki swobodny offtopic (jakby chciał ktoś miał wenę opowiadać)... czy w projektach opartych na Symfony istnieje potrzeba wdrażania własnych wzorców, czy raczej istnieją do każdego z nich "gotowce" (nie wiem jak nazwać)? Cały czas myślę, żeby "nauczyć się" stosować wzorce projektowe, ale tutaj znowu zabrakło "pretekstu"...

1

Witam,

Prosze przeczytac dobre praktyki w symfony a duzo problemow zniknie (to tak na wstepie :) ).

Kontroler powinien byc wylacznie posrednikiem pomiedzy encjami, logika biznesowa itp.
Usuniecie kontrolera nie powinno wplywac na dzialanie aplikacji (logiki) oraz dzieki usluga aplikacja robi sie testowalna.

Wracajac do pytania to juz odpowiadam.

Najlepiej utworzyc sobie usluge (service), do ktorej dodajemy obsluge doctrine (Entity Manager poprzez DI) i nastepnie w kontrolerze wywolujemy usluge.

  1. Utworzmy sobie w bundlu katalog na nasz kod np. AppBundle\Services
  2. Tworzymy nowy plik klasy np. kontrahent.php
  3. Tworzymy w nim interesujaca nas klase i metode
<?php
    namespace AppBundle\Services;
    use AppBundle\Entity\Adres;

    class kontrahent
    {
        private $em;
        public function __construct($em)
        {
            $this->em=$em;
        }     

      public function addNewAdres($idDzierzawcy,$ulica, $mieszkanie, $kodPocztowy, $miasto)
        {
           try{
            $newAdres = new Adres();
            $newAdres->ulica = $ulica;
            $newAdres->mieszkanie = $mieszkanie;
            $newAdres->kodPocztowy = $kodPocztowy;
            $newAdres->miasto = $miasto;
            $newAdres->dzierzawca = $this->em->getReference('AppBundle:dzierzawca',$idDzierzawcy);

            $this->em->persist($newAdres);
            $this->em->flush();
 
            return $newAdres->getIdAdresu();
             }catch (\Exception $e){
             return false;
             }
        }
    }
  1. Dodajemy usluge do pliku konfiguracyjnego YAML, domyslnie app/config/services.yml
services:
    app.kontrahent:
        class: AppBundle\Services\kontrahent
        arguments: [ '@doctrine.orm.entity_manager']
  1. teraz w kontrolerze mozemy wywolywac nasza usluge (w kazdym)
$this->get('app.kontrahent')->addNewAdres($idDzierzawcy,$ulica, $mieszkanie, $kodPocztowy, $miasto);

Koniec :)

P.S.
"$emt = $this->getDoctrine()->getEntityManager();" przestarzaly zapis, aktualnie uzywac sie ...->getManager();
Struktury i konfiguracji struktury katalogowej nie bede opisywal

0

(uwaga: Symfony 3)
services.yml:

    
dzierzawca_logic:
         class: AppBundle\EntityLogic\DzierzawcaLogic
         arguments: [ "@doctrine.orm.entity_manager" ]        
 
<?php

namespace AppBundle\EntityLogic;

use Doctrine\ORM\EntityManager;
use AppBundle\Entity\Dzierzawca;
use AppBundle\Entity\Adres;

class DzierzawcaLogic 
{
    protected $em;

    public function __construct(EntityManager $entityManager)
    {
        $this->em = $entityManager;
    }
    public function addNewAdres($idDzierzawcy,$ulica, $mieszkanie, $kodPocztowy, $miasto)
    {
	$dzierzawca = $this->em->getRepository('AppBundle:dzierzawca')->find($idDzierzawcy);
        $newAdres = new Adres();
        $newAdres->ulica = $ulica;
        $newAdres->mieszkanie = $mieszkanie;
        $newAdres->kodPocztowy = $kodPocztowy;
        $newAdres->miasto = $miasto;
        $newAdres->dzierzawca = $dzierzawca;

        $this->em->persist($newAdres);
        $this->em->flush();

        
        return $newAdres->getIdAdresu();
    }

}

kontroler:

 
         $this->get('dzierzawca_logic')->addNewAdres($idDzierzawcy, $ulica, $mieszkanie, $kodPocztowy, $miasto);

jeszcze raz, dzięki @axelbest... a ty pewnie Java?

@PinguVanEx - wielkie dzięki... niestety byłeś szybszy o 4 minuty :<

0

Od Symfony 2.3 bedzie dzialac, ale np. w service.yml sa lekkie zmiany tzn. w S3 trzeba argumenty dodac w "" w poprzednich bez.

Odnosnie zapisu (EntityManager $entityManager) to zadziala od PHP 5.3
Jezeli ktos uzywa PHP7 to S od wersji 2.7

$dzierzawca = $this->em->getRepository('AppBundle:dzierzawca')->find($idDzierzawcy);
Masz tutaj 2 bledy, nie uzywaj "find" jak juz to findOneBy(array('id'=>$idDzierzawcy)) do tego sprawdzaj czy obiekt otrzymany nie jest pusty.
2 waznym blede jest w ogole szukanie calego obiektu, w moim kodzie masz metode getReference ktora powinno sie stosowac ;)
Nie zapominaj o obsludze wyjatkow try{}catch{}

1 użytkowników online, w tym zalogowanych: 0, gości: 1