Symfony - Pytanie dotyczące trasowania

Question

Cześć, jak mam wstawić format datetime do adresu url który wpisuje w przeglądarce? Chodzi mi o startDate i endDate

 

<?php

namespace AppBundle\Controller;

use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Routing\Annotation\Route;
use AppBundle\Entity\Transport;

class TransportController extends Controller
{
    /**
     * @Route("/add/{user}/{startCity}/{endCity}/{cargo}/{distance}/{weight}/{damage}/{fuel}/{fueled}/{country}/{score}/{transportId}/{screenshot}/{startDate}/{endDate}/", defaults={"status" = 0})
     */
    public function addAction($user, $startCity, $endCity, $cargo, $distance, $weight, $damage, $fuel, $fueled, $country, $score, $transportId, $screenshot, $startDate, $endDate)
    {
        $transport = new Transport();

        $transport->setUser($user);
        $transport->setStartCity($startCity);
        $transport->setEndCity($endCity);
        $transport->setCargo($cargo);
        $transport->setDistance($distance);
        $transport->setWeight($weight);
        $transport->setDamage($damage);
        $transport->setFuel($fuel);
        $transport->setFueled($fueled);
        $transport->setCountry($country);
        $transport->setScore($score);
        $transport->setTransportId($transportId);
        $transport->setScreenshot($screenshot);
        $transport->setStatus(0);
        $transport->setStartDate($startDate);
        $transport->setEndDate($endDate);

        $em = $this->getDoctrine()->getManager();
        $em->persist($transport);
        $em->flush();

        return new Response("Dodano nowy transport o identyfikatorze: ".$transport->getTransportId());
    }
}

 

Comments

Pan to chyba Factory Method, Prototype lub Buildera nie zna, co? :)

Ładnie się zaczyna - tworzysz instancję obiektu Transport - jednak to nie jest miejsce na takie coś. Powinieneż stworzyć sobie jakąś prostą fabrykę, która tu by miała wywołanie: 

<?php

$transport = new FactoryTransport();

$transport->build(parametry);

I tyle.

Jednak gdyby się jeszcze bardziej uczepić i pobawić w upiększanie uelastycznianie i SOLIDne rzeczy, to warto zastosować zaczątek CQRS czyli zwykłego Commanda:

<?php
$newTransport = new AddTransportCommand(parametry);
$newTransport->execute();

Przy tworzeniu instancji byś wywoływał metody odpowiednio ustawiające klasę, a przy execute byś zadbał o obsługę tego oraz wrzucenie do repozytorium agregatu. 

(Dałem tę odpowiedź w komentarzu bo nie traktuje bezpośrednio o tym, co potrzebujesz a uznałem, że warto byś wiedział  )

---

Jak zaimportować mapowanie w rozszerzeniu .yml do encji Transport?

---

http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/tutorials/getting-started.html

Od samej góry do "Generating the Database Schema"

http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/reference/yaml-mapping.html

http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/reference/configuration.html

http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/reference/basic-mapping.html (zakładki yml w kodach)

To przykłady z dokumentacji.

Tutaj przykład z dokumentacji Symfony:

http://symfony.com/doc/current/book/doctrine.html

http://symfony.com/doc/current/reference/configuration/doctrine.html

Tu niesprawdzony przeze mnie polski link, aczkolwiek tłumaczenie oryginalne doku na polski: http://symfony-docs.pl/book/doctrine.html czasem pomaga zrozumiec dziwne słownictwo jak sie takie pojawi.

Może jeszcze to pomóc:

http://stackoverflow.com/questions/9137528/symfony2-generate-doctrine-entities-from-yml-files?rq=1

http://stackoverflow.com/questions/26135581/doctrine-mapping-in-symfony2-using-yaml?rq=1

A tu ja robię to w Silexie -

https://github.com/event15/RegistriesAPI/blob/master/config/bootstrap.php#L13-L44

https://github.com/event15/RegistriesAPI/blob/master/cli-config.php

https://github.com/event15/RegistriesAPI/blob/master/src/Infrastructure/Doctrine/Mappings/Madkom.RegistryApplication.Domain.CarManagement.Car.dcm.yml

https://github.com/event15/RegistriesAPI/blob/master/composer.json

 

---

A validation.yml, jak zaimportować do wybranego bundle?

Answer 1

Moim zdaniem tworzysz błędne rozwiązanie prostego problemu.

Z jakiego powodu do linku wrzucasz:

