이 코드 스멜을 보고 처음에는 뭐가 잘못된지도 몰랐다. 애초에 equals() 메서드의 반환 타입이 boolean 이기 때문에 if문을 사용할 필요가 없었습니다. 앞으로는 equals() 메소드 사용 시, 주의해야 겠습니다.
public boolean isActive() {
return !this.status.equals(INACTIVE);
}
'항해99' 카테고리의 다른 글
항해99 77일차 TIL1 - @Transactional(readOnly = true) (0) | 2023.03.11 |
---|---|
항해99 76일차 TIL1 - 정적 팩토리 메서드 (0) | 2023.03.10 |
항해99 74일차 TIL1 - 오토 박싱 & 오토 언박싱 (feat. 불필요한 연산을 줄이자) (0) | 2023.03.08 |
항해99 73일차 TIL1 - NGINX 적용하기(feat. chat GPT) (0) | 2023.03.06 |
항해99 72일차 TIL1 - 프로젝트 슬랙 web hook 적용하기 (0) | 2023.03.04 |