Pull to refresh

Comments 7

серверный код просто ужасен, нет тестов, не свежий Rails 4.1.1 и ужасно пахнет (константы в моделях, хотя можно enum, as_jsonn с двумя nn) все от моделей до контроллеров, не надо так!
pull request не кто не отменял. Тем более ребята старались для общего блага. Да и в такой короткий срок как выходные.
Можно было сделать еще быстрее, если:
1) для отображения картинок использовать Picasso или UniversalImageLoader
2) для работы с базой использовать какой-нибудь ORM, например ActiveAndroid
3) для формирования и выполнения http запросов использовать Retrofit

Еще я бы не называл Volley заменой HttpClient'a, т.к. они выполняют разные задачи
Согласен, еще с Retrofit отлично сочетается RxJava и Retrolambda.
А так больше похоже на сборник велосипедов. Знать как эти вещи делаются «руками» — хорошо, но если цель была написать приложение по-быстрому, то лучше выбрать правильные инструменты.
Это же круто что ребята сделали велосипед! Для себя старались и теперь знают как сделать велосипед получше.
Что-то судя по коммитам клиент явно не за несколько выходных писался.
Немного комментариев:
  • Как уже сказали ORM для работы с базой и для картинок можно было Picasso. И тут вопрос — Volley же вроде умеет картинки грузить насколько я помню? В адаптере AsyncTask как то не очень :)
  • В ноябре можно было уже прочитать про RecyclerView + Staggered Grid вместо AndroidStaggeredGrid.
  • Опять же, во всю уже Material, можно было хотя бы whiteframe из гайдов использовать.
  • Ну и опять про актуальность — метод setNavigationMode(ActionBar.NAVIGATION_MODE_TABS) deprecated. Поэтому лучше было использовать SlidingTabLayout с ViewPager'ом.


Ну и выглядит немного овехрхэдом, можно было попроще проект сделать.
Здравствуйте. Подскажите какой сделать первый шаг в разработке социальной сети на java.
Sign up to leave a comment.

Articles