udacityandroidnanodegreeprjs / moviesapp Goto Github PK

As per the Reviewer,

Your naming convention for fields is a bit inconsistent here. This variable is prefixed with an "m", whereas most of the other ones are not. You should pick one of these conventions and use it consistently to improve your code's readability. Typically, people prefix with an "m" and use camelCasing; however, I've also seen others use straight camelCasing with no "m" prefix. The choice is yours!

As per the suggestion by udacity reviewer,says that:

You should use an empty view. An empty view is a view that appears in a gridview, listview, recyclerview, or equivalent when no data is present to display. For example, if the internet is disabled

Because you are using a recyclerview, you have to implement this yourself, as gridview, and listview already have this functionality built in. You can learn how to do this a couple of different ways here: http://stackoverflow.com/questions/28217436/how-to-show-an-empty-view-with-a-recyclerview
My personal suggestion is to extend recyclerview.

This helps user experience by showing why the application is behaving a certain way. In this case, it will alert the user the app won't work correctly without internet.