Hey Brian, why are you using a Singleton Technique for a Networking Layer. Surely using a "Godlike" class is not a sustainable solution as more API call need to be made ? For example Authentication calls and data .get requests would be better off separate, no ?

Hi Brian, great lesson on keeping the VC small. Question on the async search, don't we need to handle the results and put it in the main thread while its still searching?

Oh 1 more thing, after doing this vid, I noticed that when running the simulator, when I click on the searchtab, i'm getting [MC] Lazy loading NSBundle MobileCoreServices.framework 2018-02-24 16:28:22.255180+0900 MyPodcastClone[5965:197565] [MC] Loaded MobileCoreServices.framework and when i click on the search textfield, I'm getting 2018-02-24 16:28:32.134838+0900 MyPodcastClone[5965:197565] [MC] System group container for systemgroup.com.apple.configurationprofiles path is /Users/jonathango/Library/Developer/CoreSimulator/Devices/FE27D17E-C382-49EE-BE52-44E0A0A1AEB9/data/Containers/Shared/SystemGroup/systemgroup.com.apple.configurationprofiles 2018-02-24 16:28:32.137136+0900 MyPodcastClone[5965:197565] [MC] Reading from private effective user settings. 2018-02-24 16:28:32.228017+0900 MyPodcastClone[5965:197565] +[CATransaction synchronize] called within transaction 2018-02-24 16:28:32.228485+0900 MyPodcastClone[5965:197565] +[CATransaction synchronize] called within transaction 2018-02-24 16:28:32.229263+0900 MyPodcastClone[5965:197565] +[CATransaction synchronize] called within transaction 2018-02-24 16:28:32.229706+0900 MyPodcastClone[5965:197565] +[CATransaction synchronize] called within transaction Am I doing something wrong? How should I address this?

quick question - in XCode, I'm not getting the 'Fix' on errors? Is there some setting I need to be aware of?

Hi Brian, really enjoying the podcast videos so for. In the completion handler, do you not need call this on the main thread? i.e DispatchQueue.main.async { self.podcasts = podcasts self.tableView.reloadData() } Also, in order to prevent a retain cycle, doesn't an [unowned self] need to be added?

Hi,Instead of using a completion handler technically we ca use a NSNotification right? which one is better to use between both.

Inside func searchBar(_ searchBar: UISearchBar, textDidChange searchText: String) APIService.shared.fetchPodcasts(searchText: searchText) { (podcasts) in self.podcasts = podcasts self.tableView.reloadData() } option 2: APIService.shared.fetchPodcasts(searchText: searchText) -> [podcasts] == I like your usage of the @escape clause but if we went with the “option 2:” I listed above, I feel that I we could have returned an array of podcasts and then reloaded the tableView from calling function ‘func searchBar(_ searchBar: UISearchBar, textDidChange searchText: String)‘. Is the advantage of doing it with @escaping clause to help with code organization only? Or is there a performance/logic gain that I’m overlooking. Also, thank you for uploading such an excellent, informative video.

Hi Brian for networking if we use static class method instead of singleton is there any difference.Which is the best practice

Hey Brian, love this course ! When will you made a full project with MVP or MVVM ? I'm intern and my company use MVP and sometimes it's hard to make difference between all these patterns. It use encapsulated protocol and function and often I'm lost with all these new things. What do you think about it ? Thanks a lot. Regards