• user
• startCity
• endCity
• cargo
• distance
• weight
• damage
• fuel
• fueled
• country
• score
• transportId
• screenshot
• startDate
• endDate

?

Cała masa gówna niepotrzebnego w linku. 

Co nam mówi ten link? 

Masz metodę o nazwie addAction() więc domyślam się, że chcesz dodać coś lub wszystko (bo nazwa kontrolera nie mówi mi nic o tym co chcesz dodać - co jest kiepską praktyką. Powinno się stosować nazwy metod i zmiennych, które mówią nam jak najwięcej o kodzie i jego funkcji):

• https://www.youtube.com/watch?v=z0y3IPJDyp0
• https://php-kurs.gitbooks.io/phpkurs/content/czesc-i/Zmienne/Dobre-praktyki.html
• https://www.youtube.com/watch?v=XSb_msSaoXs

Nie wiem dlaczego zaczynasz kontroler tej akcji od słowa "user" - to od razu daje sygnał, że "dla użytkownika" wykonaj akcję. 

Reszta danych może być przesyłana za pomocą klas Request i Response:

Symfony\Component\HttpFoundation\Request

Symfony\Component\HttpFoundation\Response

I w zależności od potrzeby wykonać odpowiednie zapytanie ze strony frontu w postaci jakiegoś POSTa. 

Poza tym warto byś zainteresował się RESTem:

• https://pl.wikipedia.org/wiki/Representational_State_Transfer
• http://www.ics.uci.edu/~taylor/documents/2002-REST-TOIT.pdf
• https://en.wikipedia.org/wiki/Representational_state_transfer
• http://www.restapitutorial.com/lessons/whatisrest.html
• http://martinfowler.com/articles/richardsonMaturityModel.html
• http://martinfowler.com/articles/enterpriseREST.html

Oraz HATEOAS (de facto fundament resta):

• https://spring.io/understanding/HATEOAS
• https://en.wikipedia.org/wiki/HATEOAS
• http://www.slideshare.net/zenedith/budowa-res-towegoapiwoparciuohateoasbraincodemobi2014

 

Ogólnie o REST możesz poczytać: http://restcookbook.com/

Odnośnie Twojego problemu, rozumiem, że setery, które masz w kontekście dat to jakieś obiekty DateTime()? Jeśli chcesz je odbierać z linka to są one traktowane jako tekst i musisz je po prostu przetworzyć na obiekt typu DateTime. 

Nie powinieneś w tym miejscu też wpychać bazy danych i wykonywać na niej jakiś akcji - to zła praktyka. W tym kontrolerze mieszasz akcje - które powinny być atomowe i jasno mówić nam jakie jest ich zadanie (metody nie powinny mieć więcej niż 5 wierszy w kontrolerze), logikę dziedziny stosujesz w kontrolerach a nie w oddzielnym modelu, który powinien to wszystko samodzielnie obsługiwać - czyli tak naprawdę w przypadku symfony powinien istnieć osobny Bundle - lub raczej zupełnie odseparowany kawałek kodu od całego symfony, odpowiedzialny wyłącznie za dziedzinę - czyli za wszystko związane z tym transportem, w tym klasy walidacyjne i adaptery dla repozytoriów. Dodatkowo wkładasz tutaj kod infrastruktury, który nie powinien być mieszany z kodem kontrolera, a tym bardziej z kodem logiki dziedziny. 

Gdy zwracasz za pomocą Response() to tak estetycznie raczej lepiej jest dać zwykłą okejkę niż pisać co się udało. 

Chciałbym zwrócić uwagę, że nie widze w tym kodzie nawet wzmianki o tym, że gdzieś dokonujesz walidacji zmiennych otrzymywanych od użyszkodznika. 

Comments

"/add/{user}/{startCity}/{endCity}/{cargo}/{distance}/{weight}/{damage}/{fuel}/{fueled}/{country}/{score}/{transportId}/{screenshot}/{startDate}/{endDate}/

Własnie wyobraziłem sobie jak wygląda ten link gdy wszystko się do niego wrzuci:

