Link 테이블 업데이트 관련 질의

[sangwonsheep]

Link 테이블에 created_at, updated_at 필드 추가하는 것은 어떨까요?

현재 픽 생성 시에 image_url 또는 description이 비어있을 때만 update하도록 되어 있습니다.
또한 비어 있지 않은 경우에는 아예 update를 못하도록 로직을 변경했었습니다.

// null 값이 아닌 필드만 업데이트
// 이미 값이 있으면 업데이트하지 않음.
public Link updateMetadata(String title, String description, String imageUrl) {
    if (StringUtils.isEmpty(this.title)) {
	    this.title = title;
    }
    if (StringUtils.isEmpty(this.description)) {
	    this.description = description;
    }
    if (StringUtils.isEmpty(this.imageUrl)) {
	    this.imageUrl = imageUrl;
    }
    return this;
}

픽 생성 시 해당 링크의 updated_at이 한 달이 넘었을 경우에 update를 하는 것은 어떨까요?
아니면 한 달이 넘은 링크에 대해 스케줄링을 돌려도 좋을 것 같습니다.
아예 update하지 못하도록 막는 로직도 같이 제거하면 좋을 것 같습니다.

[kimminkyeu]

퍼펙트. 아예 #928 작업을 바로 들어갈 수 있게, 오늘 자기전에 updated_at 추가해놓겠슴다.

[kimminkyeu]

픽 생성 시 해당 링크의 updated_at이 한 달이 넘었을 경우에 update를 하는 것은 어떨까요?

@sangwonsheep
이 부분은 updateMetadata() 메서드가 아닌 바깥쪽 (서비스) 에서 처리하도록 수정하는게 어떨까요
왜냐면 위 엔티티 메서드랑 이벤트 리스너 쪽 메서드랑 같은 확인 로직이 중복되는 문제도 있고
업데이트 규칙 자체는 서비스 로직이니까 서비스에 있는게 맞을 듯 싶음!

[sangwonsheep]

@kimminkyeu
엔티티의 필드를 업데이트하는 메서드는 엔티티 내부에 있어야 한다고 생각해요.
조건만 서비스 차원에서 걸고, 조건에 맞으면 엔티티 내부에서 업데이트하는 메서드를 서비스에서 가져다 쓰는건 어떨까요?
(서비스에서 링크 엔티티의 update 메서드를 사용한다.)

[kimminkyeu]

상원님 생각에 동의합니다!
저는 if 문을 통해 업데이트 할지 말지 결정하는 로직은 서비스 쪽에서만 하자는 의견이였는데
이 부분이 엔티티 내부에 있어야 한다고 생각하시는지 궁금함니다!

[sangwonsheep]

if문 관련된 것은 서비스 쪽에서만 하는 것으로 말씀드린 것이었습니다!
엔티티 내부에서는 없어도 될 것 같습니다!
지금처럼 엔티티 내부에 있다보니 헷갈리게 되기도 하고 중복 체크하게 되는 경우도 생기는 것 같습니다