Какой из этих строк кода «лучше» и почему?

Question

Первый:

 protected function isValid($orderIds)
 {
     ......

     $orderCount = Some Calculation... ; 

     return ( $orderCount == count($orderIds) );
 }

Второй:

 protected function isValid($orderIds)
 {
     ......

     $orderCount = Some Calculation... ; 

     return ( $orderCount == count($orderIds) ) ? 1 : 0;
 }

Я использую эту функцию следующим образом

......

$isValid = $this->isValid($orderIds);
if($isValid) {
   // do some thing here
}

Пожалуйста, посмотрите выше написанный код. Что-то не так, если я выбираю первую функцию вместо второй?

Answer 1

Первый метод будет возвращать только тип логический, а во втором методе вы просто добавляете издержки для возврата 1 or 0. Если вам нужно только проверить, успешно ли метод выполнил свою работу, я рекомендую вам go с первым подходом.