Skip to content

[#330] 네트워크가 연결되어있음에도 앱을 시작할 시 간헐적으로 네트워크 관련 얼럿이 뜨는 현상을 해결한다#333

Merged
opficdev merged 2 commits intodevelopfrom
fix/#330-Network
Mar 29, 2026
Merged

[#330] 네트워크가 연결되어있음에도 앱을 시작할 시 간헐적으로 네트워크 관련 얼럿이 뜨는 현상을 해결한다#333
opficdev merged 2 commits intodevelopfrom
fix/#330-Network

Conversation

@opficdev
Copy link
Copy Markdown
Owner

@opficdev opficdev self-assigned this Mar 29, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

이 풀 요청은 NWPathConnectivityProvider에서 초기 연결 상태를 nil로 설정하여 초기화 시점의 부정확한 상태 방출을 방지합니다. observeNetworkConnectivity 메서드에 compactMap을 적용하여 유효한 상태만 전달하도록 개선되었습니다. 리뷰에서는 불필요한 중복 이벤트를 방지하기 위해 removeDuplicates() 연산자 추가를 제안했습니다.

func observeNetworkConnectivity() -> AnyPublisher<Bool, Never> {
isConnectedSubject.eraseToAnyPublisher()
isConnectedSubject
.compactMap { $0 }
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

네트워크 연결 상태가 변경될 때만 이벤트를 방출하도록 removeDuplicates() 연산자를 추가하는 것을 고려해 보세요. pathUpdateHandler는 연결 상태(status)가 동일하더라도 다른 경로 속성(예: 사용 가능한 인터페이스 변경)이 변경될 때마다 호출될 수 있습니다. removeDuplicates()를 사용하면 불필요한 중복 true 또는 false 값의 전송을 막아 다운스트림 구독자의 부하를 줄일 수 있습니다.

Suggested change
.compactMap { $0 }
.compactMap { $0 }
.removeDuplicates()

@opficdev opficdev merged commit e95ff19 into develop Mar 29, 2026
1 check passed
@opficdev opficdev deleted the fix/#330-Network branch March 29, 2026 11:57
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.

네트워크가 연결되어있음에도 앱을 시작할 시 간헐적으로 네트워크 관련 얼럿이 뜨는 현상을 해결한다

1 participant