코드리뷰28 [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. [Review] 2022/04/10 2차 PR 1. 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 1. 기존 순서와 겹치는 경우 순서가 겹치는 경우 해당 1번이 뒤로 밀리고, 그 사이에 삽입 되도록 하였다. 원래의 순서 1번은 2번이 되게 될 것 이다! 2. 초기화 메서드 추출하기 초기화에 해당되는 부분이다. 위 코드는 생성자에서 한번 호출되기 때문에 사실 따로 분리시키지는 않았었다. 리뷰어님의 말씀처럼 메서드로 뽑을 수 도 있지만, 그냥 명시적으로 생성자 안에서 초기화 하고 있음을 보이는 것 또한 좋다 생각한다! 3. 로직을 함수로 더 분리하기 함수 안에 여러 로직들이 퍼져 있다. 관련된 하나의 덩어리 들 끼리 메서드로 뽑아 주어야 할것 같다. CODE SQUAD/FeedBack 정리 2022. 4. 14. [Review] 2022/04/06 1차 PR 1. 질문 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 팀원과 처음 Work라는 도메인 모델을 만들때 2가지 방식의 생성방법에 대하여 논의 하게 되었습니다. 생성자 방식 빌더패턴 1) 생성자 이용하기 public class Work { // 생략... public Work(String title, String content, String author) { this.title = title; this.content = content; this.author = author; this.workStatus = WorkStatus.TODO; // 초기값 자동 지정 this.createDateTime = LocalDateTime.now(); // 초기값 자동 지정 this.las.. CODE SQUAD/FeedBack 정리 2022. 4. 10. [Review] 웹서버 4단계 - 쿠키를 이용한 로그인 구현 (2022/03/30) [K & Shine] 웹서버 4단계 - 쿠키를 이용한 로그인 구현 by zbqmgldjfh · Pull Request #54 · codesquad-members-2022 안녕하세요 리뷰어님!! K, Shine 팀 입니다. 저희 팀의 리뷰를 해주셔서 감사합니다!!! 기존에 Servlet에 대한 공부를 해본적이 있어, 참고해 가면서 구현하려 노력하였습니다. Step04 TODO list Servlet inter github.com 1. 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 이번 리뷰는 honux가 해주셨다!! 아프신 와중에 리뷰해주셔서 감사합니다!! 1. Session Key를 저장할때 생성하기 원래 우리의 코드는 sessionDB에 저장을 할 때, sa.. CODE SQUAD/FeedBack 정리 2022. 4. 1. [Review] 웹서버 3단계 - POST로 회원 가입 (2022/03/29) [Shine & 케이] 웹서버 3단계 - POST로 회원 가입 by zbqmgldjfh · Pull Request #43 · codesquad-members-2022/java-was KShine 의 3단계 결과 안녕하세요 리뷰어님 저희팀을 리뷰해 주셔서 먼저 감사의 말을 전합니다 ㅎㅎ!! 이번 과제에서는 Request, Response 객체를 만들려 노력하였습니다. 특히 Response 같은 경우 forward( github.com 이번 과제는 팀원과 함께 pair 프로그래밍을 통해 HTTP 서버를 만드는 과제였습니다. 1, 2 단계는 간단하여 reivew 가 적었기에 본격적인 부분인 3단계 부터 리뷰정리를 하게 되었습니다. 1. 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보.. CODE SQUAD/FeedBack 정리 2022. 3. 29. [Review] 스프링 카페 5단계 - 게시글 권한부여 (2022/03/17) [Shine] 스프링 카페 5단계 - 게시글 권한부여 by zbqmgldjfh · Pull Request #94 · codesquad-members-2022/java-sprin 안녕하세요! Shine 입니다! 우선 리뷰를 해주시는 리뷰어 님께 감사의 인사를 전합니다! 이번 과제를 수행하며... 이번주가 마지막 Spring 카페 구현 기간이라 그런지 좀 급급하게 구현한 기분이 드 github.com 1. 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 1. 반환값에 대하여 생각해보기 우선 Long을 사용한 가능 큰 이유는 "정상적으로 삭제 되었는지 확인하고 싶다" 였다. 조금 다른 이야기 이기인데.. 그 학교 시스템프로그래밍 시간에 System call 들 공부할.. CODE SQUAD/FeedBack 정리 2022. 3. 25. [Review] 스프링 카페 4단계 - 로그인 구현 (2022/03/15) [Shine] 스프링 카페 4단계 - 로그인 구현 by zbqmgldjfh · Pull Request #73 · codesquad-members-2022/java-spring-cafe 안녕하세요! Shine 입니다! 우선 리뷰를 해주시는 리뷰어 님께 감사의 인사를 전합니다! 기능 구현 DbTemplate(JdbcTemplate 구현해보기) 우선 이번 과제는 지난번 Roach가 반영해주신 namedParameter를 적용하 github.com 우선 이번 리뷰를 해주신 우아한 형제들의 Roach에게 감사 인사를 전합니다! 1. 질문 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 질문 1) LoginControllerTest LoginControllerTest 를 작성하는데 .. CODE SQUAD/FeedBack 정리 2022. 3. 15. [TIL] 일일 회고 2022/03/11 오늘 한 일 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 1. 코드 리뷰 정리 오늘은 배민의 roach 가 피드백을 해주셨다! [Review] 스프링 카페 3단계 - DB에 저장하기 (2022/03/11) [Shine] 스프링 카페 3단계 - DB에 저장하기 by zbqmgldjfh · Pull Request #46 · codesquad-members-2022/java-spring-ca 안녕하세요 Shine 입니다! 먼저 리뷰를 남겨주시는 리뷰어님께 감사한 마음을 전합니.. blogshine.tistory.com 2. 마주친 예외 정리 [junit5] MockMvc에서 NestedServletException 통과시키기 직접 경함한 오류를 해결한 과정을 정리한 글 입니.. CODE SQUAD/회고록 2022. 3. 11. [Review] 스프링 카페 3단계 - DB에 저장하기 (2022/03/11) [Shine] 스프링 카페 3단계 - DB에 저장하기 by zbqmgldjfh · Pull Request #46 · codesquad-members-2022/java-spring-ca 안녕하세요 Shine 입니다! 먼저 리뷰를 남겨주시는 리뷰어님께 감사한 마음을 전합니다. Step03 ToDo-List user form 검증 로직 작성하기 (BeanValidation 사용하지 말것!) article form 검증 로직 작성하기 (BeanVal github.com 1. 질문 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 질문 1 디렉터리 구조의 개편이 필요한것 같습니다. Repository가 너무 복잡해지는 것 같은데 어떻게 디렉토리 구조를 개편해야 할까요? 질문 2 저장.. CODE SQUAD/FeedBack 정리 2022. 3. 11. [Review] 스프링 카페 2단계 - 글 쓰기 기능 구현 (2022/03/08) 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 이번 시간에는 브라이언이 리뷰를 해주셨다! [Shine] 스프링 카페 2단계 - 글 쓰기 기능 구현 by zbqmgldjfh · Pull Request #24 · codesquad-members-2022/java-spr 안녕하세요! 리뷰어님! 저의 과제물을 리뷰 해주셔서 정말 감사합니다!! 이번에는 ArticleController에 대한 간략한 Unit test를 함께 작성해 보았습니다. 이전번에 호눅스가 리뷰해 주신부분은 모두 수 github.com 다만 피드백 해주신 점을 빠르게 수정한 후, 팀원들의 피드백 또한 정리해 봐야겠다. 질문 1 저는 Article을 Form으로 부터 넘겨받을때 다음 필드에 대하여만 .. CODE SQUAD/FeedBack 정리 2022. 3. 8. [Review] 스프링 카페 1단계 - 회원 가입 및 목록 기능 (2022/03/03) 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 이번 시간에는 호눅스가 리뷰를 해주셨다! [Shine] 스프링 카페 1단계 - 회원 가입 및 목록 기능 by zbqmgldjfh · Pull Request #9 · codesquad-members-2022/ja 안녕하세요! Shine 입니다! 리뷰를 해주시는 리뷰어님들께 감사의 말 먼저 전합니다 ㅎㅎ!! 우선 commit이 너무 막 이루어진 점 죄송합니다. 수정이 빈번하다 보니... 적당한 시점에 commit을 항상 하 github.com 우선 질문했던 2가지 에 대한 답변부터 살펴보자. 질문 1 우선 저의 repository는 Map을 사용하고 있습니다. (User를 저장할때 Map에 전달하는 Key값인 ID 와.. CODE SQUAD/FeedBack 정리 2022. 3. 3. 이전 1 2 3 다음
