[Symfony] Dokładne przyrównania w metodach sprawdzających błędy formularzy |
[Symfony] Dokładne przyrównania w metodach sprawdzających błędy formularzy |
15.04.2020, 10:54:21
Post
#1
|
|
Grupa: Zarejestrowani Postów: 81 Pomógł: 0 Dołączył: 3.08.2017 Ostrzeżenie: (0%) |
O ile się nie mylę, formularz w Symfony zwraca wartości w postaci niepustego string'a lub null'a dla braku wartości. Zaleca się, aby stosować dokładne przyrównania (===) wszędzie tam, gdzie to jest możliwa. W związku z tym mam wątpliwość, czy zapis z kodu poniżej jest poprawny:
Kod /** * @Assert\IsFalse( * message="Strona www musi rozpoczynać się od znaków: http://" * ) */ public function isUrlValid(): bool { $urlValid = $this->url != '' && substr($this->url, 0, 7) != 'http://' && substr($this->url, 0, 8) != 'https://'; return $urlValid; } Czy nie powinienem zapisywać to w ten sposób (przyrównanie do null'a): Kod /** * @Assert\IsFalse( * message="Strona www musi rozpoczynać się od znaków: http://" * ) */ public function isUrlValid(): bool { return $this->url !== null && substr($this->url, 0, 7) !== 'http://' && substr($this->url, 0, 8) !== 'https://'; } Poza tym lepiej będzie chyba, jeśli return zwróci wynik bezpośrednio z równania, a nie ze zmiennej. Do tego (link poniżej) chyba źle nazwałem dwie metody sprawdzające. Powinno być isNewAndRepeatPasswordGiven() zamiast isNewOrRepeatPasswordGiven(). A także isNotUserEmail() zamiast isUserEmail(). https://github.com/webeeq/symfony.eeq/blob/...ditUserForm.php |
|
|
Wersja Lo-Fi | Aktualny czas: 28.04.2024 - 13:23 |