CODE SQUAD/FeedBack 정리25 [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. [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. [Review] 로또 3단계 - 코드 리뷰 반영 (2022/02/27) 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 이번 리뷰는 Dion께서 해주셨다! 1. 일급 컬렉션의 상속 사용? 우선 난 일급 컬렉션에 상속을 사용하면 안된다는 생각은 하였지만, 좀 구체적인 근거가 없는것 같아 팀원과 함께 질문하였다. 저희 팀은 1급 객체로 LottoTicket을 만들었습니다. 여기서 궁금한 점은, WinningNumbers 객체를 만들면서 1급 객체인 LottoTicket을 상속하는 방법이 떠올랐습니다.왜냐하면 기존의 1급 객체 에서의 코드를 재사용 하면 중복을 최소화 할 수 있기 때문이었습니다. 하지만 1급 객체는 독립적인 단위로 사용하는 것이다 보니, 상속을 하여 재사용 한다는 것 자체가 확실하지 않았습니다. 이에 대한 리뷰어님의 의견.. CODE SQUAD/FeedBack 정리 2022. 2. 27. [Review] 로또 3단계 - 수동구매 기능 추가 (2022/02/25) 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 이번주는 호눅스가 리뷰를 해주셨다! 1. 일급 컬렉션의 데이터 출력하기 1급 컬렉션의 데이터는 어디서 출력해야 할까? 이에 대한 의문으로 부터 시작한 질문이였다. 호눅스 께서는 선택의 문제라 알려주셨다. 따라서 우리팀은 1급 객체의 데이터 불변성을 보장하기 위해 필드의 list인 lottoNumbers를 반환하는 getter를 만들지 않기로 하였다. 대신 LottoTicket에게 showLottoNumbers() 메시지를 수신할 수 있도록 만들었다. 해당 Class의 코드는 다음과 같다. public class LottoTicket { private static final int NORMAL_TICKET_SIZE .. CODE SQUAD/FeedBack 정리 2022. 2. 25. [Review] 사다리 구현 5단계 - 실행결과 출력 (2022/02/22) 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 지난번에 이어서, 리뷰는 Dion이 해주셨다!! 감사합니다 Dion!! 1. Scanner의 위치 ? 원래 내가 구현한 코드는 다음과 같았다. public class InputView { public static final String INPUT_PERSON_GUIDANCE_MESSAGE = "참여할 사람 이름을 입력하세요. (이름은 쉼표(,)로 구분하세요)"; public static final String INPUT_LADDER_SIZE_GUIDANCE_MESSAGE = "최대 사다리 높이는 몇 개인가요?"; public static final String INPUT_NAME_LENGTH_ERROR_MESSAG.. CODE SQUAD/FeedBack 정리 2022. 2. 23. 이전 1 2 3 다음
