bprabhatkiran / TwitterClient

Demo Twitter Client to play around with Twitter's REST API...
0 stars 0 forks source link

Twitter Client #1

Open bprabhatkiran opened 10 years ago

bprabhatkiran commented 10 years ago

Week 3 Assignment

I tried to incorporate persistence using ActiveAndroid, but my models already extend BaseModel and since Java doesnt support multiple inheritance, I had to stop there. I tried refactoring the Model for this, but it was getting pretty complicated as I had to define relationships between a User and a Tweet model.

/cc @nesquena @timothy1ee

nesquena commented 10 years ago

:+1: nice work overall. A few notes after checking out the code:

Here's a detailed Project 3 Feedback Guide here which covers the most common issues with this submitted project. Read through the feedback guide point-by-point to determine how you could improve your submission.

Let us know if you have any other thoughts or questions about this assignment. Hopefully by now you feel pretty comfortable with all the major pieces to basic Android apps (Views, Controllers, Models, Authentication, API Communication, Preferences, ActionBar, et al) and see how they all fit together. We are close to a turning point in the course where you should be hitting a "critical mass" for your Android knowledge.

nesquena commented 10 years ago

I tried to incorporate persistence using ActiveAndroid, but my models already extend BaseModel and since Java doesnt support multiple inheritance, I had to stop there. I tried refactoring the Model for this, but it was getting pretty complicated as I had to define relationships between a User and a Tweet model.

Basically just remove the BaseModel and create fields for each of your models. Then instead of storing jsonObject, just store the field values directly. See this guide for more details.