CodeReview27 [Review] ATDD 4주차 2차 PR 최홍준 리뷰어 님께 감사의 말을 전하고 싶다!! 1. 리뷰 정리 1-1) 최단 경로 구하는 로직 원래 Service에서 최단 경로를 구하는 로직은 다음과 같았습니다. public PathResponse findShortestPath(Long source, Long target, PathCondition pathCondition) { Station upStation = stationService.findById(source); Station downStation = stationService.findById(target); List lines = lineService.findLines(); SubwayMap subwayMap = new SubwayMap(lines); Path path = subwayMap.. NEXT STEP/Review 정리 2022. 8. 12. [Review] ATDD 1주차 1차 PR NextStep에서의 첫 리뷰다! 송용주 리뷰어 님께 감사의 말을 전하고 싶다!! 1. 리뷰 정리 1-1) Private 메서드는 public 밑으로 원래 내 코드들을 보면 Private 메서드들은 public 아래 뒀었다. 근런데 이번에 왠지 모르게 위에 두고 싶었다? ㅎㅎ 바로 리뷰에서 말씀을 주셔서 다시 원상복귀 시켜야 겠다 ㅎㅎ!! 1-2) 상태코드 검증하기 다만 의문이 있는데... 1. 상태코드 확인은 부차적인것이 아닐까? 사실 인수조건(시나리오)을 봤을때 When 그 지하철역을 삭제하면 Then 그 지하철역 목록 조회 시 생성한 역을 찾을 수 없다 then 절에 해당되는 생성한 역을 찾을수 있는지가 핵심이라 생각되는데, 이때 When에서 수행한 행동의 상태코드를 꼭 검증해야 하나? 라는 생각이.. NEXT STEP/Review 정리 2022. 7. 8. [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/04 4차 PR (Aribnb 서비스) 이번 리뷰는 Dion 께서 해주셨다! 리뷰해주셔서 감사합니다 !! https://github.com/codesquad-members-2022/airbnb/pull/203#discussion_r891396646 [Team - 06][BE] 에어비앤비 프로젝트 2주차 2회차 PR by zbqmgldjfh · Pull Request #203 · codesquad-members-2022/air 안녕하세요 Dion! 우선 저희 팀의 리뷰를 담당해 주셔서 감사의 인사를 전합니다!! 질문이 한가지 있습니다! 구현 사항 이번에 구현한 사항은 다음과 같습니다. GitHub, Kakao OAuth Login 구현 JWT Token 기 github.com 1. 질문 1-1) URL의 설계 개발을 진행하던 중 URL설계에 대.. CODE SQUAD/FeedBack 정리 2022. 6. 9. [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. [TIL] 일일 회고 2022/04/17 ~ 4/22 요즘 한 일 1. 백기선님 MVC 강의 완강 보통 영한님 강의는 들으면서 나만의 방식으로 블로그에 전부 자세하기 요약정리를 하는 편인데, 기선님의 상의는 정리를 하면서 학습하지 못한것이 매우 아쉽다... 코드스쿼드 과제와 병행하기 좀 힘들다 생각하여 하지 못했다. 듣는거라도 하자는 마인드로 일단 듣기시작한 MVC강의를 드디어 완강하게 되었다! 완강하기까지 약 한달정도 걸린것 같다? 강의를 들으면서 WebMvcConfig의 사용과 dispatherServlet의 등록방식에 대한 부분이 정말 좋았다. 이후 코드스쿼드 과정이 끝난다면 2회독을 하며 강의내용을 다시 정리해볼 생각이다. 2. 코드 리뷰 정리하기 리뷰를 받은 내용중 필요한 내용을 선별적으로 정리하였다. 이런식의 피드백을 지속적으로 받는점이 매우 좋.. CODE SQUAD/회고록 2022. 4. 22. [Review] 2022/04/20 1차 PR (반찬 서비스) 이번 리뷰는 wooody 께서 해주셨다! 리뷰해주셔서 감사합니다 !! 1. 질문 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 팀원과 Spring Data Jdbc를 사용하던 도중, OneToMany 관계를 이용할 때 Map을 통해 저장하도록 하였었습니다. Category : item = 1 : N 의 관계로 말이죠! 하지만 해당 Map을 이용할 경우 어그리게이트 루트에 해당되는 Category는 auto_increment가 잘 적용되지만, item은 auto_increment가 정상작동하지 않았습니다. 혹시 Map을 사용할때는 id값을 직접 지정해줘야 하나요? @Test public void itemSaveTest() { // given Category mainCate.. CODE SQUAD/FeedBack 정리 2022. 4. 21. 이전 1 2 3 다음
