Coder Social home page Coder Social logo

Submission about rottentomatoes HOT 1 OPEN

stepan avatar stepan commented on June 24, 2024
Submission

from rottentomatoes.

Comments (1)

timothy1ee avatar timothy1ee commented on June 24, 2024

Stepan, good job! The point of this homework was to explore a simple example of a full MVC application with a RESTful API.

  • Good code styling. The New York Times has a good summary of appropriate Objective-C coding conventions, you should take a quick look here.
    • Note that by convention, getters in Objective-C don't start with the word "get". The first word of the method is the thing that is returned. For instance, it wouldn't be + (NSArray *) getMovies:(id) object;, it would be + (NSArray *)moviesWithObject:(id)object;
    • Your whitespace is a bit off, take a look at the conventions doc.
  • Make sure to put all private methods and properties in the class extension interface instead of the public interface in the header file.
    • If you have an initializer with a variable, by convention, you should make that variable available as a public property (e.g., MovieViewController). I think you have some things exposed there that I wouldn't have made public.

For your information, here is the checklist that I use when I review the homework:

  • Movie model. Did you have some way to create it from a dictionary?
  • You should have implemented a custom cell. 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.
  • You should have loaded all the images asynchronously. One way to do that was to use AFNetworking, installed via CocoaPods.
  • You should observe the UIContentMode of the UIImageView. By default, it will stretch the image to match your dimensions, which is probably not what you want.
  • If you had a custom cell, you should configure the Auto Layout parameters. We'll review that in class.
  • Did you display some loading indicator while you were fetching the movies.
  • Did you display an error if you failed while fetching the movies.

from rottentomatoes.

Related Issues (1)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.