[Feature] 근거리 통신 모듈에 URL을 이용한 통신 메서드 추가 #103

taipaise · 2024-11-20T10:00:58Z

🌁 Background

단순 data만 receive 했을 때 어떤 entity로 디코딩 해야하는지 알 수 없는 문제가 발생했습니다.
MCSession의 send 메서드에 local File의 URL과, 이름을 지정해서 보낼 수 있는 메서드가 있어 이걸 활용해 전송하기로 했습니다.

👩‍💻 Contents

근거리 통신 모듈의 delegate가 너무 많은 역할을 수행하고 있다 느껴졌습니다. 해당 모듈을 사용하는 repository에서 사용하지 않는 인터페이스가 발생할 가능성이 있다고 판단하여 delegate 프로토콜을 두 개로 나누었습니다. 링크
@ekrud99 @choijungp 생참 팀에서 현재 사용하고 있는 WhiteboardObject에도 영향이 가는 작업이기 때문에 확인해주시면 감사하겠습니다. 적절한 delegate 명이 있다면 수정하도록 하겠습니다.
근거리 통신 모듈에 URL을 이용한 송신, 수신 메서드를 구현했습니다.

✅ Testing

Framework의 동작이기 때문에 따로 테스트하지는 않았습니다.

📝 Review Note

에러 처리에 대한 정책이 필요하다고 생각합니다. 다만 아직은 구현이 더 급하기 때문에 조금 후순위로 미뤄도 괜찮을 것 같습니다.
MCSession의 sendResource 메서드는 완료 핸들러를 통해 비동기 이벤트를 처리하는데요, 이를 Swift Concurrency로 래핑하였습니다.
sendResource를 통해 데이터를 전송하는 방식은 일반 send와는 다르게 peer에게 1:1로 데이터를 송신합니다. 따라서 반복문으로 연결된 피어들에게 데이터를 전송해줘야 했습니다. 좀 더 효율적으로 전송하기 위해 TaskGroup을 이용하여 데이터를 병렬 전송하는 방법을 선택했습니다.

📣 Related Issue

📬 Reference

eemdeeks

고생하셨습니다!!

swift concurrency가 알고있는 것보다 공부할 것이 더 많군요..

eemdeeks · 2024-11-20T12:44:11Z

DataSource/DataSource/Sources/Interface/NearbyNetworkInterface.swift

+    func nearbyNetwork(
+        _ sender: NearbyNetworkInterface,
+        didReceiveURL URL: URL,
+        info: DataInformationDTO)


style: 닫히는 괄호가 밑으로 내려와야 하는 것 같습니다!

요 부분은 저희 같이 이야기 나눈대로 현상유지하도록 하겠습니다 :)

eemdeeks · 2024-11-20T13:00:33Z

NearbyNetwork/NearbyNetwork/Sources/Common/Extension/MCSession+.swift

