Привет, у меня есть проект Symfony, где у меня есть контроллер под названием CompanyController.
У меня есть функция сохранения новой компании на основе некоторых данных, которые я отправляю методом POST.
Ниже я покажу этот метод:
/**
* @param Request $request
* @param ValidatorInterface $validator
* @return string|Response
*/
public function store(Request $request, ValidatorInterface $validator) {
$company = new Company();
$company->setName($request->get('name'));
$company->setTaxNumber($request->get('taxNumber'));
$company->setStreet($request->get('street'));
$company->setCity($request->get('postalCode'));
$company->setOfficeId($request->get('officeId'));
$company->setPostalCode($request->get('postalCode'));
$errors = $validator->validate($company);
if(count($errors) > 0) {
/**
* some code
*/
}
/**
* some response
*/
}
В этом методе я пытаюсь проверить данные из ответа POST. Интересно, будет ли хорошей практикой помещать аннотацию для проверки в файл сущности Компании, подобные ниже:
<?php
namespace App\Entity;
use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Common\Collections\Collection;
use Doctrine\ORM\Mapping as ORM;
use Symfony\Component\Validator\Constraints as Assert;
/**
* @ORM\Entity(repositoryClass="App\Repository\CompanyRepository")
*/
class Company
{
/**
* @ORM\Id()
* @ORM\GeneratedValue()
* @ORM\Column(type="integer")
*/
private $id;
/**
* @ORM\Column(type="string", length=255, nullable=true)
* @Assert\NotBlank
*/
private $name;
/**
* @ORM\Column(type="string", length=255)
* @Assert\NotBlank
*/
private $taxNumber;
/**
* @ORM\Column(type="string", length=255, nullable=true)
* @Assert\NotBlank
*/
private $street;
/**
* @ORM\Column(type="string", length=255)
* @Assert\NotBlank
*/
private $city;
/**
* @ORM\Column(type="string", length=255)
* @Assert\NotBlank
*/
private $postalCode;
/**
* @ORM\Column(type="string", length=255)
* @Assert\NotBlank
*
*/
private $officeId;
/**
* @var datetime $created
*
* @ORM\Column(type="datetime")
*/
protected $createdAt;
/**
* @var datetime $updated
*
* @ORM\Column(type="datetime", nullable = true)
*/
protected $updatedAt;
/**
* @ORM\OneToMany(targetEntity="App\Entity\Employee", mappedBy="company")
*/
private $employees;
public function __construct()
{
$this->employees = new ArrayCollection();
}
/**
* Gets triggered only on insert
* @ORM\PrePersist
*/
public function onPrePersist()
{
$this->createdAt = new \DateTime("now");
}
/**
* Gets triggered every time on update
* @ORM\PreUpdate
*/
public function onPreUpdate()
{
$this->updatedAt = new \DateTime("now");
}
public function getId(): ?int
{
return $this->id;
}
public function getName(): ?string
{
return $this->name;
}
public function setName(?string $name): self
{
$this->name = $name;
return $this;
}
public function getTaxNumber(): ?string
{
return $this->taxNumber;
}
public function setTaxNumber(string $taxNumber): self
{
$this->taxNumber = $taxNumber;
return $this;
}
public function getStreet(): ?string
{
return $this->street;
}
public function setStreet(?string $street): self
{
$this->street = $street;
return $this;
}
public function getCity(): ?string
{
return $this->city;
}
public function setCity(string $city): self
{
$this->city = $city;
return $this;
}
public function getPostalCode(): ?string
{
return $this->postalCode;
}
public function setPostalCode(string $postalCode): self
{
$this->postalCode = $postalCode;
return $this;
}
public function getOfficeId(): ?string
{
return $this->officeId;
}
public function setOfficeId(string $officeId): self
{
$this->officeId = $officeId;
return $this;
}
/**
* @return Collection|Employee[]
*/
public function getEmployees(): Collection
{
return $this->employees;
}
public function addEmployee(Employee $employee): self
{
if (!$this->employees->contains($employee)) {
$this->employees[] = $employee;
$employee->setCompany($this);
}
return $this;
}
public function removeEmployee(Employee $employee): self
{
if ($this->employees->contains($employee)) {
$this->employees->removeElement($employee);
// set the owning side to null (unless already changed)
if ($employee->getCompany() === $this) {
$employee->setCompany(null);
}
}
return $this;
}
}
Это правильный подход? Второй подход, с которым я встречался, - поместить аннотацию в совершенно другой файл Entity. Например:
public function productPostAction(AccessChecker $checker, Request $request, JsonFromDbObjectConverter $converter){
$productRequest = new ProductRequest();
$productRequest->name = $request->get('name');
$productRequest->users = $request->get('users');
$productRequest->llc = (int)$request->get('llc');
$validator = $this->get('validator');
$errors = $validator->validate($productRequest);
if (count($errors) > 0) {
/**
* some code
*/
}
/**
* some response
*/
}
Отдельный класс сущностей для проверки:
<?php
namespace CblBundle\Request;
use Symfony\Component\Validator\Constraints as Assert;
class ProductRequest
{
/**
* @var string
*
* @Assert\NotBlank()
*/
public $name;
/**
* @var array
*/
public $users;
/**
* @var int
* @Assert\Type("integer")
*/
public $llc;
/**
* @Assert\IsTrue(message = "Wrong users data!")
*/
public function isUsersTypeOfNullOrArray()
{
return (is_null($this->users) || is_array($this->users));
}
}