Блог

Я разрабатываю приложение для macOS (Swift / Storyboard — RsyncOSX), и у меня возникла странная проблема в Mojave. Об этом сообщил пользователь, я разрабатываю Catalina (и Big Sur). Класс выполняет объект процесса и прослушивает два уведомления внутри func executeProcess . Есть два @escaping functions , которые выполняются каждый раз, когда появляются уведомления. Это отлично работает на Каталине и Биг-Суре, но не на Мохаве. В предыдущих версиях использовались протоколы и делегаты..

 class RsyncProcessCmdClosure: Delay {
    // Process termination and filehandler closures
    var processtermination: () -> Void
    var filehandler: () -> Void
    // Verify network connection
    ...
    // Observers
    weak var notifications_datahandle: NSObjectProtocol?
    weak var notifications_termination: NSObjectProtocol?
    // Arguments to command
    ...

    func executeProcess(outputprocess: OutputProcess?) {
        ...
        let outHandle = pipe.fileHandleForReading
        outHandle.waitForDataInBackgroundAndNotify()
        // Observator for reading data from pipe, observer is removed when Process terminates
        self.notifications_datahandle = NotificationCenter.default.addObserver(forName: NSNotification.Name.NSFileHandleDataAvailable, object: nil, queue: nil) { [weak self] _ in
            let data = outHandle.availableData
            if data.count > 0 {
                if let str = NSString(data: data, encoding: String.Encoding.utf8.rawValue) {
                    outputprocess?.addlinefromoutput(str: str as String)
                    // Send message about files
                    // ---> Execute closure
                    self?.filehandler()
                    if self?.termination ?? false {
                        self?.possibleerrorDelegate?.erroroutput()
                    }
                }
                outHandle.waitForDataInBackgroundAndNotify()
            }
        }
        // Observator Process termination, observer is removed when Process terminates
        self.notifications_termination = NotificationCenter.default.addObserver(forName: Process.didTerminateNotification, object: nil, queue: nil) { _ in
            self.delayWithSeconds(0.5) {
                self.termination = true
                // ---> Execute closure
                self.processtermination()
                // Must remove for deallocation
                ...
        }
    }
    ...

    init(arguments: [String]?,
         config: Configuration?,
         processtermination: @escaping () -> Void,
         filehandler: @escaping () -> Void)
    {
        self.arguments = arguments
        self.processtermination = processtermination
        self.filehandler = filehandler
        ...
    }

    deinit {
        self.monitor?.stopMonitoring()
        self.monitor = nil
    }
}
  

Мне кажется, что ваш код никогда не должен работать. Для меня загадка, почему это происходит.

Вы говорите

 weak var notifications_datahandle: NSObjectProtocol?
weak var notifications_termination: NSObjectProtocol?
  

weak ? Я бы ожидал, что все, что назначено этим свойствам экземпляра, будет немедленно удалено, и поэтому уведомления никогда не должны приходить.

Возможно, что-то во внутреннем управлении объектами-наблюдателями уведомлений предотвращает это на Каталине и Биг-Суре, так что вам просто повезло. Но в основном я бы сказал, удалить weak . Вы уже используете weak self в замыканиях, поэтому вы нарушили цикл сохранения, и вам больше ничего не нужно делать.

РЕДАКТИРОВАТЬ Нет, это не так! Вы забыли использовать weak self во второй функции наблюдателя:

 self.notifications_termination = NotificationCenter.default.addObserver(forName: Process.didTerminateNotification, object: nil, queue: nil) { _ in
  

Вам нужно weak self там.

Обратите внимание, что вам необходимо отменить регистрацию объектов observer. Обычно это происходит в deinit . Вы вызовете

 NotificationCenter.default.removeObserver(notifications_datahandle)
NotificationCenter.default.removeObserver(notifications_termination)
  

Затем вы можете освободить объекты observer:

 notifications_datahandle = nil
notifications_termination = nil