Надо ли проверять переменную на пустоту в методаx?

Question

[Max Ba]

class Book{
  public function read($a){
    if(empty($a)){
      return false;
    }
    echo 'A';
  }
}

$a = new Book;
$a->read('');

echo 'B';

Получается, нужно каждый раз проверять во всех методах переменные "на пустоту"? Или это надо делать не в классе, а в момент вызова?

if(!empty($var)){
  $a->read($var);
}

Где логичней-то проверять?

Answer 1

[Ramm]

Указывайте type hints

Comments

[Max Ba]

Укажу я тип int. Влетит ноль. а с нулем работать не нужно. И как быть?

[Ramm]

Max Ba, тогда указывайте в методе проверку

[Max Ba]

Ramm, а лучше там, или снаружи?

[Ramm]

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

[Max Ba]

Ramm, благодарю

Answer 2

[Nujabes37]

Если есть шанс, что в метод будет передано пустое значение, думаю логичнее будет проверить этот кейс)

Comments

[Max Ba]

а где проверить то лучше? внутри метода или снаружи?

[Nujabes37]

Max Ba, Лучше в методе конечно. Вот написали вы API, а кто-то взял и вызвал метод с пустым значением, что будете делать в таком случае ? Вы же не сможете уследить за тем, кто и что туда передает.

[Max Ba]

Nujabes37, ну это да, логично.

[Max Ba]

благодарю

[Nujabes37]

Max Ba, не за что ;)

Answer 3

[dmitriy]

как вариант использовать ValueObject
объект который при своей инициализации будет делать проверку

class ValueObject{

function __construct(string $value){
if(empty(value)){
throw new \Exception("переменная не должна быть пустой");
}
}

}

...

public function read(ValueObject $a){

echo  $a->getValue();//гарантированно не пусто
}