Flicks optional requirements

[sameekhan]

// // CollectionViewController.swift // babyFlicks // // Created by Samee Khan on 1/31/16. // Copyright © 2016 Samee Khan. All rights reserved. //

import UIKit

private let reuseIdentifier = "Cell"

class CollectionViewController: UIViewController, UICollectionViewDataSource, UISearchBarDelegate {

@IBOutlet weak var collectionView: UICollectionView!
@IBOutlet weak var networkError: UILabel!
@IBOutlet weak var search: UISearchBar!

var movies: [NSDictionary]?
var filteredData: [NSDictionary]?
var movieTitle: [String] = []

override func viewDidLoad() {
    super.viewDidLoad()

    self.networkError.hidden = true
    collectionView.dataSource = self
    search.delegate = self

    let refreshControl = UIRefreshControl()
    refreshControl.addTarget(self, action: "refreshControlAction:", forControlEvents: UIControlEvents.ValueChanged)
    collectionView.addSubview(refreshControl)

    let apiKey = "a07e22bc18f5cb106bfe4cc1f83ad8ed"
    let url = NSURL(string: "https://api.themoviedb.org/3/movie/now_playing?api_key=\(apiKey)")
    let request = NSURLRequest(
        URL: url!,
        cachePolicy: NSURLRequestCachePolicy.ReloadIgnoringLocalCacheData,
        timeoutInterval: 10)

    let session = NSURLSession(
        configuration: NSURLSessionConfiguration.defaultSessionConfiguration(),
        delegate: nil,
        delegateQueue: NSOperationQueue.mainQueue()
    )

    let task: NSURLSessionDataTask = session.dataTaskWithRequest(request,
        completionHandler: { (dataOrNil, response, error) in
            if let data = dataOrNil {
                if let responseDictionary = try! NSJSONSerialization.JSONObjectWithData(
                    data, options:[]) as? NSDictionary {
                        print("response: \(responseDictionary)")

                        self.movies = (responseDictionary["results"] as! [NSDictionary])
                        self.filteredData = self.movies
                        self.collectionView.reloadData()
                }
            }
                 else {
                self.networkError.hidden = false
                self.search.hidden = true
            }
    })
    task.resume()        // Do any additional setup after loading the view.
}

override func viewDidAppear(animated: Bool) {
    EZLoadingActivity.showWithDelay("Loading...", disableUI: true, seconds: 1.25)
}

// creating the refresh control actions
let refreshControlAction = UIRefreshControl()
func refreshControlAction(refreshControl: UIRefreshControl)
{
    // ... Create the NSURLRequest (myRequest) ...

    let myRequest = NSURLRequest()

    // Configure session so that completion handler is executed on main UI thread
    let session = NSURLSession(
        configuration: NSURLSessionConfiguration.defaultSessionConfiguration(),
        delegate:nil,
        delegateQueue:NSOperationQueue.mainQueue()
    )
    let task : NSURLSessionDataTask = session.dataTaskWithRequest(myRequest,
        completionHandler: { (data, response, error) in

            // ... Use the new data to update the data source ...

            // Reload the tableView now that there is new data
            self.collectionView.reloadData()

            // Tell the refreshControl to stop spinning
            refreshControl.endRefreshing()
    });
    refreshControl.addTarget(self, action: "refreshControlAction", forControlEvents: UIControlEvents.ValueChanged)
    collectionView.addSubview(refreshControl)
    task.resume()

    // Uncomment the following line to preserve selection between presentations
    // self.clearsSelectionOnViewWillAppear = false

    // Register cell classes
    self.collectionView!.registerClass(UICollectionViewCell.self, forCellWithReuseIdentifier: reuseIdentifier)

    // Do any additional setup after loading the view.
}

override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
    // Dispose of any resources that can be recreated.
}

/*
// MARK: - Navigation

// In a storyboard-based application, you will often want to do a little preparation before navigation
override func prepareForSegue(segue: UIStoryboardSegue, sender: AnyObject?) {
    // Get the new view controller using [segue destinationViewController].
    // Pass the selected object to the new view controller.
}
*/

// MARK: UICollectionViewDataSource

