Skip to content

fix: 우물 아이템 순서 변경 로직 개선 및 고정 순서 중복 방지#48

Merged
lilloo04 merged 1 commit intodevfrom
fix/#45
Jul 29, 2025
Merged

fix: 우물 아이템 순서 변경 로직 개선 및 고정 순서 중복 방지#48
lilloo04 merged 1 commit intodevfrom
fix/#45

Conversation

@lilloo04
Copy link
Copy Markdown
Member

연관 이슈

내용

기존 방식의 문제점

  • 고정 대상만 갱신하고 나머지 아이템은 별도 처리 없이 그대로 유지
    → 순서 충돌이나 중복, 비정상 재배치 발생 가능

  • order를 직접 지정한 일부 아이템만 처리하고
    나머지 아이템은 전체 맥락 없이 빠짐없이 채워지지 않음

  • fixedId를 기존 아이템과 비교할 때 item.id가 아닌 item.item_id로 비교 안 하는 경우 있음
    → 아이템을 찾지 못해 실패하거나 잘못된 매핑이 발생

개선 방법

  • 전체 아이템을 기준으로 고정할 아이템을 우선 배치,
    나머지는 기존 순서를 유지한 채 빈 자리에 자동 채움

  • 고정된 order는 중복 없이 지정되도록 검증하고,
    전체 아이템 수 기준으로 배열 길이를 정확히 유지

  • 기존 아이템 리스트에서 ID 매칭은 item.item_id로 명확히 비교

@lilloo04 lilloo04 requested a review from wooodypark July 29, 2025 09:49
@lilloo04 lilloo04 self-assigned this Jul 29, 2025
@lilloo04 lilloo04 merged commit f576e8a into dev Jul 29, 2025
1 check passed
@lilloo04 lilloo04 deleted the fix/#45 branch July 30, 2025 04:26
@lilloo04 lilloo04 mentioned this pull request Aug 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants