[디비 파기 | PMD] UnconditionalIfStatement, CollapsibleIfStatements

UnconditionalIfStatement

조건이 없는 if문은 피하라!

샘플코드

1 2 3 4 5 6 7 8 9 10 11 12 13

public class Foo {       public void close() {           if (true) {  // fixed conditional, not recommended               // ...           }       }   }

샘플코드와 같이 if문의 조건절에 true나 false를 직접 대입하는 문장을 은근히 자주보게 된다.

논리 전개상 true가 들어가면 무조건 실행되는 문장이고 false가 들어가면 무조건 실행되지 않는 문장이니

if문이 무슨 의미가 있는가?

 

if문의 조건절에는 상황에 따라 달라질 수 있는 문장만을 사용하자.

CollapsibleIfStatements

if문 합치기

샘플코드

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

// 원래의 코드 void bar() {     if (x) {             if (y) {             // do stuff         }     } }   // 수정된 코드 void bar() {     if (x && y) {         // do stuff     } }

두 개의 중첩된 if문은 종종 논리연산자를 통해 하나의 구문으로 만들어질 수 있다.

코드의 효율성과 가독성을 위해 가능하다면 하나의 if문으로 만들어 사용하여야한다.