Код ревью неофициального приложение для тостера (дубль 2)?

Question

[artshelom]

Исправил все баги и недочеты что написали в прошлый раз, добавил нормальное отображение текста и фотографии, теперь ссылка будет ссылкой, фотка фоткой и тд...

Добавил проект этот в github

И переписал его, чтобы он удовлетворял принципам MVP (Хотя не уверен т.к. model имеет ссылку на Presenter)

Прошу подсказать с ревью кода

Прошу прощения за прошлый код, из за моей невнимательности я код не обновил, и там был не код а кусок "г.вна"

Comments

[IceJOKER]

Пожаловаться - Это вообще не вопрос

[artshelom]

IceJOKER, Если желаете, могу переформулировать вопрос таким образом: "Как можно сделать ревью приложения тостер?" или "Что можно изменить в коде, чтобы он лучше читался и был легче?".

[Денис Загаевский]

Всё плохо. Там даже такие банальные вещи, как строки в коде. Глядя на импортные в Активити хочу убиться. Структура пакетов горно.

[Андрей Мывреник]

Что-то я не вижу где там MVP. Почитай это.

[artshelom]

Денис Загаевский, А не могли бы вы сказать, что стоит прочитать, чтобы не допускать больше таких ошибок, то немного не могу понять ошибку

[Денис Загаевский]

artshelom, нет, гугли.

[artshelom]

Андрей Мывреник, Хорошо прочту.
Просто не нашел способ не ссылаться на Presenter. Вы имеете ввиду сделать наблюдателя?
Чуть позже посмотрю, то что скинули

[artshelom]

Денис Загаевский, Хорошо, а что со строчками в коде??

[Денис Загаевский]

artshelom, то, что они не локализованы.

[artshelom]

Денис Загаевский, Аааааа. Этим занимаюсь

[Андрей Мывреник]

artshelom, я не вижу даже упоминаний слова presenter, о каких ссылках речь я тем более не понимаю. Просмотрев код я вижу только активити, фрагменты, модели (по какой-то причине называющиеся объектами) и пару прочих классов.

[IceJOKER]

Er Code, тематика ресурса немножко другая

[IceJOKER]

artshelom, могу переотправить жалобу - "Слишком общий вопрос".

Задавайте конкретные вопросы.

Прошелся по коду - как структура пакетов/название классов, так и содержимое не в очень хорошем виде, полагаю, что это ваше одно из первых приложений, поэтому рекомендую оттачивать мастерство, смотрите и листайте репозитории других Android приложений с хорошим рейтингом.

Из личного опыта могу сказать одно - читайте статьи, смотрите код других разработчиков , читайте документацию и просто добавляйте что-то, редактируйте и т.д. таким образом мое приложение, которое мне было стыдно показывать кому-либо, теперь один из лучших моих творений, хотя если прямо сейчас открою код, то найду 100500 мест, которые я бы изменил, потому что с тех пор мои навыки улучшились. Просто практикуйтесь

[artshelom]

IceJOKER, Мне стыдно немного это приложение показывать. Но я понимаю, что без вашей помощи, это будет намного дольше и врятли я добьюсь таких же результатов.

Я уже понял что во всех объектах нужно убирать конструкторы, в классе HTTP убирать ститику а ставить Singletone.
И каким то образом менять и разделять parsingPage, но как именно я не особо понимаю

[IceJOKER]

artshelom, ну да, все остальные родились уже профессионалами , чтоб перед нимы стыдились те кто начинают разрабатывать =\
А добьетесь или нет - зависит от вас.

Все зависит от конкретной ситуации, нет единого правила, что нельзя использовать конструкторы и юзать синглтон, поэтому нужно практиковаться.
Данный ресурс - это сервис вопросов/ответов по конкретным вопросам, если нужен ментор - платите или идите на форумы, где есть подобные разделы.

[artshelom]

IceJOKER, Окей, а какие есть ресурсы, чтобы найти ментора??
То гугл никаких, ссылок не выдал.

[artshelom]

Я извиняюсь, по какой-то причине, репозиторий не обновлялся(
Буду разбираться с чем это связано