유효성 검사를위한 프로그램 절차

Question

'validation code'을 작성할 때 가장 좋은 방법은 무엇입니까? 이처럼 :

if(!condition1) { 
    print('error'); 
    return false; 
} 

if(!condition2) { 
    print('error'); 
    return false; 
} 

do_something(); 
return true; 

또는 같은

:

if(condition1) { 
    do_something(); 

    if(condition2) { 
    return true; 
    } else { 
    print('error'); 
    return false; 
} else { 
    print('error'); 
    return false; 
} 

첫 번째 방법은 나에게 청소기 보이지만, 다른 이상 한 가지 방법을 사용하여 모든 advantage/disadvantage가 있는지 잘 모르겠어요.

Answer 1

두 번째 함수는 condition2가 false 인 경우에도 do_something()을 실행하기 때문에 첫 번째 함수와 첫 번째 함수는 동일하지 않습니다. 서로 다른 오류 메시지를 원하는 가정

if(!condition1 || !condition2) { 
    print('error'); 
    return false; 
} else { 
    do_something(); 
    return true; 
} 

:

어쨌든, 난 내 조건문, 두 가지 예를 결합하는 것을 선호

if(!condition1) { 
    print('error1'); 
    return false; 
} elseif(!condition2) { 
    print('error2'); 
    return false; 
} else { 
    do_something(); 
    return true; 
} 

가 나는 if/elseif 체인에서 그들에 합류했다. 그것은 당신의 의도를 분명하게합니다.

Answer 2

이이이이 http://codereview.stackexchange.com/에 더 좋을 수