HI Brian - on an iPhone 6 running iOS 11.2.6 and loading a large podcast, I was getting crashes from the CMTime extension because currentItem.duration was still undefined. I was able to fix by checking to see if self.currentItem?.status == .readyToPlay before setting the labels. Seems like a more elegant way would be to change the extension to handle undefined but unsure how to do that.

Instead of calculating a percentage, one can also just cram the value into the slider: self.timeSlider.maximumValue = Float(CMTimeGetSeconds(self.player.currentItem?.duration ?? CMTimeMake(0, 0)))

Hey Brian, Thanks very much for the great material provided, I actually decided to challenge myself to do all of this programatically from the start just to improve my skills and understanding. There are a couple of interesting challenges such as view communication and data passing between objects (I created custom stackViews for many of the elements on this screen so I need to pass data between), I have been using delegation and call backs however it is getting a bit messy, would introducing custom bindings wrappers and then maybe some RxSwift be a good solution in order to comply with the SOLID principles in this case ? Also a quick question, I made a wrong comment about reference cycles on the last video however I am pretty sure here we do have a Strong Reference Cycle created when in the closure of Periodic Time Observer ?

Hey Brian, I cant find in the project where you added the hours? Just to check that my code is the same as what you did :) thanks :)

Little tip: An easy way to get the right VC for your element in the StoryBoard to link by the assistant editor, just press option and click on the two. For example, you're in PlayerDetailsView.swift, hold option and click on PlayerDetailsView.Xib

Hi Brian I believe there is an error in the code snippet above. Need to remove the (60 * 60) in the minutes calculation. Keep up the great videos.

I think need to check duration value with the large audio file. let durationTime = self.player.currentItem?.duration if((durationTime?.value)! > 0){ self.playerDetailsView.durationLbl.text = durationTime?.toDisplayString() }

Damn that refactor -> extract method feature is sick! I love all these features you're showing us!