[Symfony] Dokładne przyrównania w metodach sprawdzających błędy formularzy Opcje

[eerie]

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