[FEAT/#115] 스티커 삭제 구현

[0Hooni]

🤔 배경

• 스티커 삭제 동작이 필요했습니다.
• 삭제 동작이 동시 편집 상황에 맞게 적용되어야 했습니다.

📃 작업 내역

• 스티커 삭제 UI를 구현했습니다.
• 스티커 삭제 이벤트 반영을 구현했습니다.

추가 작업

• EditPhotoRoomViewController가 하위 View의 인스턴스를 지속적으로 접근하던 환경을 개선하였습니다.
• ViewModel에 접근하여 메서드 호출하는 상황을 In/Out 패턴에 맞게 개선하였습니다.

✅ 리뷰 노트

책임 분리 및 리팩토링

• EditPhotoRoomViewController에서 CanvasScrollView를 직접 참조하는 상황이 많았습니다.
• 이러한 참조를 이용해서 기능하는 메서드들이 ViewController에 많은 상황이었습니다.
• 이러한 책임을 분리하기 위해 CanvasScrollView에 메서드를 요청하는 방식으로 리팩터링을 하였습니다.
• 해당 과정에서 Delegate가 추가되었습니다

protocol CanvasScrollViewDelegate: AnyObject {
    func canvasScrollView(_ canvasScrollView: CanvasScrollView, didAdd sticker: StickerEntity)
}

삭제 로직 관련

• 기존에 CanvasScrollView에서 특정 StickerView를 찾기 위해
• 이를 위해 ViewController에서 [UUID: Index] 형태의 인스턴스를 별도로 관리했었습니다.

private var stickerViewDictonary: [UUID: StickerView]

위와 같은 변경을 한 이유는 삭제시 Index기반으로 SubView 참조 하는 경우 삭제시 기존의 Index와 일치하지 않거나 Out of range와 같은 문제가 발생합니다.

이러한 상황을 개선하기 위해 CanvasScrollView에서 StickerViewDictonary에 ID를 이용해 동일하게 O(1)로 SubView에서 StickerView를 찾아낼 수 있도록 하였습니다.

어차피 해당 StickerView는 메모리에 존재하는 동안 같은 주소를 참조하고 있기에 추가적인 리소스가 들지 않는다고 생각했습니다.

현재 StickerView 삭제를 위해서는 아래와 같은 방식으로 작동합니다.

• SoT인 StickerList를 파라미터로 받는다.
• LocalStickerList의 ID만 담은 StickerIdList를 생성해줍니다.
• 로컬과 SoT를 비교하여 SoT에 없는 로컬 항목을 찾습니다
• 가서 반갈죽 해주고 옵니다 👍

private func prepareForApply(_ stickerList: [StickerEntity]) {
    let stickerIdList = stickerViewDictonary.keys.map { $0 }
    
    Set(stickerIdList)
        .subtracting(stickerList.map { $0.id })
        .forEach { deleteStickerView(with: $0) }
}

🎨 스크린샷

실행 결과

🚀 테스트 방법

• EditPhotoRoomFeatureDemo앱을 두개의 시뮬레이터에서 실행합니다
• 한쪽에서 Offer를 해줍니다
• 두 기기를 Guset와 Host로 나누어 삭제 상황을 테스팅 하시면 됩니다 👍

[youn9k]

LGTM 함수 분리가 잘되어있네요!!

[Kiyoung-Kim-57]

함수가 역할 별로 분리가 잘 되어 있어요! 그리고 성능을 위해 Set을 사용해주신 점 인상 깊었습니다!