Можно ли тут как-то избавится от плагиата?

Question

[bedolazhka]

Пациент

public function signupFormValidator()
    {
        if (empty($this->login)) {
            $this->error = 'Вы не ввели логин';
            return false;
        }
        if (!filter_var($this->login, FILTER_VALIDATE_REGEXP, [
            'options' => [
                'regexp' => '#^[a-z]+(?:\s[a-z]+)*$|^[а-яё]+(?:\s[а-яё]+)*$#ui'
            ]
        ])) {
            $this->error = 'Вводите латинские или русские буквы и пробел';
            return false;
        }
        if (mb_strlen($this->login) < 3) {
            $this->error = 'Логин короче 3 букв';
            return false;
        }
        if (mb_strlen($this->login) > 15) {
            $this->error = 'Логин длиннее 15 букв';
            return false;
        }
        if (empty($this->password)) {
            $this->error = 'Вы не ввели пароль';
            return false;
        }
        if (!filter_var($this->password, FILTER_VALIDATE_REGEXP, [
            'options' => [
                'regexp' => '#^[\S]{0,}$#'
            ]
        ])) {
            $this->error = 'Вводите пароль без пробелов';
            return false;
        }
        if ($this->login == $this->password) {
            $this->error = 'Логин и пароль не должны совпадать';
            return false;
        }
        if (empty($this->email)) {
            $this->error = 'Вы не ввели email';
            return false;
        }
        if (!filter_var($this->email, FILTER_VALIDATE_EMAIL)) {
            $this->error = 'Email введён неправильно';
            return false;
        }
        return true;
    }

Дело в том, что раньше у меня было везде

return $this->error = 'Какое-то сообщение, которое выводится в отдельном блоке вывода сообщений для пользователя';

то есть ретурном останавливаем выполнение кода и устанавливаем свойство, но как мне пояснили - это очень древний подход.

А что разве правильно, что я ретурн фолс сто раз пишу?
Как от плагиата избавится?

Answer 1

[bedolazhka]

Оставить без изменений.

Answer 2

[FanatPHP]

Валидация форм ввода - отдельный специальный случай.
И валидировать по одному полю - это действительно дикость и маразм.
Если посмотреть на любой нормальный сайт, он не пишет тебе ошибки по одной. Он подсвечивает сразу все неправильно заполненные поля.

Поэтому возврат должен быть действительно только один, а все ошибки собираться в массив.

Да, и посмотри в словаре значение слова "плагиат"

Comments

[bedolazhka]

У меня мобильный сайт, ВСЕ сообщения ВСЕГДА краткие и содержат только одно сообщение, сообщение выводится в однострочный блок, содержащий описание страницы, то есть [Описание страницы] на сером фоне меняется на [Текст сообщения]: ошибка на красном фоне, удача на зелёном.

То есть вывод массива ошибок не предусмотрен.

[FanatPHP]

Ты не срашивал, "как сделать валидацию для моего кривого сайта". Ты спросил, как сделать так, чтобы код не напоминал говно мамонта. Тебе на него тветили.

[bedolazhka]

FanatPHP, у меня сайт при выводе сообщения не раскладывается на полэкрана, блок с описанием страницы заполняется сообщением)))

<?php if (isset($_SESSION['alert'])): ?>
        <div class="alert alert-<?= $_SESSION['alert']['result'] ?> p-1 my-1"><?= $_SESSION['alert']['text'] ?></div>
        <?php unset($_SESSION['alert']); ?>
    <?php else: ?>
        <div class="alert alert-dark p-1 my-1"><?= $site->description ?></div>
    <?php endif; ?>

[Arman]

а если аяксом проверяют каждое поле?)

[FanatPHP]

Arman, эта функция не подходит для проверки одного поля.

Answer 3

[Дмитрий]

Не вижу ничего в этом плохого. Если произошла ошибка, то метод возвращает текст ошибки, что можно прировнять к true, иначе нет ошибок - false

Comments

[Arman]

не 2019
??

public function signupFormValidator(): bool {}
public function signupFormValidator(): string {}

[bedolazhka]

я раньше так делал, меня не отговаривали, но указали, что так уже не делают - старый подход

Answer 4

[Максим]

Собирайте ошибки в массив $this->errors, если они будут возникать. И в конце проверяйте, если количество элементов в массиве с ошибками больше 0, то возвращайте false. И при необходимости сможете потом все ошибки вытащить из $this->errors.

Comments

[bedolazhka]

а зачем мне массив ошибок, если ошибка на первой проверке?

[FanatPHP]

Затем что посмотри в словаре слово "юзабилити"

[Adamos]

Это не та логика, которую привел автор. У него первая же ошибка прервет проверку, у вас - выполнятся все проверки, даже если после ошибки они не имеют смысла.

[bedolazhka]

Pardon Me! Where Do I Find 4giveness?, то же самое, что юзер заполнит 5 из 5 правильно и ему опять нужно набирать ошибочные 5, потому что он мудила не может прочитать даже одно сообщение об ошибки не говоря уже о массиве ошибок, мне не это нужно.

[bedolazhka]

Adamos, да, смысл такой: не ввёл логин, значит дальше нечего проверять

[Евгений Ромашкан]

bedolazhka, Нужно показать ошибку по одному полю - возьмите один элемент из массива ошибок.
Если в программе произошла исключительная ситуация - кидайте исключения - возвращать false или строку - костыль.
Есть ещё всякие either, но не в реалиях пхп. Хотя с union types будет чуть интереснее, но тогда уже нужно делать объект ошибки, а не строку возвращать.

Answer 5

[Arman]

Если совсем не хочется как во фреймах делают, а хочется в таком простом топорном, то я бы так сделал:

public function signupFormValidator(): bool
{
    $this->error = $this->getErrorSignupForm();

    return $this->error !== NULL;
}

protected function getErrorSignupForm(): ?string
{
    if (date('d') == 1) {
        return 'Не сегодня';
    }
    // остальные проверки
}