Как избавиться от проверок аргументов?

Question

[Max Ba]

Мужики, в каждом методе надоело постоянно проверять входные переменные.
Можно ли, как-то оптимизировать или сократить код?

public function send($id, $parent_id, $name, array $data){
  if(empty($id) || empty($parent_id) || empty($name)){
    //Exception
  }
  //ok
}

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

Comments

[Max Ba]

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

[Max Ba]

Владимир, Warning выкинет и всё. Ни туда, ни сюда)

Answer 1

[Immortal_pony]

1. Выкинуть лишние проверки из функции
2. Заставить PHP кидать Exception, а не waning'и. Примернно так:

set_error_handler(function ($severity, $message, $file, $line) {
    if (!(error_reporting() & $severity)) {
        return;
    }
    throw new ErrorException($message, 0, $severity, $file, $line);
});

3. В удобном месте поставить обаботку Exception'ов

try {
   // do something
   send($id, $parentId, $name, $data)
} catch (\Throwable $e) {
    // handle error
};

Answer 2

[Дмитрий]

Передавать аргументы массивом через compact, в функции проверять в цикле, если все ок - извлекать через extract

Comments

[Max Ba]

Если массивом, то зачем implode?)

[Дмитрий]

Max Ba, Прошу прощения. Исправил: compact и extract