[REQ] Wish 중복 저장 해결

[Dr-KoKo]

기능

Wish의 중복 저장을 방지한다.

설명

TripVote-BE/app/src/main/java/fc/be/app/domain/wish/service/WishService.java

Line 26 in 2769d16

public WishAddResponse addWish(WishAddRequest wishAddRequest) {

Wish를 저장할 떄 DB에 동일한 Wish가 있는지 확인하는 로직이 부재인 상태입니다. 이 때 두 번 이상의 wish/add 요청이 오게되면 wish에는 동일한 member, place 에 대해 두 개 이상의 wish가 저장되게 됩니다. 그렇게 되면 향후 getWishes 메서드를 호출할 때

TripVote-BE/app/src/main/java/fc/be/app/domain/wish/service/WishService.java

Line 40 in 2769d16

public WishGetResponse getWishes(Long memberId, Pageable pageable) {

중복된 Wish들이 섞여서 반환되게 됩니다. 따라서 add 할 때 중복을 방지하기 위해 현재 유저가 요청 받은 place에 찜한 이력이 있는지 확인하는 방어 로직을 추가하면 좋을 것 같습니다.

기대 효과

Wish가 중복 저장되는 오류를 방지할 수 있다.

[laigasus]

@Dr-KoKo 피드백 주셔서 감사합니다 🙇

autoincrement의 기본키를 없애고, 복합키를 이용하여 선언했기 때문에
한 회원에 대해 중복된 장소가 저장되진 않습니다.

이미 존재하는 값에 대해서는 update로 자동 수행(1회 쿼리 수행)합니다.
존재하는지 확인하고 추가하는 수행(2회 쿼리 수행) 보다 db 접근 횟수가 적다고 생각했기 때문입니다.

현 시점에서의 기능 결함이 있을까요?

TripVote-BE/app/src/main/java/fc/be/app/domain/wish/entity/WishId.java

Lines 8 to 17 in bebcc8d

	@NoArgsConstructor
	@EqualsAndHashCode(onlyExplicitlyIncluded = true)
	public class WishId implements Serializable {
	@EqualsAndHashCode.Include
	private Long memberId;
	@EqualsAndHashCode.Include
	private Integer place;
	}

TripVote-BE/app/src/main/java/fc/be/app/domain/wish/entity/Wish.java

Lines 12 to 28 in bebcc8d

	@IdClass(WishId.class)
	@SuperBuilder
	@Entity
	@NoArgsConstructor(access = AccessLevel.PROTECTED)
	@Getter
	@Comment("찜")
	public class Wish extends BaseTimeEntity {
	@Id
	@Comment("회원 id(FK)")
	private Long memberId;
	@Id
	@ManyToOne
	@JoinColumn(name = "place_id")
	@Comment("장소 id(FK)")
	private Place place;
	}

마구 insert 실행해도 물리적 데이터는 그대로인 모습입니다.

[Dr-KoKo]

네넵 확인했습니다. 복합키로 되어있는 부분을 확인 못했네요. 확인해주셔서 감사합니다.