Question

我试图改进我分叉的GitHub项目（https://github.com/giacmarangoni/Swift-Radio-Pro/tree/xcode8）。经过一些修复和改变后，一切似乎都很好但突然间我发现了一个非常奇怪的行为。当我打开＆＃34; NowPlayingViewController＆＃34;第一次和站开始流，一切正常，AVPlayer委托按预期更新用户界面（songLabel，titleLabel和albumArtwork）。之后，在没有停止无线电流的情况下，我试图回到＆＃34; StationsViewController＆＃34;并立即重新打开＆＃34; NowPlayingViewController＆＃34;使用＆＃34;正在播放＆＃34;按钮。
此时委托仍处于活动状态，流式传输正在进行，但是当歌曲更改时，此视图控制器中的所有变量都会更新，但我不能对用户界面说同样的内容。我试图调试，我注意到标签已填充但未更新。主线程中的UI更新和setNeedDisplay没有帮助。

NowPlayingViewController

AVPlayer设置：

func setUpPlayer(){
        radioPlayer = Player.radio
        radioPlayer.rate = 1
        NotificationCenter.default.addObserver(
            self,
            selector: #selector(self.playerItemDidReachEnd),
            name: NSNotification.Name.AVPlayerItemDidPlayToEndTime,
            object: self.radioPlayer.currentItem
        )

    }

您可以在这里找到func onMetaData(_ metaData: [AVMetadataItem]?)）。

//*****************************************************************
// MARK: - AVPlayerItem Delegate (for metadata)
//*****************************************************************
extension NowPlayingViewController: CustomAVPlayerItemDelegate {
    func onMetaData(_ metaData: [AVMetadataItem]?) {
        if let metaDatas = metaData{
            startNowPlayingAnimation()
            let firstMeta: AVMetadataItem = metaDatas.first!
            let metaData = firstMeta.value as! String
            var stringParts = [String]()
            if metaData.range(of: " - ") != nil {
                stringParts = metaData.components(separatedBy: " - ")
            } else {
                stringParts = metaData.components(separatedBy: "-")
            }
            // Set artist & songvariables
            let currentSongName = track.title
            track.artist = stringParts[0].decodeAllChars()
            track.title = stringParts[0].decodeAllChars()
            if stringParts.count > 1 {
                track.title = stringParts[1].decodeAllChars()
            }                
            if track.artist == "" && track.title == "" {
                track.artist = currentStation.stationDesc
                track.title = currentStation.stationName
            }

            DispatchQueue.main.async {
                if currentSongName != self.track.title {
                    if kDebugLog {
                        print("METADATA artist: \(self.track.artist) | title: \(self.track.title)")
                    }

                    // Update Labels
                    self.artistLabel.text = self.track.artist
                    self.songLabel.text = self.track.title
                    self.updateUserActivityState(self.userActivity!)
                    // songLabel animation
                    self.songLabel.animation = "zoomIn"
                    self.songLabel.duration = 1.5
                    self.songLabel.damping = 1
                    self.songLabel.animate()
                    // Update Stations Screen
                    self.delegate?.songMetaDataDidUpdate(self.track)
                    // Query API for album art
                    self.resetAlbumArtwork()
                    self.queryAlbumArt()
                }
            }
        }
    }
}

此方法在＆＃34; CustomAVPlayerItem＆＃34;根据{{1}}关键路径;每当AVPlayer元数据发生变化时，它就会被解雇。该类是AVPlayerItem的子类：

timedMetaData

以下是我的AVPlayer：

import MediaPlayer
import Foundation

protocol CustomAVPlayerItemDelegate {
    func onMetaData(_ metaData:[AVMetadataItem]?)
}

//*****************************************************************
// Makes sure that observers are removed before deallocation
//*****************************************************************
class CustomAVPlayerItem: AVPlayerItem {

    var delegate : CustomAVPlayerItemDelegate?

    init(url URL:URL)
    {
        if kDebugLog {print("CustomAVPlayerItem.init")}
        super.init(asset: AVAsset(url: URL) , automaticallyLoadedAssetKeys:[])
        addObserver(self, forKeyPath: "timedMetadata", options: NSKeyValueObservingOptions.new, context: nil)
    }

    deinit{        
        if kDebugLog {print("CustomAVPlayerItem.deinit")}
        removeObserver(self, forKeyPath: "timedMetadata")
    }

    override func observeValue(forKeyPath keyPath: String?, of object: Any?, change: [NSKeyValueChangeKey : Any]?, context: UnsafeMutableRawPointer?) {
        if let avpItem: AVPlayerItem = object as? AVPlayerItem {
            if keyPath == "timedMetadata" {                
                delegate?.onMetaData(avpItem.timedMetadata)
            }
        }
    }
}

这是我用来打开＆＃34; NowPlayingViewController＆＃34;。 StationsViewController

的segue函数

import MediaPlayer

//*****************************************************************
// This is a singleton struct using Swift
//*****************************************************************
struct Player {
    static var radio = AVPlayer()
}

Answer 1

以下是我认为您不理解的内容以及实际发生的事情。

通常，当你＆＃34;回去＆＃34;从推送的视图控制器中，推送的视图控制器弹出并销毁。推送的视图控制器是NowPlayingViewController。当你＆＃34;回去＆＃34;它应该被销毁从它到StationsViewController。因此，当您再次显示NowPlayingViewController 时，您必须创建一个新的，不同的 NowPlayingViewController。

好的，到目前为止这么好，只要你了解所有这些。但是在你的案例中还有一个复杂的问题：你有泄漏！您的旧NowPlayingViewController 不被销毁。因此，当你＆＃34;回去＆＃34;到StationsViewController并再次显示NowPlayingViewController，现在有两个 NowPlayingViewControllers - 你看到的新的，以及旧的泄漏。

好的，所以你的日志记录继续显示旧 NowPlayingViewController，它仍在观察和更新。但是你的眼睛正在看到 new NowPlayingViewController，它什么也没做。这解释了你所描述的现象。

如果这是对的 - 并且，从你所说的，我很确定它是 - 那么你需要重新组织你的架构，以便你不会发生这样的泄漏当您第二次显示相同 NowPlayingViewController而不是创建另一个时，显示NowPlayingViewController。（第一种方法会更好。）

标签不会使用Swift更新

1 个答案: