[논의] Store의 startTime과 endTime의 로직에 대한 논의

[Hyeon-Uk]

배경

현재 Store의 도메인 엔티티 기준은 아래와 같습니다.

package camp.woowak.lab.store.domain;

import java.time.LocalDateTime;
import java.util.UUID;

import camp.woowak.lab.store.exception.NotEqualsOwnerException;
import camp.woowak.lab.vendor.domain.Vendor;
import jakarta.persistence.Column;
import jakarta.persistence.Embedded;
import jakarta.persistence.Entity;
import jakarta.persistence.FetchType;
import jakarta.persistence.GeneratedValue;
import jakarta.persistence.GenerationType;
import jakarta.persistence.Id;
import jakarta.persistence.JoinColumn;
import jakarta.persistence.ManyToOne;
import lombok.AccessLevel;
import lombok.Getter;
import lombok.NoArgsConstructor;

@Entity
@NoArgsConstructor(access = AccessLevel.PROTECTED)
@Getter
public class Store {

	@Id
	@GeneratedValue(strategy = GenerationType.IDENTITY)
	private Long id;

	@ManyToOne(fetch = FetchType.LAZY)
	@JoinColumn(name = "vendor_id", nullable = false)
	private Vendor owner;

	@ManyToOne
	@JoinColumn(name = "store_category_id", nullable = false)
	private StoreCategory storeCategory;

	@Column(nullable = false)
	private String name;

	@Embedded
	private StoreAddress storeAddress;

	@Column(nullable = false)
	private String phoneNumber;

	@Column(nullable = false)
	private Integer minOrderPrice;

	@Embedded
	private StoreTime storeTime;

	public Store(Vendor owner, StoreCategory storeCategory, String name, String address, String phoneNumber,
				 Integer minOrderPrice, LocalDateTime startTime, LocalDateTime endTime
	) {
		StoreValidator.validate(owner, storeCategory, name, address, minOrderPrice, startTime, endTime);
		this.owner = owner;
		this.storeCategory = storeCategory;
		this.name = name;
		this.storeAddress = new StoreAddress(address);
		this.phoneNumber = phoneNumber;
		this.minOrderPrice = minOrderPrice;
		this.storeTime = new StoreTime(startTime, endTime);
	}

	public void validateOwner(Vendor owner) {
		if (!this.owner.equals(owner)) {
			throw new NotEqualsOwnerException("가게의 점주가 일치하지 않습니다." + this.owner + ", " + owner);
		}
	}

	public boolean isOwnedBy(UUID ownerId) {
		return owner.getId().equals(ownerId);
	}

	public boolean isOpen() {
		LocalDateTime openTime = storeTime.getStartTime();
		LocalDateTime closeTime = storeTime.getEndTime();

		LocalDateTime now = LocalDateTime.now();

		return (now.isEqual(openTime) || now.isAfter(openTime)) && now.isBefore(closeTime);
	}
}

startTime과 endTime에 Day와 Year도 포함되어 아래와 같은 문제가 생길 수 있습니다.

• startTime = 2024.08.17 08:50:00 / endTime = 2024.08.18 01:00:00 라고 했을 경우
• 2024.08.18 09:00:00 에 isOpen을 호출했을 때 결과값은 true가 나와야하나, 실제로는 false가 나옵니다.

이를 해결하기 위해 startTime과 endTime을 응집해둔 StoreTime 객체 내부에 열리는 시간의 시,분과 닫히는 시간의 시,분 만을 저장한 뒤, 해당 정보를 가지고 isOpen의 로직을 변경해야 할 것 같습니다!

세부 주제

1.
2.
3.