Yelp Client Submission

[mdn0420]

@nesquena @timothy1ee

[timothy1ee]

Good job! Linking the view controllers via the client singleton with event registration is an interesting approach. Beware overusing singletons, as you know, it's a fine line, and can lead to classes being overly coupled and hard to maintain.

• Defining custom delegates is an important pattern, you should definitely experiment with it.
• Alternatively, you should check out NSNotificationCenter for a local pub/sub system that is a more data-centric approach for defining interaction between classes.

The purpose of this assignment was to explore the following:

• Table views with dynamic height rows and custom cells with Auto Layout.
• Table views that contain controls for user input and binding those controls to form data.
• Advanced features of table views such as row animation and section views.
• Linking two view controllers view a custom delegate pattern.

Here's my checklist as I'm reviewing the submissions:

• Code styling. The New York Times has a good summary of appropriate Objective-C coding conventions, you should take a quick look here.
• Custom cells.
  • Are the subviews of the cell exposed in the public header or are they properly encapsulated?
  • Do the nibs have the proper Auto Layout constraints?
  • Are the cells registered with the table view?
• Filter view controller
  • Are the controls properly bound with the form data?
  • Is there a custom delegate to notify the search view controller of new filter settings?
  • Is there a section in the table view that accordians out?