Код ревью неофициального приложение для тостера? [закрыт]

Исправил все баги и недочеты что написали в прошлый раз, добавил нормальное отображение текста и фотографии, теперь ссылка будет ссылкой, фотка фоткой и тд...
Добавил проект этот в github
И переписал его, чтобы он удовлетворял принципам MVP (Хотя не уверен т.к. model имеет ссылку на Presenter)
Прошу подсказать с ревью кода

Ответ

По одному классу CommentAnswerObject. Думаю в остальных так же.
Зачем тут два огромных конструктора, фактически дублирующих друг друга?
public CommentAnswerObject(String urlImage, String name, String dogName, String text, String date, int numberUserComments, int numberLikeComents, boolean like, List toggleObjects) { this.urlImage = urlImage; this.name = name; this.dogName = dogName; this.text = text; this.date = date; this.numberUserComments = numberUserComments; this.numberLikeComents = numberLikeComents; this.like = like; this.toggleObjects = toggleObjects; }
public CommentAnswerObject(String urlImage, String name, String dogName, String text, String date, int numberUserComments, int numberLikeComents, boolean like) { this.urlImage = urlImage; this.name = name; this.dogName = dogName; this.text = text; this.date = date; this.numberUserComments = numberUserComments; this.numberLikeComents = numberLikeComents; this.like = like; }
Второй легко заменяется на
public CommentAnswerObject(String urlImage, String name, String dogName, String text, String date, int numberUserComments, int numberLikeComents, boolean like){ this(urlImage, name, dogName, text, date, numberUserComments, numberLikeComments, like, new ArrayList<>()); }
Такое огромное количество аргументов конструктора в 99% случаев говорит о том, что класс неправильно спроектирован.
Действительно ли в этом объекте нужны все эти поля? Как он используется? Какова его ЕДИНСТВЕННАЯ ОТВЕТСТВЕННОСТЬ?
Вот это экстрактится в класс CommentContent
this.urlImage = urlImage; this.name = name; this.dogName = dogName; //имя собаки? this.text = text;
Для чего используются эти два свойства?
this.numberUserComments = numberUserComments; this.numberLikeComents = numberLikeComents;
По CardObject. Все то-же, но и еще: у вас мешанина методов. Один конструктор сверху, другие разбросаны по тексту. Почитайте Роберта Мартина "Чистый код" раздел "Вертикальное форматирование"
Класс HTTP. По феншую лучше Http. Статика - зло. В данном случае лучше применить паттерн Singletone. Он тоже не сахар, но при умелой готовке вполне себе ничего.
ParsingPage - вообще жесть. Чистый Код в руки и читать до дыр. Методы адски перегружены логикой и вызывают кровотечение из глаз при попытке понять, что происходит. Для начала разбить монстров на кучу мелких методов, выполняющих атомарные операции и понять, что половина из них дублирует друг друга. Вообще данный класс штуки на 4 разбивать надо, на глаз если.
Например класс ParsingPage метод parsFullTag (закроем глаза на статику), первая итерация "причесывания"
private static NameAndTagFullInfoObject full; private static String html;
public static NameAndTagFullInfoObject parsFullTag(String html) { full = new NameAndTagFullInfoObject(); ParsingPage.html = html;
setName(); setUrlPhoto(); if (html.contains("

")) setDopName(); if (html.contains("

")) { setContribution(); } setQuestion(); setAnswer(); setDecisions(); setTextInto();
return full; }
private static void setName() { full.setName(html.split("