20241128 시놀 인턴 4일차 코드리뷰내용

2024. 11. 29. 10:46TIL

1. 트랜잭션은 기존에 있던 코드들과 비슷하게 하느게 좋음
- 내 코드에선 서비스딴에서 트랜잭션을 걸었지만 기존코드들을 좀더 자세히 살펴보고 따라가는게 좋음
 기존 코드에선 컨트롤러쪽에서 트랜잭션을 걸음.
2. 기존에 만들어둔 디폴트 리포지토리를 조금더 이용하는게 유지보수면에서 좋음
- 내 코드는 기껏 디폴트리포지토리를 상속받았지만 정작 사용은 제대로 안한면이 있음
3. 중요!! MbtiClass에서 쿼리문을 그대로 보여주는 부분이있는데 이런건 보안측면에서 상당히 위험함
- 이부분은 쿼리 인젝션 관련해서 조금더 찾아볼것!
4. 시간이 난다면 통계쿼리쪽도 알아보고 공부할 것, 성능을 위하려면 최대한 한쿼리안에 반복문 없이 작성 해볼것.