20241128 시놀 인턴 4일차 코드리뷰내용
2024. 11. 29. 10:46ㆍTIL
1. 트랜잭션은 기존에 있던 코드들과 비슷하게 하느게 좋음
- 내 코드에선 서비스딴에서 트랜잭션을 걸었지만 기존코드들을 좀더 자세히 살펴보고 따라가는게 좋음
기존 코드에선 컨트롤러쪽에서 트랜잭션을 걸음.
2. 기존에 만들어둔 디폴트 리포지토리를 조금더 이용하는게 유지보수면에서 좋음
- 내 코드는 기껏 디폴트리포지토리를 상속받았지만 정작 사용은 제대로 안한면이 있음
3. 중요!! MbtiClass에서 쿼리문을 그대로 보여주는 부분이있는데 이런건 보안측면에서 상당히 위험함
- 이부분은 쿼리 인젝션 관련해서 조금더 찾아볼것!
4. 시간이 난다면 통계쿼리쪽도 알아보고 공부할 것, 성능을 위하려면 최대한 한쿼리안에 반복문 없이 작성 해볼것.
'TIL' 카테고리의 다른 글
20241220 시놀인턴 마지막날 (1) | 2024.12.20 |
---|---|
20241204 시놀인턴 8일차 오류수정 (0) | 2024.12.04 |
20241127 시놀 인턴 3일차 (0) | 2024.11.27 |
20241125 시놀 인턴 1일차 (1) | 2024.11.25 |
20240801 (목) 한달 인턴 과제 및 온보딩 진행 - 시큐리티 관련 오류 해결 (0) | 2024.08.01 |