Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[App Fundamentals] Flicks 2 - Review my app #6

Open
Brian-Kaplan opened this issue Feb 2, 2016 · 1 comment
Open

[App Fundamentals] Flicks 2 - Review my app #6

Brian-Kaplan opened this issue Feb 2, 2016 · 1 comment

Comments

@Brian-Kaplan
Copy link
Owner

My week 2 prework is complete. /cc @codepathreview

@codepathreview
Copy link

👍 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, 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants