Почему некоторые методы возвращают разный тип данных?

Question

[AlexandrMa]

Очень часто встречается такая конструкция

public function test(){
  if(empty($this->id)){
    return false;
  }
  return $this->getRows();
}

Это ошибка или такая практика нормальная? Ведь можно: а) выбросить исключение если пустой $this->id. б) сделать return [];

Answer 1

[Adamos]

Это нормальная практика старого РНР. Сейчас попахивает и делать так не стоит.
Вариант Б может ввести в заблуждение. Пустой результат, отсутствие результата и ошибка, не позволяющая получить результат - это три разных результата.

Comments

[AlexandrMa]

Получается, единственно правильный вариант, сделать исключение?

public function test(){
  if(empty($this->id)){
    throw new Exception('empty id');
  }
  return $this->getRows();
}

[Adamos]

AlexandrMa, единственный правильный вариант - думать, что вы делаете и для чего. Например, если это API - вам все равно нужно вернуть какой-то ответ, а не просто прекратить исполнение. Сделаете вы это, выкинув исключение здесь и перехватив его выше, или как-то иначе - зависит от того, как у вас все устроено.
Исключения - тоже не волшебная палочка, а механизм, который не должен ломать работу всего остального.

[Александр Лыкасов]

Всё зависит от того, надо ли в вызывающем коде реагировать на ошибки. Бывает так, что не важно почему данные не получены - ошибка ли какая-то или просто данные не найдены.

[Adamos]

Александр Лыкасов, ну, например, Почта России в своем API расчета стоимости отправления просто возвращает цену = 0, если индекс не существует.
Это пример неудачной реакции на ошибки. Им-то, может, и неважно...

[Александр Лыкасов]

Adamos, Это да, не тот случай когда не важно :)

[AlexandrMa]

Спасибо