[Feature/#78] 동시에 공유컨테이너로 진입하는 기능 구현 #79
Conversation
jungseokyoung-cloud
requested review from
jungseokyoung-cloud,
around-forest and
LURKS02
and removed request for
jungseokyoung-cloud and
around-forest
| private extension OpenSharedContainerUseCase { | ||
| func bind() { | ||
| repository.receivedEvent | ||
| .sink { [weak self] event in |
There was a problem hiding this comment.
P2: 지혜님이랑 마찬가지로 Core의 'sink(with:onReceive)'사용하시면 편하게 사용하실 수 있습니다!
| // Created by Yune gim on 12/2/24. | ||
| // | ||
|
|
||
| import Foundation |
There was a problem hiding this comment.
P1: 불필요한 파일이 잘못올라 간것 같습니다!
|
@jungseokyoung-cloud |
이따 한번 논의해보시죠 |
Data/Data/EventRepository.swift
Outdated
| public func notice(event: Event) { | ||
| guard let eventData = try? JSONEncoder().encode(event) else { return } | ||
| socketProvider.sendAll(data: eventData) | ||
| } | ||
| } | ||
|
|
||
| private extension NoticedEventRepository { | ||
| func bind() { | ||
| socketProvider.dataShared.sink { [weak self] (data, _) in | ||
| guard let event = try? JSONDecoder().decode(Event.self, from: data) else { return } | ||
| self?.receivedEvent.send(event) | ||
| } | ||
| .store(in: &cancellables) | ||
| } | ||
| } |
There was a problem hiding this comment.
P2: 만약 이렇게 NoticedEventRepository를 따로 사용하게 된다면 JSONEncoder / JSONDecoder는 따로 빼서 재사용 가능하게 하면 어떨까요?
There was a problem hiding this comment.
좋습니다!
그리고 이렇지 않아도 Core에서 JSONEncoder/Decoder를 두고 다른데서 재사용해야 할 필요가 생긴것 같아요.
Data에서 인코딩/디코딩이 많이 일어나기 시작해 보여요
관련 이슈
✅ 완료 및 수정 내역
🛠️ 테스트 방법
📝 리뷰 노트
-이벤트를 발행하고 전달받는 레포지토리 구현했습니다.
📱 스크린샷(선택)