Может ли кто-то критиковать мой класс регистрации / проверки PHP?

Question

http://pastie.org/931617 - ссылка на исходный код.Если есть лучший способ сделать это, дайте мне знать.

Одна вещь, в которой я не уверен, это то, как я обрабатываю дочерние / родительские конструкции в классах валидации по направлению вниз.

Спасибо.

Хорошо, я воспользовался вашим советом, используя ассоциативный массив.Теперь я просто добавляю переменную $ _POST в мою конструкцию.Как это выглядит - http://pastie.org/931715

Answer 1

Вы обрабатываете конструкции правильно.

Очень длинные списки аргументов хороши в строго типизированных языках, таких как C ++, но не очень удобны и безопасны для PHP. Мой совет - использовать ассоциативные массивы.

function __construct(&$args)
{
    parent::__construct($args);

    $this->contactphone = $this->get($args, 'contactphone'); // check if $args['contactphone'] is specified, otherwise return null
    $this->firmname = $this->get($args, 'firmname');
    // ...
}

Использование:

$args = array(
'username' => $username, 
'password' => $password, 
'confirmpassword' => $confirmpassword
);
$e = new EmployerRegister($args);

Ваши преимущества:

1. Вам не нужно запоминать точный порядок аргументов. Создайте массив аргументов в любом порядке. Меньше ошибок. Меньше печатать.

2. Некоторые аргументы могут быть пустыми, поэтому отправлять их не нужно. Лучшая производительность.