코드리뷰28 [TIL] 일일 회고 2022/08/12 요즘 한 일 1. 리뷰 정리 https://blogshine.tistory.com/502 [Review] ATDD 4주차 1차 PR 최홍준 리뷰어 님께 감사의 말을 전하고 싶다!! 1. 질문 1-1) 인수테스트 작성과 문서 작성 우선 ATDD 사이클에 의하여 인수테스트 작성 RestDocs 작성 순서로 진행하게 되었습니다. 다만 인수테스트 blogshine.tistory.com https://blogshine.tistory.com/508 [Review] ATDD 4주차 2차 PR 최홍준 리뷰어 님께 감사의 말을 전하고 싶다!! 1. 리뷰 정리 1-1) 최단 경로 구하는 로직 원래 Service에서 최단 경로를 구하는 로직은 다음과 같았습니다. public PathResponse findShortestPa.. NEXT STEP/회고록 2022. 8. 12. [Review] ATDD 3주차 2차 PR 권승철 리뷰어 님께 감사의 말을 전하고 싶다!! 1. 질문 1-1) 공통의 테스트를 만들었어야 하는가? 미션중 TokenAuthenticationInterceptor와 UsernamePasswordAuthenticationFilter 를 추상화 하는 단계에서는, 두 필터에 공통적으로 적용 가능한 테스트를 작성한 후, 공통 부분을 추상화 했어야 할까요? 두 필터의 공통으로적용할 테스트를 작성하기 어렵다 생각되어 새로운 테스트는 작성하지 못하고, 기존의 테스트 코드로 구현하게 된것 같아 아쉽습니다. 리뷰어 님이라면 어떻게 하셨을까요? 의견 궁금합니다!! 답변: 공통의 테스트를 작성하기시 보다는, 구현체 위주의 테스트를 작성한다고 하셨다. 추상 클래스를 기반으로 하는 테스트는 인수테스트 선에서 처리하시는 것 .. NEXT STEP/Review 정리 2022. 8. 3. [Review] ATDD 2주차 3차 PR 권승철 리뷰어 님께 감사의 말을 전하고 싶다!! 1. 질문 1-1) CustomException의 사용 이번에 CustomException을 좀더 세분화 하기위해, 기존의 SectionException을 SectionsAdd, SectionsDelete 별로 나눠 CustomException을 구현하게 되었습니다. 그보다 더 세부적인 내용들은 예외 안에서 메시지로 구별하였습니다! 우선 모든 예외가 공통적으로 상속받을 BusinessException을 구현하였습니다. 해당 class 안에는 HttpStatus를 저장 할 수 있습니다. public class BusinessException extends RuntimeException { private final HttpStatus httpStatus; pu.. NEXT STEP/Review 정리 2022. 7. 25. [Review] ATDD 2주차 2차 PR 권승철 리뷰어 님께 감사의 말을 전하고 싶다!! 1. 질문 1-1) 일급 컬렉션에 대한 재질문 지난번 대화를 통해 sections을 방어적 복사를 해도, sections에 담긴 원소인 section은 변경의 위험이 있음을 인지하게 되었습니다. 따라서 리뷰어님의 의견을 수용하여 Sections.getSections() 메서드는 제거하였지만, "순서대로 역을 조회하는 기능" 때문에 어차피 역들은 변경의 위험에 있을 수 밖에 없지 않나? 란 생각이 들었습니다. 일급컬렉션은 컬렉션의 불변성만 보장하면 됐던 것 아닐까? 란 생각도 들구요 ㅎㅎ 변경 위험의 노출 정도를 줄였다 생각하면 될까요? 이에 대한 의견이 궁금합니다!! 답변: 1-2) 구간 길이 검증의 불가능 미션의 요구사항 중 다음과 같은 사항이 있습니다... NEXT STEP/Review 정리 2022. 7. 22. [Review] 2022/06/23 마지막 PR (Issue Tracker) 이번 리뷰는 Brain 께서 해주셨다! 리뷰해주셔서 감사합니다 !! https://github.com/codesquad-members-2022/issue-tracker/pull/200 [Team-26][BE : Shine] Issue-Tracker 2주차 2회차 PR by zbqmgldjfh · Pull Request #200 · codesquad-members-2022/issue-tracke 안녕하세요 Shine입니다!! 저의 리뷰를 담당해주시게 되어 감사하다는 말씀 전하고 싶습니다. 질문 3가지 우선 로직상의 질문이 아닌, 사용법 에 관한 질문을 드리게 되어 유감스럽게 생각합니다 github.com https://github.com/codesquad-members-2022/issue-tracker/p.. CODE SQUAD/FeedBack 정리 2022. 7. 5. [Review] 2022/06/23 3차 PR (Issue Tracker) 이번 리뷰는 Brain 께서 해주셨다! 리뷰해주셔서 감사합니다 !! https://github.com/codesquad-members-2022/issue-tracker/pull/124#issuecomment-1163833790 [Team-26][BE : Shine] Issue-Tracker 2주차 1회차 PR by zbqmgldjfh · Pull Request #124 · codesquad-members-2022/issue-tracke 안녕하세요! Shine 입니다! 저의 리뷰를 담당해주시게 되어 감사하다는 말씀 전하고 싶습니다. 질문 1가지 1. API에서의 Redirction은? Client 측에서 POST로 자원 생성 요청이 올 경우, Server에서 사용자가 github.com 1. 질문 과연 A.. CODE SQUAD/FeedBack 정리 2022. 6. 24. [Review] 2022/06/21 2차 PR (Issue Tracker) 이번 리뷰는 Brain 께서 해주셨다! 리뷰해주셔서 감사합니다 !! [Team-26][BE : Shine] Issue-Tracker 1주차 2회차 PR by zbqmgldjfh · Pull Request #94 · codesquad-members-2022/issue-tracker 안녕하세요! Shine 입니다! 우선 저의 리뷰를 담당해주시게 되어 감사하다는 말씀 먼저 전하게 됩니다. 질문 2가지 1. DTO 디렉토리의 위치 DTO를 controller → service 로 넘길때도 사용하고, service → con github.com 1. 코드리뷰 1-1) JWT token 응답은 body로! 원래 나는 사용자가 로그인 하면 응답으로 JWT token을 Header에 담아서 보내고 있었다. 리뷰어의 말.. CODE SQUAD/FeedBack 정리 2022. 6. 21. [Review] 2022/06/09 5차 PR (Aribnb 서비스) 이번 리뷰는 Dion 께서 해주셨다! 리뷰해주셔서 감사합니다 !! [Team - 06][BE] 에어비앤비 프로젝트 3주차 1회차 PR by leejohy-0223 · Pull Request #237 · codesquad-members-2022/a 안녕하세요 Dion! 매번 저희 팀의 리뷰를 담당해 주셔서 감사의 인사를 전합니다!! 1. 구현 사항 이번에는 API를 구분하여 각각 구현해보았고, 추가된 부분은 다음과 같습니다. Wish API 구현 Reservation github.com 1. 코드리뷰 1-1) 환경 변수에 대한 고민 내가 스크립트를 짜면서도 Dion과 같은 생각을 했다... 이걸 항상 다 설정해줘야 하나?? .... 이에 대한 고민은 너무 길어지는 것 같아 따로 글로 작성하였다. https.. CODE SQUAD/FeedBack 정리 2022. 6. 12. [Review] 2022/06/01 3차 PR (Aribnb 서비스) 이번 리뷰는 Dion 께서 해주셨다! 리뷰해주셔서 감사합니다 !! https://github.com/codesquad-members-2022/airbnb/pull/140 [Team - 06][BE] 에어비앤비 프로젝트 2주차 1회차 PR by leejohy-0223 · Pull Request #140 · codesquad-members-2022/a 안녕하세요 Dion! 우선 저희 팀의 리뷰를 담당해 주셔서 감사의 인사를 전합니다!! 주요 변경사항으로는 Native Query를 JPQL로 변경한 부분입니다. 수정사항이 많지는 않습니다만, 한 가지 질문이 있 github.com 1. 코드 리뷰 1-1) Spring Actuator 리뷰어 께서 Spring Actuator에 대하여 학습해보길 권해주셨다! 본질.. CODE SQUAD/FeedBack 정리 2022. 6. 4. [Review] 2022/05/27 2차 PR (Aribnb 서비스) 이번 리뷰는 Dion 께서 해주셨다! 리뷰해주셔서 감사합니다 !! https://github.com/codesquad-members-2022/airbnb/pull/88 [Team - 06][BE] 에어비앤비 프로젝트 1주차 2회차 PR by zbqmgldjfh · Pull Request #88 · codesquad-members-2022/airb 안녕하세요 Dion! 우선 저희 팀의 리뷰를 담당해 주셔서 감사의 인사를 전합니다!! 이번 2차 PR을 진행하면서 한가지 고민거리가 있었는데... 이에 대하여 답변해주시면 감사하겠습니다 ㅠ,ㅠ 1. Nat github.com 1. 질문 Native Query 질문 다음 두 메서드 searchByCondition 과 searchByConditionQueryDsl .. CODE SQUAD/FeedBack 정리 2022. 6. 1. [Review] 2022/05/25 1차 PR (Aribnb 서비스) 이번 리뷰는 Dion 께서 해주셨다! 리뷰해주셔서 감사합니다 !! https://github.com/codesquad-members-2022/airbnb/pull/40#discussion_r883130404 [Team - 06][BE] 에어비앤비 프로젝트 1주차 1회차 PR by leejohy-0223 · Pull Request #40 · codesquad-members-2022/ai 안녕하세요 Dion! Shine & Lucid 입니다! 1주차 1회차 PR 리뷰 요청드립니다. 주요 구현 사항 논리적 모델링 물리적 모델링 엔티티 생성 및 연관관계 형성 질문 사항 Shine 1.위치 검색에 대한 질문 사용 github.com 1. 질문 1-1) 위치 검색에 대한 질문 사용자의 위치(위도, 경도)를 중심으로.. CODE SQUAD/FeedBack 정리 2022. 5. 27. [Review] 2022/04/22 2차 PR (반찬 서비스) 이번 리뷰는 wooody 께서 해주셨다! 리뷰해주셔서 감사합니다 !! [Team-12 BE] 1주차 두번째 PR (2022/4/22) by geombong · Pull Request #106 · codesquad-members-2022/sidedish 안녕하세요 리뷰어님 팀12 백엔드 샤인&검봉 입니다. 저희 팀의 리뷰를 담당해주셔서 우선 감사의 말 전합니다! 1차 PR 리뷰 수정사항 categoryService 에서 pageId 받는 부분 수정하기 schema 처음과 끝 f. github.com 1. 질문 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 1) 요구사항에 없는 아이템 등록(상품 등록) 같은 기능 또한 고려하여 DB 설계를 진행해야 하는지 궁금합니다. 2) .. CODE SQUAD/FeedBack 정리 2022. 4. 27. 이전 1 2 3 다음
