N+1 관련해서 질문있습니다.

안녕하세요. 우선 좋은 강의 제작해주신 토비님께 항상 감사하고 있어요. 이제 배운지 1년된 왕초보입니당..

혼자 배워보면서 개인 프로젝트를 만들고 있는데 JPA를 사용하고 있어요.

제가 궁금한 것이... N+1 관련한 문제입니다. 아 일단 프로젝트 주제는 복식부기 가계부에요.

@Entity
...
public class Journal extends BaseEntity {
  @ManyToOne(fetch = FetchType.LAZY)
  @JoinColumn(name = "ledger_id", nullable = false, updatable = false)
  @OnDelete(action = OnDeleteAction.CASCADE)
  private Ledger ledger;

  ...
  @OneToMany(mappedBy = "journal", fetch = FetchType.LAZY, cascade = CascadeType.ALL,
  orphanRemoval = true)
  private List<EntryLine> entries = new ArrayList<>(2);

  ...
  public EntryLine getEntryLine(EntrySide side) {
    switch (side) {
      case CREDIT : this.entries.stream().filter(line -> line.isCredit()).findFirst()
                    .orElseThrow(...);
      case DEBIT  : this.entries.stream().filter(line -> line.isDebit()).findFirst()
                    .orElseThrow(...);
      default : throw new ...
    }
  }

  ...
  // Service에서 저장되기 전에 호출
  public void validateSavable() {
    ...
    validateJournalSave();
  }
  
  private void validateJournalSave() {
    AccountType debit = getEntryLine(EntrySide.DEBIT).getAccountType();
    AccountType credit = getEntryLine(EntrySide.CREDIT).getAccountType();

    if(!this.transactionType.isValidPlacement(debit, credit)) {
      throw new ...
    }
  }
}

Journal Class에서 EntryLine List에 접근하고 있어요.

그리고 EntryLine Class는 이렇게 생겼어요.

@Entity
...
public class EntryLine extends BaseEntity {
  @ManyToOne(fetch = FetchType.LAZY)
  @JoinColumn(name = "journal_id", nullable = false, updatable = false)
  private Journal journal;

  @ManyToOne(fetch = FetchType.LAZY)
  @JoinColumn(name = "account_id", nullable = false)
  private Account account;

  ...
  // private package 접근제어자 사용
  // Account는 Category를 참조중이에요.
  AccountType getAccountType() {
    return this.account.getCategory().getAccountType();
  }
}

거래가 저장되기 전에 Journal : validateJournalSave() 에서 this.transactionType에 따라 차변과 대변에 올바르게 위치하고 있는지 검사한 후 저장하고 있는데 이것을 생성과 수정할 때 두 곳에서 사용하고 있어요.

Ledger에 5개 Category가 있고, Account는 그 Category를 참조하고 Category에서만 AccountType이 있어요.

Journal이 각 EntryLine의 AccountType을 얻기 위해

Journal -> EntryLine -> Account -> Category -> getAccountType() 이렇게 흘러가네요.

이렇게 접근해도 설계상 괜찮은걸까요?

Journal을 저장할때는 @Query 사용해서 Fetch Join으로 필요한 Account를 가져오고 있는 상황이에요.

Journal이라는 엔티티가 비즈니스 로직 수행을 위해서 다른 엔티티의 필드까지 깊게 참조?? 가져오도록 설계하는게 옳은건지 모르겠어요.