[Fix] EventMonitor의 저장 프로퍼티 관련 경고 해결

[youz2me]

👻 PULL REQUEST

📄 작업 내용

• OAuthErrorMonitor 구현 시 선언한 저장 프로퍼티로 인해 발생한 경고를 해결했어요.

💻 주요 코드 설명

EventMonitor의 condition 속성 변경

final class OAuthErrorMonitor: EventMonitor {
    let queue = DispatchQueue(label: "OAuthErrorMonitorQueue", attributes: .concurrent)
    private static var condition = false
 
    var isUnauthorized: Bool {
        get {
            queue.sync { OAuthErrorMonitor.condition }
        }
        set {
            queue.async(flags: .barrier) {
                OAuthErrorMonitor.condition = newValue
            }
        }
    }
 
...

• 원래 저장 프로퍼티 형식으로 선언되어 있던 condition을 정적 프로퍼티로 변경했습니다.
  • 이 방식은 경고를 해결하긴 하지만 여러 OAuthErrorMonitor 인스턴스가 있는 경우 모두 동일한 상태를 공유하고 한 인스턴스에서 인증 실패를 감지하면 모든 인스턴스에 영향을 미칠 수 있다는 단점이 있는데요.
  • OAuthErrorMonitor는 결국 앱에서 하나의 인스턴스만 선언되어야 하기 때문에 변경 가능하다는 결론을 내렸습니다.

👀 기타 더 이야기해볼 점

• 자고싶어요

🔗 연결된 이슈

• Resolved: [Fix] Interceptor 오류 수정 #106

[JinUng41]

고생하셨습니다~

OAuthErrorMonitor가 앱 내에서 하나만 존재해여 한다면, 싱글톤 패턴을 사용해야 할까 하는 고민이 생겼습니다.

[JinUng41]

Self.condition도 가능하답니다~

[youz2me]

위와 같은 오류가 발생해 부득이하게 OAuthErrorMonitor를 명시했습니다!

[JinUng41]

저는 대문자 Self.condition을 말한거긴 한데, 혹시 이것도 적용이 안되나요?

[youz2me]

오마이갓 클래스 단위는 Self로 참조가 가능하군요 ... 바로 적용했습니다 감사합니닷