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

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]

Спасибо

Answer 2

[Mellorn]

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

Возвращать false в случае ошибки - плохо. И да, это наследие раннего php. В описаниях многих стандартных php-функций мы встречаем: возвращает что-то там в случае успеха, или false в случае ошибки. Но так делать не стоит.
В случае ошибки - бросаем исключение.

Например, получаем данные пользователя по ID.
Если получить результат не удалось (именно не удалось) - exception.
Если всё отработало корректно, но юзера с таким ID не существует, можно вернуть null. А уже в вызывающем метод коде выбросить какой-нибудь NotFoundHttpException например. Или ещё каким-либо образом обработать отсутствие результата.

Comments

[AlexandrMa]

Спасибо

[Adamos]

В том и дело, что единственно правильное решение в одном этом коде невозможно. Проблема должна решаться комплексно и в основном выше уровнем, чем конкретные функции. А вот для этого, действительно, обычно проще использовать исключения.

[Mellorn]

Adamos, полностью с вами согласен. Собственно, об этом и писал. Возможно, не совсем чётко изложил свою мысль.

Единственное, что не полностью понял вот это:

А вот для этого, действительно, обычно проще использовать исключения.

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

Но бывают и исключения из правил.

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