pgmjun / jump-rope-checker Goto Github PK

View Code? Open in Web Editor NEW

0.0 1.0 0.0 339 KB

한국줄넘기협회 - 줄넘기 대회용 경기 기록 시스템

Java 99.31% Dockerfile 0.30% Shell 0.38%

jump-rope-checker's Introduction

jump-rope-checker

줄넘기 경기 기록 시스템

jump-rope-checker's People

Contributors

Watchers

jump-rope-checker's Issues

코드 리팩터링 [중요🔥]

실제로 사용되는 서비스를 단기간 안에 만들어야했기 때문에 클린코드, 객체지향을 거의 신경쓰지 못하며 개발
때문에 코드의 리팩터링이 시급

리팩터링 규칙

한 메서드에 최소한의 들여쓰기(indent)만 허용했는가?

최대 depth : 2 까지만 허용
else 예약어를 쓰지 않았는가?
모든 원시값과 문자열을 포장했는가?
콜렉션에 대해 일급 콜렉션을 적용했는가?
3개 이상의 인스턴스 변수를 가진 클래스를 구현하지 않았는가?
getter/setter 없이 구현했는가?
핵심 로직을 구현하는 도메인 객체에 getter/setter를 쓰지 않고 구현했는가? 단, DTO는 허용한다.
코드 한 줄에 점(.)을 하나만 허용했는가?
메소드의 인자 수를 제한했는가? 4개 이상의 인자는 허용하지 않는다. 3개도 가능하면 줄이기 위해 노력해 본다.
디미터(Demeter)의 법칙(“친구하고만 대화하라”)을 지켰는가?예를 들어 location.current.representation.substring(0, 1)와 같이 여러 개의 점(.)이 등장하면 리팩토링할 부분을 찾아본다.
메소드가 한가지 일만 담당하도록 구현했는가?
클래스를 작게 유지하기 위해 노력했는가?

메서드당 line을 10까지만 허용

길이가 길어지면 메서드로 분리
매직 리터럴 / 매직 넘버 사용을 자제하고 상수 사용

선수 CRUD 구현

선수 등록
- 일반 등록 ✔
- CSV파일로 등록 (진행중)
조회 기능 ✔
- 선수 id로 조회 ✔
- 선수명으로 조회 ✔
- 단체ID로 선수 조회 ✔
선수 정보 수정 기능 ✔
선수 개인정보 변경 기능 ✔
선수 삭제 기능 ✔
- 선수 id로 삭제 ✔

Swagger3 적용

공식 문서
https://springdoc.org/v2/#Introduction

기본 URL
http://localhost:8080/swagger-ui/index.html#/

[에러 메시지]
Access to XMLHttpRequest at 'http://192.168.0.60:8080/competition/add' from origin 'http://localhost:3000' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource.