Как сделать простой, если оператор пузыря стиль?

Question

Как сделать это if-утверждение более простым?как функция работает хорошо, но я думаю, что это плохо для кодирования.

это код:

        if (empty($checkMaxID)) 
        {
            $this->model->insert_temp_code($code_request,$cabang_code);
        }

        $checkHasTempCode = $this->model->checkHasTempCode($user_id);

        if ($checkMaxID['tempcode_created_by'] !=  $user_id ) {
            $data['code_request'] = str_pad($checkMaxID['tempcode_value'] + 1, 5, 0, STR_PAD_LEFT);        

            if (empty($checkHasTempCode) ) {
                $this->model->insert_temp_code($data['code_request'],$cabang_code);
            }
        }
        else
        {
            $data['code_request'] = $code_request;
        }

`

кто-нибудь может мне помочь, пожалуйста?

Спасибо

Answer 1

Используйте троичный оператор, если у вас есть условие if else.Кроме того, вы можете избежать нескольких вложенных условий if в одном операторе if.Я надеюсь, что это поможет.

$checkHasTempCode = $this->model->checkHasTempCode($user_id);
if (empty($checkMaxID))  {
    $this->model->insert_temp_code($code_request,$cabang_code);
}

$data['code_request'] = ( $checkMaxID['tempcode_created_by'] !=  $user_id ) ? 
str_pad($checkMaxID['tempcode_value'] + 1, 5, 0, STR_PAD_LEFT) : $code_request;
if ( empty($checkHasTempCode) && $checkMaxID['tempcode_created_by'] !=  $user_id ) {
    $this->model->insert_temp_code($data['code_request'],$cabang_code);
}