자바 접을 수있는 if 문

Question

코드를 확인하려면 PMD을 사용합니다. 대부분의 경우에 매우 유용한 힌트를 제공하지만 다음 시나리오에서 개선 할 수있는 것을 파악할 수는 없습니다.

if ((getSomething() != null && getSomethingElse() != null) 
    || (getSomething() == null && getSomethingElse() == null)) 
{ 
    ... 
} 

PMD는 저에게 말한다 :

원래의 코드는 다음과 같이 보입니다

때로는 두 개의 문은 부울 단락 운영자 자신의 조건을 분리하여 통합 할 수있다 '가' .

편의를 위해 부울 변수로 a와 b를 사용합시다. 그런 다음이 코드 조각은 다음과 같습니다 :이 다음 중 하나에 변환 할 수

if ((!a && !b) || (a && b)) 

:

if ((!a || b) && (a || !b)) 
if (!(a^b)) 

마지막

if (a==b) 

을 그래서

에 내 코드를 단순화

if ((getSomething() == null) == (getSomethingElse() == null)) 

그러나 PM D는 불평을 계속합니다 (사실 세 가지 버전 모두에 대해). 이것은 거짓 긍정입니까, 아니면 if-condition을 작성하는 더 좋은 방법입니까? 개인적으로, 나는 == null의 경우를 나는이 (가) ==가 null와 b 처리 할 수 ​​있도록이 이전에 if 문을 확인 널을 할 거라고 ..although

Answer 1

문제는 다른 것이 었습니다. (코드 검증 - 방법에서 유래) 경우 경우 문이 또 다른 내부의 코드 만했다 다음 PMD-메시지가 의미

if (...) 
{ 
    ... 
} 
else if (...) 
{ 
    ... 
} 
else if (...) 
{ 
    if ((getSomething() == null) == (getSomethingElse() == null)) 
    { 
     ... 
    } 
} 

을, 나는의 조건을 결합 할 수 있다는 것입니다 마지막으로 다른-경우와 내부의 경우 절 : 나는이 작업을 수행 할 수 있습니다 경우 원래 버전이 이해하기가 훨씬 명확하게 보이기 때문에

if (...) 
{ 
    ... 
} 
else if (...) 
{ 
    ... 
} 
else if (... && ((getSomething() == null) == (getSomethingElse() == null))) 
{ 
     ... 
} 

는 그러나, 나는, 확실하지 않다.

Answer 2

if ((a != null) && (b != null) && (a==b)) 

개별적으로

Answer 3

큰 조건 블록은 논리적으로 어렵다는 것이 문제입니다.

OTOH, 모든 경고 PMD 방출 필요가 없습니다 - ROI를 고려해야합니다. 리팩토링이나 구조 조정이 가치있는 일인가? 동일한 기능을 다른 방식으로 처리 할 수 ​​있습니까?

가치가없는 경우 신경 쓰지 마세요.