[자바 코딩의 기술] 1장 : 우선 정리부터

1.1 불필요한 비교 피하기

microscope.isInorganic(sample) 이 Bool을 반환한다면 불필요하게 True, False로 비교해서 조건문을 체크하지 말고 간단하게 없애자.

if(microscope.isInorganic(sample) == true) 
if(microscope.isInorganic(sample) == false)

보단

if(microscope.isInorganic(sample))
if(!microscope.isInorganic(sample)) 를 사용하자.

1.2 부정 피하기

1. 메서드가 반환 값으로 False(부정)을 반환하게 되면 한번 더 생각하며 이해해야 한다.

2. 반환 값이 긍정인 것을 굳이 `!`을 이용해서 부정으로 바꾸면 한번 더 생각하며 이해해야 한다.

 

따라서, 1번의 경우에는 긍정 표현의 메서드를 호출하고 2번 경우는 `!`를 삭제하고 if와 else의 위치를 바꿔서 사용하자.

 

일반적인 경험에 따르면 부정적 메서드는 헷갈린다.

또한 코드 확장시에도 긍정표현이 훨씬 좋다!!

 

직관적으로 이해하기 쉽게 긍정 표현을 사용하도록 하자. (긍정 반환값)

 

1.3 불 표현식을 직접 반환

불 표현식을 반환할 때는 if문을 덕지덕지 사용하지 말고 그냥 조건 자체를 반환하자.

boolean isValid() {
	return mission >= 0 && name != null;
}

1.4 불 표현식 간소화

2.1의 규칙을 지키게 되면 조건문이 너무 길어져 이해하기 힘든 수준이 된다.

 

조건을 추상화할 수 있는 메서드를 만들어서 사용하도록 하자.

 

메서드를 만들 때 조건을 추상화할 수 있는 의미있는 이름을 지으면 좋다.

 

이렇게 하면 코드의 줄이 증가할지라도 코드 이해도는 훨씬 향상된다.

 

메서드를 만들어서 불 표현식에 의미를 제공하자!!

 

1.5 조건문에서 NullPointerException 피하기

메서드의 인수를 검증할 때는 순서가 가장 중요하다.

 

반드시 null을 먼저 확인한 후 도메인에 따라 "유효하지 않은" 값을 검사하도록 하자.

 

-> JAVA8의 Optional을 사용하면 NULL처리하기 좋다.

 

1.6 스위치 실패하기

스위치 문의 Case에 의도적으로 break를 뺐다면 주석을 통해 다른 개발자들에게 알려주자.

 

이것은 오류를 유발할 수 있기 때문이다.

1.7 항상 괄호 사용하기

자바는 파이썬이 아니다!!!

항상 중괄호를 사용해서 조건문의 범위를 확실히 지정하자! (버그 발생 확률을 저하시킨다)

 

오류를 찾기도 가독성도 훨씬 좋아진다.

1.8 코드 대칭 이루기

if, else if, else if 사용 시 각각의 조건이 서로 비슷한 관심사를 표현하는지 생각해보자.

 

만약, 하나의 분기는 "권한을 부여하지 않는 코드"이고 나머지는 "권한을 부여하는 코드"라면
두 가지를 다른 조건문을 나눠서 사용하도록 하자 -> if {} // if {} else if{}

 

이렇게 하면 코드의 대칭성을 유지할 수 있다.

1.9 1장에서 배운 내용

불필요한 코드를 피하는 법에 대해서 배웠다.