// func numberOfSectionsInCollectionView(collectionView: UICollectionView) -> Int { // // #warning Incomplete implementation, return the number of sections // // }

 func collectionView(collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
    // #warning Incomplete implementation, return the number of items
    if let movies = filteredData {
        return filteredData!.count
    } else {
        return 0
        }
}

 func collectionView(collectionView: UICollectionView, cellForItemAtIndexPath indexPath: NSIndexPath) -> UICollectionViewCell {
    let cell = collectionView.dequeueReusableCellWithReuseIdentifier("MovieCollCell", forIndexPath: indexPath) as! MovieCollCell

    let movie = filteredData![indexPath.row]
    let flowPosterPath = movie["poster_path"] as! String

    let baseUrl = "http://image.tmdb.org/t/p/w500"

    let imageUrl = NSURL(string: baseUrl + flowPosterPath)

    let imageRequest = NSURLRequest(URL: imageUrl!)
    cell.flowPosterView.setImageWithURLRequest(
        imageRequest,
        placeholderImage: nil,
        success: { (imageRequest, imageResponse, image) -> Void in

            // imageResponse will be nil if the image is cached
            if imageResponse != nil {
                print("Image was NOT cached, fade in image")
                cell.flowPosterView.alpha = 0.0
                cell.flowPosterView.image = image
                UIView.animateWithDuration(1.2, animations: { () -> Void in
                    cell.flowPosterView.alpha = 1.0
                })
            } else {
                print("Image was cached so just update the image")
                cell.flowPosterView.image = image
            }
        },
        failure: { (imageRequest, imageResponse, error) -> Void in
            // do something for the failure condition
    })
    cell.flowPosterView.setImageWithURL(imageUrl!)

    print("row \(indexPath.row)")
    return cell
}

func searchBar(searchBar: UISearchBar, textDidChange searchText: String) {

    filteredData = searchText.isEmpty ? movies : movies!.filter({(movie: NSDictionary) -> Bool in

        let title = movie["title"] as! String
        let overview = movie["overview"] as! String

        return title.rangeOfString(searchText, options: .CaseInsensitiveSearch) != nil || overview.rangeOfString(searchText, options: .CaseInsensitiveSearch) != nil
    })
    collectionView.reloadData()
}

@IBAction func onTap(sender: AnyObject) {
    view.endEditing(true)
}

// MARK: UICollectionViewDelegate

/*
// Uncomment this method to specify if the specified item should be highlighted during tracking
override func collectionView(collectionView: UICollectionView, shouldHighlightItemAtIndexPath indexPath: NSIndexPath) -> Bool {
    return true
}
*/

/*
// Uncomment this method to specify if the specified item should be selected
override func collectionView(collectionView: UICollectionView, shouldSelectItemAtIndexPath indexPath: NSIndexPath) -> Bool {
    return true
}
*/

/*
// Uncomment these methods to specify if an action menu should be displayed for the specified item, and react to actions performed on the item
override func collectionView(collectionView: UICollectionView, shouldShowMenuForItemAtIndexPath indexPath: NSIndexPath) -> Bool {
    return false
}

override func collectionView(collectionView: UICollectionView, canPerformAction action: Selector, forItemAtIndexPath indexPath: NSIndexPath, withSender sender: AnyObject?) -> Bool {
    return false
}

override func collectionView(collectionView: UICollectionView, performAction action: Selector, forItemAtIndexPath indexPath: NSIndexPath, withSender sender: AnyObject?) {

}
*/

}

[sameekhan]

I have completed (most of) the optional requirements. It is ready for review. /cc @codepathreview

[codepathreview]

:+1: nice work. The point of this homework was to explore a simple example of a full MVC application with a RESTful API.

Here's a checklist of things that I'm looking for in this project. You should review the checklist, and make any changes to your project, if necessary.

• Code styling.
  • In Objective-C, The New York Times has a good summary of appropriate Objective-C coding conventions, you should take a quick look here.
  • In Swift, check out the Ray Wenderlich Swift Style Guide and the Github Swift Style Guide
• In Objective-C, review your .h files. In Swift, review the properties and methods that you've made public. Your classes should reveal a minimum interface and only expose things that must be used by other classes.
• If you're using Objective-C, you can consider using Mantle, a lightweight ORM that eliminates a lot of boilerplate. It's also compatible with Core Data if your project requires that.
• If you're using Swift, you can consider using SwiftyJSON for easier deserialization of the JSON response.
• When creating a custom cell, make sure never to use imageView, titleLabel, or subtitleLabel. Those are already used by UITableViewCell and using them in your custom cell will cause subtle things to happen such as image views not loading until you click on the cell.
• The images should be downloaded asynchronously. One way to do that was to use AFNetworking, installed via CocoaPods.
• Observe the UIContentMode of the UIImageView. By default, it will stretch the image to match your dimensions, which is probably not what you want.
• Next week, we'll cover how to use Auto Layout constraints to implement dynamic height rows and adjust to synopsis of different length.

In general, you should challenge yourself to create visually polished applications. That's where you'll find all the rough edges of the iOS framework and earn the next level of mastery.