Комментарии 14
Резонный вопрос вам уже задали в Google Play — как насчёт контакта? Или вы решили поддерживать только мировых международных лидеров?)
+4
Крутяк, а В контакте есть? )
-1
Кое-какие вопросы/предложения:
1) Зачем в dependencies signpost-core, если вы его не используете? Просто не увидел его в импортах в ваших файлах.
2) Для TW вы используете twitter4j, хотя, по сути, только реализовали авторизацию и публикацию сообщений. Только один twitter4j-core-4.0.1.jar весит около 300кб. Я бы рекомендовал использовать signpost для авторизации (он весит 50кб), а для публикации сообщений обычные http запросы. Понимаю, что сейчас многие уже и не смотрят на размер итогового .apk. Но всё же…
3) Быть может стоит разнести логику по соц. сетям в разные классы/пакеты. Я про классы из com.androidsocialnetworks.lib.impl.*. Взять хотя бы
p.s. хотелось бы ещё VK видеть в списке.
1) Зачем в dependencies signpost-core, если вы его не используете? Просто не увидел его в импортах в ваших файлах.
2) Для TW вы используете twitter4j, хотя, по сути, только реализовали авторизацию и публикацию сообщений. Только один twitter4j-core-4.0.1.jar весит около 300кб. Я бы рекомендовал использовать signpost для авторизации (он весит 50кб), а для публикации сообщений обычные http запросы. Понимаю, что сейчас многие уже и не смотрят на размер итогового .apk. Но всё же…
3) Быть может стоит разнести логику по соц. сетям в разные классы/пакеты. Я про классы из com.androidsocialnetworks.lib.impl.*. Взять хотя бы
TwitterSocialNetwork
. У вас слишком много внутренних классов. Рекомендовал бы отдельный пакет сделать (например com.androidsocialnetworks.lib.impl.tw), а там по разным классам разнести всё это. Потому что, если вы будете наращивать функционал вашей библиотеки, при этом продолжая в одном классе всё писать, то получится слишком толстый контроллер.p.s. хотелось бы ещё VK видеть в списке.
+1
1) Точно не помню, но вроде linkedin-j-android.jar зависит от signpost-core.
2) Пока возможно, но при расширении и наращивании функционала, будет довольно тяжело самому все реализовывать, + нужно будет следить за обновлениями в API twitterа. Я думаю проблему размера нужно решить с помощью proguard и модульности, не всем нужны все 4 соц сети, но руки все никака не дойдут.
3) Я согласен, но думаю пока это делать рано, нужно будет подумать о том как можно это лучше реализовать, сейчас все не очень удобно.
Насчет VK я уже понял, работаю над этим :)
2) Пока возможно, но при расширении и наращивании функционала, будет довольно тяжело самому все реализовывать, + нужно будет следить за обновлениями в API twitterа. Я думаю проблему размера нужно решить с помощью proguard и модульности, не всем нужны все 4 соц сети, но руки все никака не дойдут.
3) Я согласен, но думаю пока это делать рано, нужно будет подумать о том как можно это лучше реализовать, сейчас все не очень удобно.
Насчет VK я уже понял, работаю над этим :)
0
Насколько я могу судить, в Scribe только авторизация реализована.
0
Верно, но и в авторизации есть разница, Scribe не позволяет авторизироваться через Facebook App например, если он установлен. А это намного более удобно для конечного пользователя, чем OAuth и постоянный ввод пароля.
+1
Он также сделан retainedInstance(true) для того, чтобы сетевые запросы нормально переживали изменения lifecycle…
Почему же не использовали Loader'ы?
0
offtop: забавно бы смотрелось сокращение вашей библиотеки до «AsocialNetworks»
0
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
AndroidSocialNetworks — удобная работа с социальными сетями