3조 코드 리뷰

최세민

인상 깊었던 코드

• 커넥션 풀을 직접 구현하신 부분이 인상적이었습니다.
• xml이나 어노테이션을 사용하지 않았습니다. filter에서 response status를 잡아서 예외처리를 해줬습니다.
• View를 보여주는 로직과 Api 로직을 분리한 것이 좋았습니다.

피드백

• 모델 객체에 createdAt default value로 넣어줬는데 이렇게 하면 테스트가 어려울 수 있다는 피드백을 받았습니다. 전체 필드 생성자가 있어서 상관없긴 하지만 그런 관점에서 생각해보지 못해서 좋았습니다!

고민사항

• URL에 쿼리스트링으로 데이터를 적는 것이 왜 위험한지에 대해서 고민해봤습니다.
  • HTTPS 통신을 한다면 QueryString도 암호화 되기 때문에 문제가 없을 것 같다고 생각했습니다.
  • 실질적인 이유는 URL의 길이 제한, 데이터 포맷 제한, 브라우저 캐싱 등의 문제가 있는 것 같습니다.

박혜성

• 권한 검증을 따로 필터로 빼서 중복 코드를 줄이신 분들이 많아 인상적이었습니다.
• jdbctemplate을 구현하셔서 중복 코드를 줄이셨네요.
• 직접 css, html을 작성하셔서 보기 좋게 만드셨습니다.
• 좋은 테스트를 위해서는 시간을 외부에서 어떻게 받아야 할까요... 코틀린이나 타입 스크립트는 디폴트로 값을 적용할 수 있지만 자바는 없어서 고민됩니다.

조희승

• 서블릿을 rest api 스럽게 url 을 설계하면 "*" 처리때문에 한 서블릿에서 분기해야하는 건 어쩔 수 없는 문제인가 봅니다
• 시간이 들어가는 로직은 파라미터로 빼야 테스트 코드를 짤 수 있다는걸 오랜만에 상기했습니다.
• script template 라는걸 요구사항을 통해 알게 되었습니다.
• transaction 처리를 하신 분이 있어서 락도 포함해 동시성 처리에 대한 고민도 해봐야겠다는 생각을 했습니다.
• connection pool 을 간단히 만든분이 계서서 신기했습니다.

김승수

• filter 에서 blackList 혹은 whiteList로 공통 인증을 하는 것도 좋은 것 같습니다
• testable 한 코드를 구현하는 것도 좋은 것 같습니다.
• jsp 사용할 때 js 와 공통된 문법이 있어 설정하여 사용할 수 있는 것을 알게되었습니다.

김현우

• 커넥션 풀을 구현하신분을 보아서 신기했습니다.