Review my app

[h2kuo]

Please review my app! /cc @codepathreview

[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.

-- dirk

[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.
• in ChoiceCell.xib, make sure you resolve all the Auto Layout constraint warnings.
• in MainViewController, cellForRowAtIndexPath, I'm not a big fan of controlling code paths with return statements mid method
• Nice work linking the filter model with the controls. See this popular form library for an alternative to implementing forms manually.
• 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