В последнее время, чтобы избежать чрезмерно вложенных If
я пользуюсь принципом раннего возврата. Однако возникают ситуации, когда этот принцип может "засорить" код. Например, сегодня нужно вернуть json
с результатами, поэтому можно делать так:
public function canSendSms( ) {
$shopId = $this->request->post['shop_id'];
$dateEnd = $this->request->post['dateEnd'];
$orderId = $this->request->post['order_id'];
if ( ! $shopId || !$dateEnd || !$orderId ) {
$result["error"][] = "Неверные данные";
header("Content-type: application/json");
return json_encode($result, JSON_UNESCAPED_SLASHES, JSON_UNESCAPED_UNICODE);
}
// Собираем оставшиеся данные
$this->load->model('sale/order');
$order_info = $this->model_sale_order->getOrder($orderId);
if ( !$order_info ) {
$result["error"][] = "Не получилось найти заказ под номером " . $orderId;
header("Content-type: application/json");
return json_encode($result, JSON_UNESCAPED_SLASHES, JSON_UNESCAPED_UNICODE);
}
// финальный код
$result["success"][] = "Сообщение можно отправить";
header("Content-type: application/json");
return json_encode($result, JSON_UNESCAPED_SLASHES, JSON_UNESCAPED_UNICODE);
}
Как видно из примера - тут 3 раза повторяются 2 строки вывода
header("Content-type: application/json");
return json_encode($result, JSON_UNESCAPED_SLASHES, JSON_UNESCAPED_UNICODE);
Если метод небольшой, и изменений не планируется, то проблем нет. Но что если таких проверок будет 10, а формат вывода поменяется с json
на какой-нибудь другой? Ну да, придется тупо изменить вывод в этих 10 проверках.
В связи с этим, я начал использовать рекурсию, в итоге финальный возврат теперь всегда в подобных случаях осуществляется в начале метода. Выглядит это таким образом
public function canSendSms( $result = NULL ) {
if ( $result ) {
header("Content-type: application/json");
echo json_encode($result, JSON_UNESCAPED_SLASHES, JSON_UNESCAPED_UNICODE);
return;
}
$shopId = $this->request->post['shop_id'];
$dateEnd = $this->request->post['dateEnd'];
$orderId = $this->request->post['order_id'];
if ( ! $shopId || !$dateEnd || !$orderId ) {
$result["error"][] = "Неверные данные";
return $this->canSendSms( $result );
}
// Собираем оставшиеся данные
$this->load->model('sale/order');
$order_info = $this->model_sale_order->getOrder($orderId);
if ( !$order_info ) {
$result["error"][] = "Не получилось найти заказ под номером " . $orderId;
return $this->canSendSms( $result );
}
// какой-то код
$result["success"][] = "Сообщение можно отправить";
return $this->canSendSms( $result );
}
Вопрос, насколько это правильно, насколько понимаемо и встречается ли подобное в вашей практике?