2013-06-10 6 views
0

더 명확하고 왜 그럴까요? 논리 아래 코드에서"if"문에 깨끗한 코드를 작성하는 방법은 무엇입니까?

이 동일 (사용 아파치 코 몬즈 라이브러리) :

if(!ArrayUtils.isEmpty(someArray)) { 
    some code here ... 
} 

//or this variant 
if(ArrayUtils.isNotEmpty(someArray)) { 
    some code here ... 
} 

많은 감사합니다!

+2

CodeReview에서 질문하십시오. – mwerschy

+2

두 번째 변형은 더 깨끗합니다. 깨끗한 코드는'!'또는'! ='를 포함해서는 안됩니다. – darijan

+0

네, 제 2 변형을 사용하는 것이 더 명확 합니다만, Java 명명 규칙에 따라 NOT NOT. http://geosoft.no/development/javastyle.html#Specific –

답변

1

옵션 # 2에 투표합니다. 읽을 수 있습니다. Commons가 옵션 # 1을 사용하지 않는 것에 대해 정확히 메소드를 추가했다는 것은 확실합니다.

0

또한 옵션 2가 더 읽기 쉽기 때문에 더 좋다고 생각합니다. 이 두 가지 옵션에서 성능 차이가 나타나지 않습니다.

+0

질문은 성능에 관한 것이 아니라 청결도 즉 가독성에 관한 것입니다. –

2
  1. 두 번째 것 (isNotEmpty 사용)은 일반 영어와 비슷하게 읽습니다.
  2. 사용중인 라이브러리가 필요한 기능을 제공하는 라이브러리를 사용하는 경우 (예 : 해당되는 경우)
  3. 특별한 경우 ArrayUtils에서 array가 비어 있지 않거나 null인지 확인하는 것은 API 디자이너가 별도의 함수를 추가 할 것을 제안하는 경우가 일반적입니다.

바보는 컴퓨터가 이해할 수있는 코드를 작성할 수 있습니다. 좋은 프로그래머는 인간이 이해할 수있는 코드를 작성합니다. - Martin Fowler

+1

계속 : 코드는 작성된 것보다 더 많이 읽혀질 것입니다. 가독성을 위해 작성하십시오. –