2011-08-24 6 views
0

코드 점화 장치를 사용할 때 이와 비슷한 모양의 많은 함수를 만듭니다.if 문에 값을 할당 하시겠습니까?

function info($id){ 
    $r = $this->db->select('id', 'name', 'age')->from('users')->where('id', $id)->get(); 
    return ($r->num_rows() == 0 ? false : $r->result()); 
} 

이 함수를 사용할 때이 함수를 사용하여 사용자가 존재하는지 확인하고 변수에 할당합니다. 내 질문은

if(($user = $this->user->info($_GET['id'])) === false) 
    die('User not found'); 

//now we can continue and $user contains the user info 

같은 것을 사용할 수 있도록

,이 나쁜 생각 어떤 이유가?

은 내가 아는 한이

$user = $this->user->info($_GET['id']); 
if($user === false) 
    die('User not found'); 

그러나 내 의견으로는 실제로 따라하기 쉽게에서 동일합니다.

답변

2

두 가지 방법이 동일합니다. 유일한 차이점은 명확성입니다. 나는 두 번째 방법이 첫 번째 방법보다 훨씬 더 명확하다고 믿는다. 프로젝트를 진행하는 다른 개발자와 협의하여 코드 작성 표준을 설정해야합니다. 일관성이 가장 중요합니다.

첫 번째 방법의 단점은 대신 ==을 사용할지 여부를 판단하기가 어렵다는 것입니다. 두 번째 방법에는 그러한 모호성이 없습니다.

관련 문제