Нужно ли проверять типы свойств/параметров?

Question

[galithr]

Имеется класс, например:

class MyClass {

 private $_flag;

 public function setData($value){
  $this->_flag = $value;
 }

 public function getData(){
  return $this->_flag;
 }
}

т.е. свойство flag задается через геттеры/сеттеры. К примеру я ожидаю что в этом свойстве может храниться только целое число (или другой конкретный тип данных). Но пользователь ведь может в этом случае присвоить свойству все что угодно, хоть объект. Собственно вопрос стоит ли в таких случаях, например в геттере, проверять тип параметра, или оставить все на совести пользователя? вообще как лучше выйти из такой ситуации?

Под пользователям имею ввиду программиста использующего мой класс

Answer 1

[Vitaliy Orlov]

тут всего скорее под "пользователем", ты имеешь ввиду программиста использующего класс.. тогда, просто укажи тип в сигнатуре метода:
public function setData(int $value){

ну а если это ввод от какого-то пользователя, то его нужно валидировать до попадания в класс

P.S. Оставлять все на совести пользователей не стоит.. Входящие параметры надо ограничивать и валидировать, это упростит использование класса и предотвратит появление неявных ошибок

Например сейчас в WP есть двухгодичный баг, как раз демонстрирующий такие последствия:
https://core.trac.wordpress.org/ticket/29750

Comments

[galithr]

в php ниже 7-й версии нельзя к сожалению сделать setData(int $value)

[Vitaliy Orlov]

galithr: я уже давно все перевел на php 7, и стал забывать о прошлых версиях :)

Если нет поддержки 7, тогда проверяй на входе в сеттер тип переменных:

if (!is_int($a)) throw new InvalidArgumentException('function only accepts integers. Input was: '.$a);

В общем, валидация в сеттере - это правильно и она нужна.

Answer 2

[trevoga_su]

если очень критично - можно приводить к нужному типу.