[1주차] 피어 리뷰 일지

📜 피어 리뷰 가이드

1️⃣ : Commit은 기능 구현마다 한 번씩 나누어서 진행하기!
2️⃣ : 정답 코드 대신 참고 자료 올려주기!
3️⃣ : 명령조가 아닌 의견 제시형으로 리뷰하기!
4️⃣ : 코드 컨벤션 고려하여 리뷰하기!
5️⃣ : 자신의 코드와 비교하여 리뷰하기!
6️⃣ : 누구나 봐도 이해하기 쉬운 클린코드 리뷰하기!

🌟 준영님

📃 Mission Record

매 기능 구현마다 참고하신 레퍼런스를 첨부해주신 부분이 인상깊었다.

💻 Project Code

1️⃣ BaseEntity 부분

➡️ 생성일과 수정일만 포함시켰는데 생각해보니 id 도 BaseEntity 에 포함될 수 있다!

➡️ @EntityListeners(AuditingEntityListener.class) => 생성일과 수정일을 자동으로 생성해주는 어노테이션 (🔗참고)

 

2️⃣ Test 코드 작성

    @Test
    void modifyPassword() {
        // given
        Member foundMember = memberService.findByUsername("testUser");
        String rawPassword = "testUser1234!";
        String changePassword = "abcdefu1234!";

        // (given check)
        assertThat(passwordEncoder.matches(rawPassword, foundMember.getPassword())).isTrue();

        // when
        memberService.changePassword(foundMember, changePassword);

        // then
        assertThat(passwordEncoder.matches(changePassword, foundMember.getPassword())).isTrue();
    }

➡️ 아주 짧은 주석이지만 가독성을 확 높여주는 것 같았다. 적절한 주석 사용의 예시였다.

🌟 동현님

📃 Mission Record

프로젝트 시작 전 기본 구성을 충분히 생각하시고 진행하신게 눈에 보였다. 구성대로 진행하면서 스스로 고찰하신 부분들도 인상깊었다.

💻 Project Code

① 해시태그 등 여러가지 요소에서 stream 을 사용하신 모습이 가독성이 좋았다.

List<String> keywordContents = Arrays.stream(keywordContentsStr.split("#"))
        .map(String::trim)
        .filter(s -> s.length() > 0)
        .collect(Collectors.toList());

② Optional 에서 isPresent&get 과 orElse 의 적절한 사용. (🔗참고1) (🔗참고2)

🌟 석영님

📃 Mission Record

코드와 함께 특정 어노테이션이나 로직의 사용 이유를 적어주셔서 좋았음. 다른 기능 구현에 대한 내용도 담겨 있었다면 더 좋았을 듯!

💻 Project Code

🌟 상원님

📃 Mission Record

lombok 이 무작정 좋다고만 생각했는데 Setter 에 대한 고찰이 인상 깊었다.

@Setter를 사용하여 Entity를 직접 다루는 행위 금지
① 객체의 일관성을 보장하기 어려움
② Setter를 사용 할 경우 그 의도를 알기 어려운 경우 존재

💻 Project Code

① BindingResult 로 1차, Optional 객체 확인을 통해 로그인한 멤버인지 2차 유효성을 확인했는데 @isauthenticated 로 2차 유효성은 해결되는게 아닌지 궁금하다.

 

📚 생각 정리 & 남겨주신 리뷰 내용 피드백

1️⃣ 생갹 정리

아는 만큼 보인다고 아직 회원 관련 기능이나 글 부분에서도 해시태그 기능을 구현하지 못했더니 나도 구현한 코드 외에는 선뜻 알아보고 리뷰 하기가 쉽지 않았다. 피어리뷰 과정 자체는 힘들지만 확실히 다른 사람의 코드를 봐야 더 좋은 코드를 작성할 수 있는 것 같다.

2️⃣ 리뷰 피드백

준영님께서 DTO 와 VO 사용에 대한 피드백을 남겨주셨다.

https://maenco.tistory.com/entry/Java-DTO%EC%99%80-VO%EC%9D%98-%EC%B0%A8%EC%9D%B4

[JAVA] DTO와 VO의 차이

과거 내 블로그에도 작성했던 내용인데 제대로 이해하지 못한 상태에서 썼던 것 같다..! 글을 읽고 리팩토링 한 뒤 기존 글을 수정해보아야겠다!