+                toPeer: peer
+            ) { error in


style: 소괄호 올라가 있어야 할 것 같습니다ㅜㅜ

새로 생성 해 준것이라고 착각했었네요...

바로 수정하겠습니다!!!!!

e626d5c 반영했습니다!!

eemdeeks · 2024-11-20T13:04:02Z

NearbyNetwork/NearbyNetwork/Sources/Common/Extension/MCSession+.swift

+    ) async throws {
+        typealias Continuation = CheckedContinuation<Void, Error>
+
+        try await withCheckedThrowingContinuation { (continuation: Continuation) in


이것을 통해 swift Concurrency처럼 사용할 수 있게 해주는 군요..!

이부분은 공부를 해봐야할 것 같습니다..!ㅜㅜ

꺆 !!!!!!!!!!!!!! 쏘 어렵 ... 내일 설명해주기 .......................................

eemdeeks · 2024-11-20T13:05:27Z

NearbyNetwork/NearbyNetwork/Sources/NearbyNetworkService.swift

+        await withTaskGroup(of: Void.self) { taskGroup in
+            session.connectedPeers.forEach { peer in
+                taskGroup.addTask {
+                    do {
+                        try await self.session.sendResource(
+                            at: fileURL,
+                            withName: infoJsonString,
+                            toPeer: peer)
+                    } catch {
+                        self.logger.log(level: .error, "\(peer)에게 file 데이터 전송 실패")
+                    }
+                }
+            }
+        }


이부분도 공부해보겠습니다!

다시봐도 어렵네유

choijungp

어푸푸 어푸푸푸 어푸푸 어푸푸푸푸 굉장히 어려운 부분 .....................................
Concurrency 공부할 기회를 주셔서 감사 ..... 😇👏🏻

choijungp · 2024-11-20T14:01:32Z

DataSource/DataSource/Sources/Interface/NearbyNetworkInterface.swift

@@ -8,7 +8,7 @@
 import Foundation

 public protocol NearbyNetworkInterface {
-    var delegate: NearbyNetworkDelegate? { get set }
+    var connectionDelegate: NearbyNetworkConnectionDelegate? { get set }


그러니까 근거리통신 연결 로직을 NearbyNetworkConnectionDelegate가,
근거리통신 송수신 로직을 NearbyNetworkReceiptDelegate가 담당하게 되는게 맞나요 ??

똑똑한 조이 맞습니다!!

choijungp · 2024-11-20T14:02:19Z

DataSource/DataSource/Sources/Model/DataInformationDTO.swift

+import Foundation
+
+public struct DataInformationDTO: Codable {
+    let identifier: UUID


단순 궁금증인데 다른 Entity들은 id로 되어 있는데 이 친구는 왜 identifier로 췤 하는지 ?? 요

id 로 수정하겠습니다

e626d5c 반영했습니다!!

choijungp · 2024-11-20T14:05:10Z

NearbyNetwork/NearbyNetwork/Sources/Common/Extension/MCSession+.swift

+    ) async throws {
+        typealias Continuation = CheckedContinuation<Void, Error>
+
+        try await withCheckedThrowingContinuation { (continuation: Continuation) in


꺆 !!!!!!!!!!!!!! 쏘 어렵 ... 내일 설명해주기 .......................................

ekrud99

고생하셨씁니다!

ekrud99 · 2024-11-22T14:59:55Z

NearbyNetwork/NearbyNetwork/Sources/NearbyNetworkService.swift

+        await withTaskGroup(of: Void.self) { taskGroup in
+            session.connectedPeers.forEach { peer in
+                taskGroup.addTask {
+                    do {
+                        try await self.session.sendResource(
+                            at: fileURL,
+                            withName: infoJsonString,
+                            toPeer: peer)
+                    } catch {
+                        self.logger.log(level: .error, "\(peer)에게 file 데이터 전송 실패")
+                    }
+                }
+            }
+        }


다시봐도 어렵네유

ekrud99 · 2024-11-22T15:00:50Z

DataSource/DataSource/Sources/Interface/NearbyNetworkInterface.swift

@@ -55,3 +55,19 @@ public protocol NearbyNetworkDelegate: AnyObject {
    /// 주변 기기와의 연결에 실패했을 때 실행됩니다.
    func nearbyNetworkCannotConnect(_ sender: NearbyNetworkInterface)
 }
+
+public protocol NearbyNetworkReceiptDelegate: AnyObject {


connection, receipt 적절한 기준으로 잘 나눈거같아용

taipaise requested review from choijungp, eemdeeks and ekrud99 November 20, 2024 10:00

taipaise self-assigned this Nov 20, 2024

taipaise added this to the 데이터 송수신 milestone Nov 20, 2024

taipaise force-pushed the feature/transferFileData branch 2 times, most recently from c0adb53 to 044ee71 Compare November 20, 2024 10:19

eemdeeks approved these changes Nov 20, 2024

View reviewed changes

choijungp approved these changes Nov 20, 2024

View reviewed changes

taipaise added 4 commits November 20, 2024 23:43

feature: 근거리 통신 모듈 인터페이스에 송수신 메서드 추가(#99, #100)

b8b9980

fix: NearbyNetworkDelegate를 ISP에 따라 둘로 분리

4cc4ec9

feature: 근거리 통신 모듈에 url이용한 전송 메서드 추가(#99)

18b5eba

feature: 근거리 통신 모듈에 url이용한 수신 메서드 추가(#100)

3ae769e

taipaise force-pushed the feature/transferFileData branch from e626d5c to 2c0bf1d Compare November 20, 2024 14:44

refactor: 코드리뷰 반영

6db839f

taipaise force-pushed the feature/transferFileData branch from 2c0bf1d to 6db839f Compare November 20, 2024 14:48

taipaise merged commit 621694e into develop Nov 20, 2024
1 check passed

taipaise deleted the feature/transferFileData branch November 21, 2024 04:24

ekrud99 reviewed Nov 22, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feature] 근거리 통신 모듈에 URL을 이용한 통신 메서드 추가 #103

[Feature] 근거리 통신 모듈에 URL을 이용한 통신 메서드 추가 #103

taipaise commented Nov 20, 2024

eemdeeks left a comment

eemdeeks Nov 20, 2024

taipaise Nov 20, 2024

eemdeeks Nov 20, 2024

taipaise Nov 20, 2024

taipaise Nov 20, 2024

eemdeeks Nov 20, 2024

choijungp Nov 20, 2024

eemdeeks Nov 20, 2024

ekrud99 Nov 22, 2024

choijungp left a comment

choijungp Nov 20, 2024

taipaise Nov 20, 2024

choijungp Nov 20, 2024

taipaise Nov 20, 2024

taipaise Nov 20, 2024

choijungp Nov 20, 2024

ekrud99 left a comment

ekrud99 Nov 22, 2024

ekrud99 Nov 22, 2024

[Feature] 근거리 통신 모듈에 URL을 이용한 통신 메서드 추가 #103

[Feature] 근거리 통신 모듈에 URL을 이용한 통신 메서드 추가 #103

Conversation

taipaise commented Nov 20, 2024

🌁 Background

👩‍💻 Contents

✅ Testing

📝 Review Note

📣 Related Issue

📬 Reference

eemdeeks left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

choijungp left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ekrud99 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment