과제 피드백

[Eisring]

2번 과제 도 정말 수고 많으셨습니다 -!

유닛 테스트와 리드미가 없었다, 라는 점을 빼곤 사실 아쉬운 점이 딱히 보이지 않았습니다.

몇 가지 개선을 해 볼만한 사항들을 제안드리자면,

1. 이벤트 처리 로직을 엔티티 내부로 이동 시켜 응집도 향상을 노려볼 수 있을 것 같습니다.

• ProductService 의 부분인데요**, updateStockStatus 메서드에서 StockStatus.OUT_OF_STOCK 에 대한 이벤트를 발행**하는 것으로 보입니다. 이 부분의 이벤트 발행에 관해서 Product Entity 내부에서 상태 변경과 함께 처리되면 더 좋았을 것 같습니다.

2. 페이징 방식에 대한 개선이 좀 더 있었으면 싶습니다.

• 현재 페이징 처리에 관해서 PageRequest 와 Cursor 두 가지 모두를 받아오지만, 실제 쿼리를 보니 pageable을 사용하지 않으시는 것으로 보입니다. 레포지토리를 보기전에는 pageable 과 cursor 가 혼용되어 난해해 보이고, 막상 레포지토리를 열어보니 pageable 을 사용하지 않으셨는데, 사용하지 않는 부분은 코드에서 삭제하면 더 좋았을 것 같습니다.

3. 트랜잭션 전파 레벨을 명확하게 했으면 더 좋았을 것 같습니다

• 현재 @transactional 만 사용하고 계신데, 필요에 따라 @transactional(readOnly = true)와 @transactional의 차이를 적절히 나누어 설정했다면 더 좋았을 것 같습니다 -!

4. 로깅 메시지가 에러 사항에서만 들어나는데, 성공적으로 동작한 경우에 대한 로그도 추가로 작업하셨으면 더 나아질 것 같습니다.

• 현재 로그에서 log.error만 사용되는데, 성공적인 동작의 경우 log.info나 log.debug를 추가로 사용하여 서비스 동작을 보여줬다면 더 좋았을 것 같습니다 -!

과제 하시느라 고생 정말 많으셨고, 프로젝트 주차도 파이팅 하시길 기원하겠습니다 -!