pmd14 [디비 파기 | PMD] MisplacedNullCheck MisplacedNullCheck우선순위 : 3 다음의 샘플 코드의 if문 내에 있는 null 체크 구문은 모두 잘못된 위치이다. 이 구문들은 만일 a가 null이라면NullPointerException이 발생한다. 샘플코드112345public class Foo { void bar() { if (a.equals(baz) && a != null) {} }} 샘플코드212345public class Foo { void bar() { if (a.equals(baz) || a == null) {} }} 부연 설명논리 연산에서 &&와 ||는 연산자 앞에 있는 조건을 먼저 확인하고 그 결과에 따라 연산자 뒤에 있는 조건을확인할 지 말 지 결정을 하게 된다. 즉 어떤 상황이건 a.equals(baz)를 먼저 수행하.. 2016. 2. 28. [디비 파기 | PMD] AvoidDecimalLiteralsInBigDecimalConstructor AvoidDecimalLiteralsInBigDecimalConstructor우선순위: 3 사람들은 보통 new BigDecimal(0.1)이 정확히 0.1과 같을 것이라고 생각한다. 하지만 실제로는0.1000000000000000055511151231257827021181583404541015625의 값을 갖는다.이것은 0.1이 정확히 double을 표현하는 수가 아니기 때문이다. 따라서 생성자를 통해 전달되는 숫자형의 값은 기대와는 다르게 0.1과 같지 않게 된다. 반면에 String형을 사용하는 생성자에서는 그 값이 정확하게 일치한다. new BigDecimal(“0.1”)은정확하게 0.1과 같다. 그러므로 일반적으로 문자열을 파라미터로 전달하는 생성자의 사용이 권장되는 것이다. 샘플 코드 1123.. 2016. 2. 28. [디비 파기 | PMD] ClassCastExceptionWithToArray ClassCastExceptionWithToArray특정 클래스가 담긴 Collection 객체로부터 Array를 얻어올 경우 Collection.toArray() 메소드의 파라미터로특정 클래스의 배열형을 넘겨주어야 한다. 그렇지 않으면 ClassCastException이 발생을 한다. 샘플 코드123456789Collection c = new ArrayList();Integer obj = new Integer(1);c.add(obj); // PMD 룰에 위배되며 이 문장을 실행하는 경우 ClassCastException 발생Integer[] a = (Integer [])c.toArray(); // PMD 룰에 위배되지 않으며 정상 처리된다.Integer[] b = (Integer [])c.toArray.. 2016. 2. 28. [디비 파기 | PMD] UnconditionalIfStatement, CollapsibleIfStatements UnconditionalIfStatement조건이 없는 if문은 피하라! 샘플코드12345678910111213public class Foo { public void close() { if (true) { // fixed conditional, not recommended // ... } } } 샘플코드와 같이 if문의 조건절에 true나 false를 직접 대입하는 문장을 은근히 자주보게 된다.논리 전개상 true가 들어가면 무조건 실행되는 문장이고 false가 들어가면 무조건 실행되지 않는 문장이니if문이 무슨 의미가 있는가? if문의 조건절에는 상황에 따라 달라질 수 있는 문장만을 사용하자. CollapsibleIfStatementsif문 합치기 샘플코드123456789101112131415// 원래.. 2016. 2. 28. [디비 파기 | PMD] ReturnFromFinallyBlock외 8건 ReturnFromFinallyBlock Avoid returning from a finally block, this can discard exceptions.finally 블록에서 리턴하는 것을 자제하라. exception 처리가 무시될 수 있다. 샘플 코드123456789101112131415161718192021public class Bar { public String foo() { try { throw new Exception( "My Exception" ); } catch (Exception e) { throw e; } finally { return "A. O. K."; // 이 곳에서 리턴 하는 경우 catch문이 무시되고 정상인 것처럼 리턴 } // 처리된다. } } 다들 아시다시피 fina.. 2016. 2. 28. 이전 1 2 다음 반응형