최근 코드 리뷰에 대한 요약 및 생각

다행히도 최근에는 코드 리뷰를 받을 기회가 많아졌습니다.

정리해서 리팩토링에 적용하려고 합니다.

Smage Dev Camp 코드 리뷰


인증 필터 처리에서 헤더는 비어 있을 수 있습니다.

따라서 NPE가 발생하지 않는지 확인하는 것이 매우 중요합니다.

최근 효과적인 자바에 관련 내용이 많이 등장하고 있다.


이 코드를 작성한 이유는 무엇입니까? SRP 원칙을 따르자.

+ 메서드가 단순히 다른 메서드를 반환하는 경우 해당 메서드를 대담하게 병합합니다.


VO가 무엇인지 살펴보겠습니다.

DTO는 데이터 전송, 즉 데이터 전송 개체입니다.

데이터를 전달하는 데 사용되는 개체와 레이어 간에 데이터를 전달하는 개체입니다.

getter/setter 메서드만 있습니다.

다른 논리는 없습니다.

VO는 가치 표현을 의미합니다.

자신과 비교되는 값입니다.

와 같이 고유번호가 달라도 나. 지폐로 보면 또 만원이 아니다.

각 개체의 주소와 같이

생성자를 사용하여 값을 초기화합니다.

DTO와 달리 논리를 포함할 수 있습니다.

전체 VO를 생성하려면 해시코드 및 같음을 재정의해야 합니다.


트랜잭션도 정렬해야 합니다.


반성… 당신이 알고 있는 것에 대한 이유를 설명할 수 있어야 합니다.

다른 프로젝트의 코드 검토

  • 요구 사항을 잘 이해하고 관계를 매핑하는 방법을 알고 있어야 합니다.

  • 가장 중요한 것은 사양을 신중하게 작성하는 것입니다(코드와 다르지 않도록).
  • 서비스 함수에 if 문을 나열했지만 도메인 내의 함수로 취급합니다.

  • 서비스 변경 시 동적인 대응 고려

코드만의 문제가 아닙니다.

저는 좋은 디자인이 프로그래머에게 중요한 자질이라는 것을 깨달았습니다.