개발자 기록22 [Review] ATDD 3주차 2차 PR 권승철 리뷰어 님께 감사의 말을 전하고 싶다!! 1. 질문 1-1) 공통의 테스트를 만들었어야 하는가? 미션중 TokenAuthenticationInterceptor와 UsernamePasswordAuthenticationFilter 를 추상화 하는 단계에서는, 두 필터에 공통적으로 적용 가능한 테스트를 작성한 후, 공통 부분을 추상화 했어야 할까요? 두 필터의 공통으로적용할 테스트를 작성하기 어렵다 생각되어 새로운 테스트는 작성하지 못하고, 기존의 테스트 코드로 구현하게 된것 같아 아쉽습니다. 리뷰어 님이라면 어떻게 하셨을까요? 의견 궁금합니다!! 답변: 공통의 테스트를 작성하기시 보다는, 구현체 위주의 테스트를 작성한다고 하셨다. 추상 클래스를 기반으로 하는 테스트는 인수테스트 선에서 처리하시는 것 .. NEXT STEP/Review 정리 2022. 8. 3. [Review] ATDD 3주차 1차 PR 권승철 리뷰어 님께 감사의 말을 전하고 싶다!! 와 개인적으로 이번 3주차는 Spring Security를 비슷하게 만든 뼈대 코드 위에서 인증 처리 테스트를 작성하는데, 진짜 어려우면서 꿀잼이다. 아직 Spring Security를 공부해본적은 없지만, 큰 틀을 배워가는 것 같아 너무 재미있다. 1. 질문 1-1) 인터셉터 처리 방식 public void addInterceptors(InterceptorRegistry registry) { registry.addInterceptor(new SecurityContextPersistenceFilter()); registry.addInterceptor(new UsernamePasswordAuthenticationFilter(loginMemberService).. NEXT STEP/Review 정리 2022. 8. 1. [TIL] 일일 회고 2022/07/25 요즘 한 일 1. ATDD 리뷰 정리 2차, 3차 PR에 대한 리뷰를 정리하였다. 부족한 부분이야 항상 많지만, 다양한 주제로 리뷰어와 예기할 수 있어 좋다! https://blogshine.tistory.com/479 [Review] ATDD 2주차 2차 PR 권승철 리뷰어 님께 감사의 말을 전하고 싶다!! https://github.com/next-step/atdd-subway-path/pull/299 2단계 - 지하철 구간 추가 리팩터링 by zbqmgldjfh · Pull Request #299 · next-step/atdd-subway-pat.. blogshine.tistory.com https://blogshine.tistory.com/482 [Review] ATDD 2주차 3차 PR 권승철 .. NEXT STEP/회고록 2022. 7. 25. [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/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. [TIL] 일일 회고 2022/06/08 요즘 한 일 1. Aribnb 클론 코딩 진행중 어느덧 이번 과제의 마지막 3주차가 끝나가고 있다... 시간 너무 빠른것 같다... 이번 과제를 진행하면서 부족하다 느낀점도 많고, 배워가는 점도 많은것 같다. 매번 다양한 문제를 경험하고 있어 만족도는 높다! https://github.com/codesquad-members-2022/airbnb/pull/203 [Team - 06][BE] 에어비앤비 프로젝트 2주차 2회차 PR by zbqmgldjfh · Pull Request #203 · codesquad-members-2022/air 안녕하세요 Dion! 우선 저희 팀의 리뷰를 담당해 주셔서 감사의 인사를 전합니다!! 질문이 한가지 있습니다! 구현 사항 이번에 구현한 사항은 다음과 같습니다. GitH.. CODE SQUAD/회고록 2022. 6. 8. [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. [TIL] 일일 회고 2022/05/22 요즘 한 일 1. ByteCode 조작해보기 소스코드를 컴파일하면 바이트 코드가 나오게 되는데, 이를 ByteBuddy를 통해 조작함으로써 신기한 기술을 몇가지 사용해 봤다. [Java] 바이트코드 조작하기 소스코드의 커버리지를 측정하는, 가령 Jacoco와 같은 tool들이 바이트코드를 이용하여 측정한다. 이러한 소스코드 커버리지는 내 전체 소스코드에서 얼만큼을 테스트로 커버 했는지? 를 알려주는 blogshine.tistory.com 이러한 기술을 사용하는 도구들주 예로 Jacoco가 있다. Jacoco는 코드 커버리지를 측정하는 도구인데, 바이트 코드상에 가감을 통해 이를 측정한다고 한다. 실로 놀라운 기술이다. 2. Reflection 공부하기 [Java] Reflection 이번 글은 Sprin.. CODE SQUAD/회고록 2022. 5. 22. 이전 1 2 다음
