swift メモリ破棄

Question

現在CustomViewでAVPayerを使って動画を流しているのですが、
addPeriodicTimeObserverForIntervalこのメソッドのところを記述してから遷移してもメモリが破棄されなくなってしまいました。何が原因なのでしょうか？

class CustomView {
var player: AVPlayer!
@IBOutlet weak var slider: UISlider!
private var token: dispatch_once_t = 0

func setPlayer() {
        if let url = NSURL(string: ("動画URL")!) {
            let player = AVPlayer(URL: url)
            let playerlayer = AVPlayerLayer(player: player)
            self.player = player
            playerlayer.frame = CGRect(x: 0, y: 0, width: self.frame.width, height: self.rame.height)
            self.layer.addSublayer(playerLayer)
            player.play()
            playerlayer.addSublayer(slider.layer)
        }
}

func setSlider() {
  slider.value = 0
  slider.minimumValue = 0
  slider.maximumValue = 60
  let interval : Double = Double(0.5 * slider.maximumValue) / Double(slider.bounds.maxX)
  let time : CMTime = CMTimeMakeWithSeconds(interval, Int32(NSEC_PER_SEC))
  player?.addPeriodicTimeObserverForInterval(time, queue: dispatch_get_main_queue()) { (time) -> Void in
    let time = CMTimeGetSeconds(self.player!.currentTime()) 
    let value = Float(self.slider.maximumValue - self.slider.minimumValue) * Float(time) / Float(60) + Float(self.slider.minimumValue)
    self.finish(time) 
    self.slider.value = value self.timeLabel.text = "0:0\(Int(time))
    }
}

  func finish(time: Float64) {
    if time >= Float64(60) {
       dispatch_once(&self.token) {
         self.player?.pause()
         self.player?.seekToTime(CMTimeMakeWithSeconds(0, Int32(NSEC_PER_SEC)))
        }
     }
  }
}

１分たったら、動画を止めるメソッドです。

Comments

[fuzzball]

addPeriodicTimeObserverForIntervalの中のBlocksは質問に書かれているようにカラなのでしょうか？また、「メモリが破棄されない」と思った根拠を書いてください。

[tosi_1211]

Blocks内で、
let time = CMTimeGetSeconds(self.player!.currentTime())
let value = Float(self.slider.maximumValue - self.slider.minimumValue) * Float(time) / Float(10) + Float(self.slider.minimumValue)
self.finish(time)
self.slider.value = value
self.timeLabel.text = "0:0\(Int(time))"

動画の現在時間を取得して、ラベルのテキストに表示しています。
xcodeの左側のmenubarの
CPUをみるところで、Memoryが動画を再生するたびに増えていきます。

[fuzzball]

self.finishは何をしているのでしょうか？コードを教えて下さい。（コードは質問内に追記して下さい。ここに書かれると見難いです）

[tosi_1211]

追記しました！

[fuzzball]

こちらで症状を再現させてから調べようと思っていましたが、情報を出していただくたびに不明点が出てくる状態で、これではキリがないので対応を中止します。他の方からの回答をお待ち下さい。

[tosi_1211]

わかりました。ご丁寧な対応ありがとございました。

Answer 1

動画再生されるたびにメモリ使用量が増えるとありますが、**setPlayer()**は、再生するたびに毎回呼ばれますか？

Comments

[tosi_1211]

毎回呼ばれています！

[Asunaro]

回答ありがとうございます。

setFrame()内の下記コードが気になりました。

>self.layer.addSublayer(playerLayer)

再生されるたびにAVPlayerが生成されて、self.layerに対して貼り付けているようにみえるのですが。。。
（以前にaddしているAVPlayerがremoveされているのであれば、すみません！）

[Asunaro]

すみません、setPlayerでした。。。

あと、毎回Playerを生成したり、PlayerLayerを生成する必要があるのでしょうか？

[tosi_1211]

認識違いでした、setPlayerはinit時にしか呼ばれないです！

[Asunaro]

であれば、動画再生はどこでコールされますか？
記載して頂いているソースコードでは、setPlayer()でしか再生していないように見えます。

[tosi_1211]

動画再生もsetPlayerでのみplayしています。
画面遷移したと同時に動画を流す仕様を作っています。

[tosi_1211]

addPeriodicTimeObserverForIntervalこのメソッドの中をすべて消したらメモリが破棄されたので、そこに問題があります。

[Asunaro]

すみません、問題点を見失っていました。

addPeriodicTimeObserverForIntervalは、通知をObserverに対して登録するため、画面を離れる際に通知をremoveしてあげる必要があります。
多分ですが、通知登録したまま画面遷移したことでObserverに情報が残ってしまい、ブロック内に書かれているplayerやsliderの参照が外れず解放されていないのではと思われます。

因みにObserver解除は、下記になります。

removeTimeObserver

[tosi_1211]

ブロック内に書かれているplayerやsliderの参照が外れず解放されていないのではと思われます。

参照の開放はどのようにすればいいのでしょうか？

[Asunaro]

メモリ管理方式がARCなので、playerやsliderの参照（このオブジェクトを知っている人）がなくなれば、勝手に解放されます。

今回参照を持っている人は、ブロック自体になるのでこのブロック自体がなくなれば、参照が消えます。
よって、オブジェクトがメモリから解放されるはずです。
（playerは、強参照になっているため）

なので、playerに対して、removeTimeObserver()をコールしてみてください。

[tosi_1211]

ありがとうございます！実装してみます。

[tosi_1211]

removeTimeObserverしたのですがダメでした。

[tosi_1211]

let time = CMTimeGetSeconds(self.player!.currentTime())
原因を突き詰めたところ、ブロック内のこの一行が悪さをしていました。

[tosi_1211]

しかしどうすれば良いのかがわかりません。

[Asunaro]

>removeTimeObserverしたのですがダメでした

具体的にどの箇所に追加してダメだったか等を記載して頂かないと原因の追求が出来ません。

[tosi_1211]

self.player.addPeriodicTimeObserverForInterval(CMTimeMake(150, 600), queue: dispatch_get_main_queue(), usingBlock: {[unowned self](time) in

})

addPeriodicTimeObserverForIntervalメソッドを[unowned self]とすることで解決いたしました