/add/event15/Gdańsk/Oświęcim/jedzenie/800/49000/100/5/500/Polska/12034/adf(lKl!@#$j234fa094r5/_preview.png/16-05-2016/20-05-2016/ 

Co mnie przekonuje, że brniesz w coś ze złym podejściem. 

Jeśli masz więcej rzeczy niż transport, które możesz dodać to przede wszystkim wywal z linku słówko "add" które jest niepotrzebne gdy używasz restowego api - wystarczy informacja, że potrzebujesz POST. Zauważ - używając metody POST nie będziesz mieć linku. Jednak moim zdaniem to nie powinien być problem. 

Masz sobie widok, który jest pod adresem domena.pl/transport

Pod tym linkiem masz dostępne api: POST który jest /transport POST i jsonem wysyłane dane które potrzebujesz - ta akcja dodaje transport

Gdy potrzebujesz pobrać ostatni transport:

/transport/last

Gdy potrzebujesz pobrać transport z przedziału datowego:

/transport/criteria/16-05-2016:20-05-2016/ 

(na przykład) zamiast criteria może być "search"

Gdy potrzebujesz wyświetlic konkretny transport:

/transport/{transportId}

Gdy potrzebujesz zmodyfikować konkretny transport:

/transport/{transportId} PATCH lub PUT z danymi. 

Nie wgryzam się w szczegóły bo jest dużo teorii odnośnie tego kiedy używać patch a kiedy put.

Do wgrywania zdjęć warto zrobić sobie w ogóle osobną obsługe api. Na przykład /api/file Dzięki temu możesz w osobnym miejscu twrorzyć rozwiązania wyłącznie dla plików ze zdjęciami - obsługe jpg, gif bmp czy png. Ich cropowanie, nadawanie im różnych masek czy nie wiem - rozpoznawanie twarzy. Wtedy nie są one związane z samym transportem - a pewnie masz w innych miejscach w "projekcie" też obsługe obrazków. 

---

Ale spokojnie, @Route(i milion zmiennych) stworzyłem tylko po to żeby pobawić się doctrine, zobaczyć jak działa, zrozumieć a wy mi tu piszecie że zrobiłem milion błędów, tak wiem, zrobiłem ale chciałem tylko zrozumieć doctrine, a później zrobić to tak jak należy. Ale dziękuje również za podpowiedzi, i rozwiązania.

---

Zabawa z doctrine powinna mieć miejsce w Repozytorium a konfiguracja encji w plikach yml i generalnie nie powinno się stosować adnotacji ponieważ php tego natywnie nie obsługuje i zaśmieca tylko kod.

Takie sprawy powinny być robione za pomocą plików konfiguracyjnych - YAML jest najlepszy.

Doctrine, jak i Symfony udostępniają możliwość tworzenia konfiguracji w plikach yml. Pozwalają na korzystanie z ormów wszędzie, aczkolwiek dobrą praktyką jest wydzielenie warstwy infrastruktury a do repo wrzucałbyś odpowiednio spreparowaną encję.

Answer 2

A pokaż te encje Transport.

Comments

Mógłby ją przekazać jako argument kontrolera, Symfony powinno samo wstrzyknac odpowiednie dane.

---

<?php

namespace AppBundle\Entity;

use Doctrine\ORM\Mapping as ORM;

/**
 * @ORM\Entity
 * @ORM\Table(name="transports")
 */
class Transport
{
  /**
   * @ORM\Column(type="integer")
   * @ORM\Id
   * @ORM\GeneratedValue(strategy="AUTO")
   */
  private $id;
  
  /**
   * @ORM\Column(type="string")
   */
  private $user;

  /**
   * @ORM\Column(type="string")
   */
  private $startCity;

  /**
   * @ORM\Column(type="string")
   */
  private $endCity;

  /**
   * @ORM\Column(type="string")
   */
  private $cargo;

  /**
   * @ORM\Column(type="integer")
   */
  private $distance;

  /**
   * @ORM\Column(type="integer")
   */
  private $weight;

  /**
   * @ORM\Column(type="integer")
   */
  private $damage;

  /**
   * @ORM\Column(type="integer")
   */
  private $fuel;

  /**
   * @ORM\Column(type="integer")
   */
  private $fueled;

  /**
   * @ORM\Column(type="string")
   */
  private $country;

  /**
   * @ORM\Column(type="integer")
   */
  private $score;

  /**
   * @ORM\Column(type="string")
   */
  private $transportId;

  /**
   * @ORM\Column(type="string")
   */
  private $screenshot;

  /**
   * @ORM\Column(type="integer")
   */
  private $status;

  /**
   * @ORM\Column(type="datetime")
   */
  private $startDate;

  /**
   * @ORM\Column(type="datetime")
   */
  private $endDate;

    /**
     * Get id
     *
     * @return integer 
     */
    public function getId()
    {
        return $this->id;
    }

    /**
     * Set user
     *
     * @param string $user
     * @return Transport
     */
    public function setUser($user)
    {
        $this->user = $user;

        return $this;
    }

    /**
     * Get user
     *
     * @return string 
     */
    public function getUser()
    {
        return $this->user;
    }

    /**
     * Set startCity
     *
     * @param string $startCity
     * @return Transport
     */
    public function setStartCity($startCity)
    {
        $this->startCity = $startCity;

        return $this;
    }

    /**
     * Get startCity
     *
     * @return string 
     */
    public function getStartCity()
    {
        return $this->startCity;
    }

    /**
     * Set endCity
     *
     * @param string $endCity
     * @return Transport
     */
    public function setEndCity($endCity)
    {
        $this->endCity = $endCity;

        return $this;
    }

    /**
     * Get endCity
     *
     * @return string 
     */
    public function getEndCity()
    {
        return $this->endCity;
    }

    /**
     * Set cargo
     *
     * @param string $cargo
     * @return Transport
     */
    public function setCargo($cargo)
    {
        $this->cargo = $cargo;

        return $this;
    }

    /**
     * Get cargo
     *
     * @return string 
     */
    public function getCargo()
    {
        return $this->cargo;
    }

    /**
     * Set distance
     *
     * @param integer $distance
     * @return Transport
     */
    public function setDistance($distance)
    {
        $this->distance = $distance;

        return $this;
    }

    /**
     * Get distance
     *
     * @return integer 
     */
    public function getDistance()
    {
        return $this->distance;
    }

    /**
     * Set weight
     *
     * @param integer $weight
     * @return Transport
     */
    public function setWeight($weight)
    {
        $this->weight = $weight;

        return $this;
    }

    /**
     * Get weight
     *
     * @return integer 
     */
    public function getWeight()
    {
        return $this->weight;
    }

    /**
     * Set damage
     *
     * @param integer $damage
     * @return Transport
     */
    public function setDamage($damage)
    {
        $this->damage = $damage;

        return $this;
    }

    /**
     * Get damage
     *
     * @return integer 
     */
    public function getDamage()
    {
        return $this->damage;
    }

    /**
     * Set fuel
     *
     * @param integer $fuel
     * @return Transport
     */
    public function setFuel($fuel)
    {
        $this->fuel = $fuel;

        return $this;
    }

    /**
     * Get fuel
     *
     * @return integer 
     */
    public function getFuel()
    {
        return $this->fuel;
    }

    /**
     * Set fueled
     *
     * @param integer $fueled
     * @return Transport
     */
    public function setFueled($fueled)
    {
        $this->fueled = $fueled;

        return $this;
    }

    /**
     * Get fueled
     *
     * @return integer 
     */
    public function getFueled()
    {
        return $this->fueled;
    }

    /**
     * Set country
     *
     * @param string $country
     * @return Transport
     */
    public function setCountry($country)
    {
        $this->country = $country;

        return $this;
    }

    /**
     * Get country
     *
     * @return string 
     */
    public function getCountry()
    {
        return $this->country;
    }

    /**
     * Set score
     *
     * @param integer $score
     * @return Transport
     */
    public function setScore($score)
    {
        $this->score = $score;

        return $this;
    }

    /**
     * Get score
     *
     * @return integer 
     */
    public function getScore()
    {
        return $this->score;
    }

    /**
     * Set transportId
     *
     * @param string $transportId
     * @return Transport
     */
    public function setTransportId($transportId)
    {
        $this->transportId = $transportId;

        return $this;
    }

    /**
     * Get transportId
     *
     * @return string 
     */
    public function getTransportId()
    {
        return $this->transportId;
    }

    /**
     * Set screenshot
     *
     * @param string $screenshot
     * @return Transport
     */
    public function setScreenshot($screenshot)
    {
        $this->screenshot = $screenshot;

        return $this;
    }

    /**
     * Get screenshot
     *
     * @return string 
     */
    public function getScreenshot()
    {
        return $this->screenshot;
    }

    /**
     * Set status
     *
     * @param integer $status
     * @return Transport
     */
    public function setStatus($status)
    {
        $this->status = $status;

        return $this;
    }

    /**
     * Get status
     *
     * @return integer 
     */
    public function getStatus()
    {
        return $this->status;
    }

    /**
     * Set startDate
     *
     * @param \DateTime $startDate
     * @return Transport
     */
    public function setStartDate($startDate)
    {
        $this->startDate = $startDate;

        return $this;
    }

    /**
     * Get startDate
     *
     * @return \DateTime 
     */
    public function getStartDate()
    {
        return $this->startDate;
    }

    /**
     * Set endDate
     *
     * @param \dateTime $endDate
     * @return Transport
     */
    public function setEndDate(\dateTime $endDate)
    {
        $this->endDate = $endDate;

        return $this;
    }

    /**
     * Get endDate
     *
     * @return \dateTime 
     */
    public function getEndDate()
    {
        return $this->endDate;
    }
}

 

---

Ło matko. 

https://gist.github.com/event15/d79900af46fef424d5cb66f860bcece2

To teraz pacz:

Tworzysz 16 pól. W tym id i transportId się duplikują (chyba, że się mylę).

Teraz zobacz na logikę swojego zachowania. Tworzysz prywatne pola, następnie tworzysz publiczne akcesory do niej. Czyli równie dobrze możesz zrobić te wszystkie pola jako publiczne i wywalić 197 wierszów nadmiarowego kodu. 

W ogóle nie myślałeś nad tym, co zamierzasz zrobić. Stworzyłeś encję transport i wrypałeś tam ogromny śmietnik. 

Co ta klasa ma robić w ogóle?

class Transport
{
    // Identyfikatory
    private $id;
    private $user; // userId
    private $transportId;

    // Lokalizacje i odległości
    private $country;
    private $startCity;
    private $endCity;
    private $distance;

    // Daty
    private $startDate;
    private $endDate;

    // O ładunku
    private $cargo; // o ładunku
    private $weight;
    private $damage;

    // O transporterze
    private $fuel;
    private $fueled;

    // Jakieś punkty
    private $score;

    // czy potrzebne?
    private $screenshot;
    private $status;

}

Tutaj masz pole do popisu. Przede wszystkim powinienes zastanowić się czy rzeczywiście w tym miejscu potrzebujesz ID usera. Bo mam nadzieję, że nie potrzebujesz całej instacji. Nie musisz robić osobnego ID dla klasy i osobnego ID dla transportu. Doctrine to załatwia dla Ciebie. 

Odnośnie lokalizacji itp. to powinna być osobna encja City, którą byś tu wrzucił przez jakieś Dependency Injection. Coś jak funkcja(CityInterface $newCity). W niej byś oprogramował odpowienie metody, mógłbyś walidować dane w osobnej klasie. 

Daty możesz załatwić jakąś arrayką, iteratorem itp. 

Dane ładunku są jako ValueObject, ewentualnie jeśli będziesz potrzebował to jakoś bardziej rozbudowywać i te ładunku będą miały zdefiniowane jakieś działanie to kolejna encja. 

Transporter to w tym przypadku moze być jako VO i odnośnie screenschota to powinno być to w innym miejscu a tutaj tylko Dependency Injection do tego. 

Przede wszystkim encja to nie tylko getery i setery. To co tutaj dałeś to bardziej wygląda na DTO (Data Transfer Object), jednak nie masz żadnej fabryki do tworzenia tych obiektów wiec po prostu tak to zrobiłeś. 

W tej encji powinno byc zachowanie - może ona być agregatem jeśli jest to kluczowe w tej dziedzinie. Jeśli jest agregatem to ona ma zachowania odpowiedzialne za przeniesienie ładunku, załadowanie, rozładowanie, i tym podobne. 

Naprawdę, zachęcam Cie do poczytania o tych kilku rzeczach:

• Command Pattern
• Factory Method Pattern (albo Builder lub Abstract Factory) - wszystkie są podobne i łatwo ogarnąć
• Repository Pattern

http://zawarstwaabstrakcji.pl/20151020-save-repository-from-save/

http://zawarstwaabstrakcji.pl/20151008-jeste-buildere/

http://zawarstwaabstrakcji.pl/20151102-avatar-uzytkownika-upload-plikow/

Jest dobrze opisane to, o czym mówię. Nie jest do długa lektura. 

---

W dokumentacji są takie przykłady, więc aby zrozumieć działanie Doctrine zrobiłem kod dokładnie na wzór tego z dokumentacji.

transportId, to będzie identyfikator transportu np: XXYX/XYX/XZ123/PL
status, jeżeli 0 to transport jest w weryfikacji a jeżeli 1 to transport jest zatwierdzony.

W encji Transport potrzebuję tylko getery i setery.

---

Chcesz pisać gówno to pisz.

---

Zrób z miejsca wszystkie pola publiczne wtedy bedziesz miec to samo działanie i tylko 30 linii kodu a nie 300. Oszczęðność czasu, liczby wierszy i myślenia. A jak się uprzesz to już __get() i __set().

Answer 3

Użyć metody GET.

?startDate=123456789&&endDate=987654321