멘토님 말씀패키지 구조, 아키텍처 모두 리팩토링하세요.YAML 설정파일 빼는거CORS 도 LIST 되어야합니다.시크릿 외부화 해야합니다.HttpMethod 별로 Pattern도 다르게 해야합니다.Criteria 적용하세요 Querydsl 잘 사용하면 좋지만 표준 스펙부터 잘 다룰 줄 알아야합니다.기술문서도 다 채워넣으십쇼 매번 동기적으로 물어보면 리소스 낭비입니다.커서방식 페이지네이션 꼭 적용하세요 백엔드 기본 소양입니다.Swagger 꼭 써야합니다. Restdocs는 테스트 코드가 필수인데 테스트 코드 못 짜는 경우도 있습니다DB Lock, 동시성, 동적쿼리 적용안되어있습니다.
멘토님 말씀
지금 하고 있는건 노가다성 작업이고 의미가 없어보입니다.
정리되지 않으면 디테일이 보이지않습니다!
기능은 스탑입니다 많은 경험 제대로된 경험이 필요합니다.
소나 큐브, Jacoco는 지금 하지마십쇼
패키지 구조, 아키텍처 모두 리팩토링하세요.
- 클래스가 너무 뚱뚱해졌습니다. 정리해야합니다.
YAML 설정파일 빼는거
CORS 도 LIST 되어야합니다.
시크릿 외부화 해야합니다.
- db 계정/ 패스워드
HttpMethod 별로 Pattern도 다르게 해야합니다.
- JwtFilter도 메소드 별로 거쳐야 되는 것도 아닌 것도 있다.
- 예를 들어 개인 페이지 같은 경우 get, post 는 안될 수 도 있는
- 메소드 패턴 별로 세분화 하세요.
Criteria 적용하세요 Querydsl 잘 사용하면 좋지만 표준 스펙부터 잘 다룰 줄 알아야합니다.
- JPA Crieteria 사용하세요.
package javax.persistence.criteria; import java.math.BigDecimal; import java.math.BigInteger; import java.util.Collection; import java.util.Map; import java.util.Set; import javax.persistence.Tuple; /** * Used to construct criteria queries, compound selections, * expressions, predicates, orderings. * * <p> Note that <code>Predicate</code> is used instead of <code>Expression<Boolean></code> * in this API in order to work around the fact that Java * generics are not compatible with varags. * * @since 2.0 */ public interface CriteriaBuilder { ... }
기술문서도 다 채워넣으십쇼 매번 동기적으로 물어보면 리소스 낭비입니다.
커서방식 페이지네이션 꼭 적용하세요 백엔드 기본 소양입니다.
- 커서 꼭‼️