使用Swift

Question

我有以下情况。根控制器是UITabViewController。有一个ProfileViewController，其中我创建了一个观察者，用户开始成为朋友（然后屏幕功能发生变化）。可以使用5个中的4个选项卡打开ProfileViewController，因此当前用户可以在四个位置使用相同的用户打开屏幕。在以前的版本中，当ProfileViewController在一个地方打开时，我删除了deinit中的观察者并仅通过ref.removeAllObservers()进行删除，现在当用户案例是这样的时候，我开始在viewDidDisappear中使用句柄和删除观察者。我想展示一下代码，看看它是否可以改进，以及我是否在这种情况下做得正确。

我在viewWillAppear

中调用此函数

   fileprivate func firObserve(_ isObserve: Bool) {
        guard let _user = user else { return }
        FIRFriendsDatabaseManager.shared.observeSpecificUserFriendshipStart(observer: self, isObserve: isObserve, userID: _user.id, success: { [weak self] (friendModel) in
        }) { (error) in

        }
    }

这是在FIRFriendsDatabaseManager

中

fileprivate var observeSpecificUserFriendshipStartDict = [AnyHashable : UInt]()


func observeSpecificUserFriendshipStart(observer: Any, isObserve: Bool, userID: String, success: ((_ friendModel: FriendModel) -> Void)?, fail: ((_ error: Error) -> Void)?) {
        let realmManager = RealmManager()
        guard let currentUserID = realmManager.getCurrentUser()?.id else { return }
        DispatchQueue.global(qos: .background).async {
            let specificUserFriendRef = Database.database().reference().child(MainGateways.friends.description).child(currentUserID).child(SubGateways.userFriends.description).queryOrdered(byChild: "friendID").queryEqual(toValue: userID)

            if !isObserve {
                guard let observerHashable = observer as? AnyHashable else { return }
                if let handle = self.observeSpecificUserFriendshipStartDict[observerHashable] {
                    self.observeSpecificUserFriendshipStartDict[observerHashable] = nil
                    specificUserFriendRef.removeObserver(withHandle: handle)
                    debugPrint("removed handle", handle)
                }
                return
            }

            var handle: UInt = 0

            handle = specificUserFriendRef.observe(.childAdded, with: { (snapshot) in
                if snapshot.value is NSNull {
                    return
                }
                guard let dict = snapshot.value as? [String : Any] else { return }
                guard let friendModel = Mapper<FriendModel>().map(JSON: dict) else { return }

                if friendModel.friendID == userID {
                    success?(friendModel)
                }
            }, withCancel: { (error) in
                fail?(error)
            })

            guard let observerHashable = observer as? AnyHashable else { return }
            self.observeSpecificUserFriendshipStartDict[observerHashable] = handle
        }
    }

Answer 1

关于维护对每个viewController的引用的实现，我会考虑将逻辑移动到viewController本身的扩展。

如果你想避免像以前一样调用ref.removeAllObservers()，并假设每个viewController只有一个这样的监听器。我让侦听器在视图控制器上引用变量。

这样一切都只包含在viewController中。如果其他类型的viewControllers将对侦听器进行类似类型的管理，它也可能是创建协议的良好候选者。