코드리뷰정리24 [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 3주차 3차 PR 권승철 리뷰어 님께 감사의 말을 전하고 싶다!! 1. 질문 1-1) Bearer 인증방식에서 권한이 없을 경우 public abstract class AuthenticationChainingFilter implements HandlerInterceptor { @Override public boolean preHandle(HttpServletRequest request, HttpServletResponse response, Object handler) throws Exception { if(isAlreadyLoginUser()) { return true; } try { AuthenticationToken token = convert(request); UserDetails userDetails = findU.. NEXT STEP/Review 정리 2022. 8. 4. [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 1주차 3차 PR 송용주 리뷰어 님께 감사의 말을 전하고 싶다!! 1. 질문 @BeforeEach void init() { upStationId = 지하철역_생성("강남역").jsonPath().getLong("id"); downStationId = 지하철역_생성("건대입구역").jsonPath().getLong("id"); } @Test @DisplayName("정상적으로 역을 생성하여 기존 노선 끝에 구간을 추가한다") public void addSection() { // given Long lineId = 지하철_노선_생성("2호선", "bg-blue-600", upStationId, downStationId, 10).jsonPath().getLong("id"); Long stationId = 지하철역_생성("성수역.. NEXT STEP/Review 정리 2022. 7. 15. [Review] ATDD 1주차 2차 PR 송용주 리뷰어 님께 감사의 말을 전하고 싶다!! 1. 질문 1-1) Dto -> Domain 변환의 장소는? 조금 인수테스트 와는 상관없는 Production Code에 대한 부분이기는 한데... 저의 Line 이라는 Entity 안에는 다음과 같이 edit 이라는 메서드가 구현되어 있습니다. public void edit(Line line) { if (line.getName() != null && !line.getName().isBlank()) { name = line.getName(); } if (line.getColor() != null && !line.getColor().isBlank()) { color = line.getColor(); } if (line.getUpStationId() != nu.. NEXT STEP/Review 정리 2022. 7. 10. [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/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 다음
