CodeReview27 [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] 5단계 - 동적 HTML (2022/04/03) [케이 & Shine] 5단계 - 동적 HTML by zbqmgldjfh · Pull Request #65 · codesquad-members-2022/java-was 안녕하세요 리뷰어님!! K, Shine 팀 입니다. 저희 팀의 리뷰를 해주셔서 감사합니다!!! 질문 저희 팀이 로그인 처리를 위해서 LoginFilter 를 구현하게 되었는데, 기능은 정상 작동 하지만, 전반적으로 github.com 1. 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 이번 리뷰는 Roach가 해주셨다! 감사합니다 로치!! 1. JSessionId 구현 비슷하게 나마 sessionId를 유사하게 구현하려 노력했던 부분이다. save를 하면서 저장을 하고, 반환값으로 session.. CODE SQUAD/FeedBack 정리 2022. 4. 3. [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] 스프링 카페 6단계 - 댓글 (2022/03/17) [Shine] 스프링 카페 6단계 - 댓글 by zbqmgldjfh · Pull Request #120 · codesquad-members-2022/java-spring-cafe 안녕하세요! Shine 입니다! 우선 리뷰를 해주시는 리뷰어 님께 감사의 인사를 전합니다! 이번 과제를 수행하며... 요구사항에서는 게시물에 자신이 쓴 댓글이 있으면 삭제 불가라 되어있지만... 그 github.com 1. 질문 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 질문 1) 댓글을 작성하면 Session은 뷰, 서버 중 어디서 넘겨야 하는가? 이번 댓글 기능 구현 에 사용한 DTO는 다음과 같습니다. public class ReplyDto { private String userId; .. CODE SQUAD/FeedBack 정리 2022. 3. 25. [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. [TIL] 일일 회고 2022/03/14~16 오늘 한 일 1. 피드백 받은점 수정하고 기록하기 [Review] 스프링 카페 4단계 - 로그인 구현 (2022/03/15) [Shine] 스프링 카페 4단계 - 로그인 구현 by zbqmgldjfh · Pull Request #73 · codesquad-members-2022/java-spring-cafe 안녕하세요! Shine 입니다! 우선 리뷰를 해주시는 리뷰어 님께 감사의 인사를 전합니.. blogshine.tistory.com 이번에는 저번에 이어 Roach가 리뷰해 주셨다! 다른 리뷰어들도 열심히 해주시지만, 로치가 내가 잘 몰랐던 부분이나, 사용 이유에 대하여 이유를 자주 물어보셔서 좋다! 2. 영한님 ExceptionHandler 복습하기 예외 처리 방식에 대한 방법이 잘 생각 나지 않아서.. CODE SQUAD/회고록 2022. 3. 16. [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. [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 다음
