Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌 Summary
🤔 고민한 점 및 질문
session.getSessionTypeName()처럼 위임 메서드를 통해 접근하도록 했습니다. AI에게 물어보니 Repository는 객체를 테이블로 매핑하는 역할을 수행하므로, 인프라 계층 내에서는 객체 그래프를 탐색하는 것이 허용될 수 있다는 답변을 받았습니다. 하지만 아직 거기까지 이해하지를 못해서... 여쭤보고 싶습니다.EnrollmentRepository.findBySessionId()의 반환 타입을 일급 컬렉션이 아닌List<Enrollment>로 구현했습니다. AI에게 물어보니 리포지토리는 순수한 자료구조만 넘기고 비즈니스 로직이 포함된 일급 컬렉션은 서비스 계층에서 만드는 게 역할 분리에 맞다는 답변을 받았는데요... 여기도 아직 이해를 못해서 우선 AI 답변에 따라 구현을 진행했는데 적절한지 궁금합니다.Session테이블에 필드로 포함시키지 않고CoverImage를 별도 테이블로 분리했습니다. 이미지 자체의 독립적인 검증 로직이 존재하고, 향후 확장성을 고려했을 때 별도 엔티티로 관리하는 것이 더 맞다고 판단했는데 올바른 접근이 맞을까요?Enrollment클래스를 새로 만들었습니다. AI에게 물어보니 DB 테이블과 매핑을 깔끔하게 하려면 중간 지점 역할을 하는 클래스가 필요하다고 해서 해당 클래스를 추가했습니다. 사실 제가 이런 구조적인 분리를 왜 해야 하는지 완벽히 이해하지 못한 상태라... 적절한지 궁금합니다.