Week 2 submission

[mspielberg]

This isn't really as polished or feature-rich as I would like it. I ended up putting a log of time into getting iOS <= 7 dynamic table row height working as an educational experience, and having to time-box the project as a whole without implementing additional optional or extra features.

@codepath

[codepathreview]

:+1: nice work. With this assignment, we've now explored all the main patterns for building MVC clients! If this were 2009, you would be well on your way to building most apps that you could find in the app store. Over the next few weeks, we'll be focusing on custom views and view controllers to implement the interactions and visual effects that we find in more modern iOS apps.

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.

• In Objective-C, review your .h files. Your classes should reveal a minimum interface and only expose things that must be used by other classes.
• Each .xib file or Storyboard scene should have Auto Layout constraints specified for each view.
• There should be no yellow Auto Layout warnings, even if the layout seems to work. Auto Layout warnings are like having bad HTML. The browser will try to render it, and it might even look right, but it won't be fully predictable.
• 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.

[codepathreview]

:+1: nice work. The primary challenge of this homework was to learn how to use UITableView to manage a complex form, establish communication between view controllers, and apply the correct Auto Layout constraints. For each point in the homework feedback checklist, I'll either confirm that you're on the right track with the implementation, or I'll provide some feedback:

• For the restaurant cell, nice work encapsulating the properties into the private class extension.
• Nice work specifying the Auto Layout constraints.
• in FiltersViewController, your way of opening cells is not standard. I would still advise to only use dequeueReusableCellWithIdentifier, and make sure that every time you on cellForRowAtIndexPath you set the right values
• Nice work connecting the view controllers via the custom filter delegate. This is a popular pattern for handling events between view controllers.

This was a challenging assignment, congrats on completing it successfully!

-- dirk