최근 받은 코드리뷰들에 대한 정리 및 생각들

근래 감사하게도 코드리뷰받을 기회가 많았다. 정리해서 리팩토링에 적용해보려고 한다

스마게 데브 캠프 코드 리뷰

 

인증 필터 처리에서, Header가 비어있을 수도 있다.

이와 같이, NPE가 안 나도록 검증하는 것은 매우 중요하다.

최근 effective java에서도 관련된 내용이 많이 나왔다.

 

나 왜 이런 코드짯지..? SRP원칙을 지키자.

+ 메서드가 단순히 다른 메서드를 return 하는 상태라면, 과감하게 그 메서드를 합치자

 

VO가 무엇인지 정리해보았다.

DTO는 데이터 전달용 즉 Data Transfer Object

데이터를 전달하기 위해 사용하는 객체이며 계층 간 데이터를 전달하기 위한 객체이다.

오직 getter/setter 메서드만 갖는다.

다른 로직을 갖지 않는다.

 

VO는 값 표현용

값 그 자체로 비교되는 것.

지폐 같은 고유번호가 다르더라도 다른 만원이 아님.

각 객체의 주소처럼

생성자를 통해서 값을 초기화한다.

DTO와는 달리 로직을 포함할 수 있다.

완전한 VO를 만들기 위해 hashcode & equals 오버라이딩해야 한다

 

Transaction도 정리할 필요가 있다

 

반성한다... 알고 있는 내용에 대한 이유를 설명할 줄 알아야 한다.

 

다른 프로젝트의 코드리뷰

• 요구사항을 잘 파악하고, 연관관계를 매핑할 줄 알아야 한다.
• 명세서는 어느 무엇보다도 공들여서 작성하자(코드와 다른 부분이 없도록)
• service 함수 내에서 if문을 나열하셨는데 이를 도메인 내 함수로써 처리하기.
• 서비스가 바뀌었을 때의 동적 대응을 생각해 보기

단순 코드의 문제가 아니다. 설계를 잘하는 것도, 프로그래머의 중요한 자질임을 깨달았다.