PayAccount와 AccountHistory의 연관관계에 대한 문제

[Hyeon-Uk]

Link

이슈 #3 에서 파생된 문제입니다.

문제 정의

• PayAccount와 AccountHistory는 1 : N 관계이다. 연관관계의 주인은 AccountHistory이다.
• PayAccount의 입/출금 기능을 호출한 뒤, 성공하면 AccountHistory를 반드시 기록해야한다.
• 따라서 단방향 매핑관계에서는 도메인 자체에서 History를 기록할 수 있는 수단이 존재하지 않다고 생각하며, 타 서비스에서 입/출금에 대한 히스토리를 남기기 위해서는 WithdrawService 혹은 DepositService를 호출해서 사용해야한다.

@Service
@Slf4j
public class PayAccountDepositService {
	private final PayAccountRepository payAccountRepository;
	private final PayAccountHistoryRepository payAccountHistoryRepository;

	public PayAccountDepositService(PayAccountRepository payAccountRepository,
									PayAccountHistoryRepository payAccountHistoryRepository) {
		this.payAccountRepository = payAccountRepository;
		this.payAccountHistoryRepository = payAccountHistoryRepository;
	}

	@Transactional
	public long depositAccount(AccountTransactionCommand command) {
		PayAccount payAccount = payAccountRepository.findByIdForUpdate(command.payAccountId())
			.orElseThrow(() -> new NotFoundAccountException("Invalid account id with " + command.payAccountId()));
                //history를 기록하는 로직을 필수적으로 작성해야함.
		PayAccountHistory depositHistory = payAccount.deposit(command.amount());
		log.info("A deposit of {} has been completed into Account ID {}.", command.amount(), command.payAccountId());
		payAccountHistoryRepository.save(depositHistory);

		return payAccount.getBalance();
	}
}

• 하지만 history에 대한 관심사는 PayAccount에 강하게 결합되어있어야 한다고 생각합니다.
• 또한, 페이 포인트를 충전하기 위해서는 일일 한도는 100만원이다 라는 요구사항은 PayAccount에서 확인하는 책임까지 있다고 생각합니다.
• 이를 위해 양방향 매핑을 통해 AccountHistory를 PayAccount에서 쉽게 관리하고자 하는데 찬성, 반대, 더 좋은 방향 등 많은 의견 부탁드립니다.

[Dr-KoKo]

저는 PayAccount를 연관관계의 주인으로 두는 것이 바람직하다고 생각합니다. AccountHistory는 로그성 데이터이며, 생명주기(생성/삭제)가 전적으로 PayAccount에 의존하기 때문입니다.

1. AccountHistory의 생성은 (삭제 여부 등을 확인하기 위해) PayAccount 조회가 선행되어야 합니다.
2. AccountHistory는 로그성 데이터이므로 이에 대한 단건조회를 할 일이 없을 것 같습니다.

[Hyeon-Uk]

로그성 데이터지만, AccountHistory의 테이블에 PayAccount의 ID값이 외래키로 걸려있는 구조입니다.
연관관계의 주인은 외래키를 쥐고있는 AccountHistory가 되어야한다고 생각합니다.

말씀해주신대로 로그성 데이터이며, 생명주기가 PayAccount에 의존하지만, 외래키로 인해 연관관계의 주인이 PayAccountHistory가 되어버렸기 때문에, PayAccount -> PayAccountHistory 로 향하는 단방향 매핑을 추가하여 쉽게 관리하고자 한다는 의견이였습니다!

[Hyeon-Uk]

결론 : 현재는 PayAccountHistory -> PayAccount 방향의 단방향 매핑을 추가해서 PayAccount에서 아래와 같이 History 로그를 쉽게 관리하도록함.

@Entity
@Slf4j
public class PayAccount {
	@Id
	@GeneratedValue(strategy = GenerationType.IDENTITY)
	private Long id;

	@Column(name = "balance", nullable = false)
	@ColumnDefault("0")
	private long balance;

	@OneToMany(mappedBy = "payAccount", fetch = FetchType.LAZY, cascade = CascadeType.PERSIST)
	private List<PayAccountHistory> history = new ArrayList<>();

	public PayAccount() {
		this.balance = 0;
	}

	public Long getId() {
		return id;
	}

	public long getBalance() {
		return this.balance;
	}

        //domain method 실행과 함께 History를 저장할 수 있습니다.
	public PayAccountHistory charge(long amount) {
		validateTransactionAmount(amount);
		validateDailyChargeLimit(amount);
		this.balance += amount;

		return issueAndSavePayAccountHistory(amount, AccountTransactionType.CHARGE);
	}

	private PayAccountHistory issueAndSavePayAccountHistory(long amount, AccountTransactionType type) {
		PayAccountHistory payAccountHistory = new PayAccountHistory(this, amount, type);
		this.history.add(payAccountHistory);

		return payAccountHistory;
	}
	...
}