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

#android #инспекция_кода

        
             
                
                    
                        
                            Закрыт. Этот вопрос не по теме. Ответы на него в данный
момент не принимаются.
                            
                        
                    
                
                            
                                
                
                        
                            
                        
                    
                        
                            Хотите улучшить этот вопрос? Переформулируйте вопрос,
чтобы он соответствовал тематике «Stack Overflow на русском».
                        
                        Закрыт 2 года назад.
                                                                                
           
                
        
Исправил все баги и недочеты что написали в прошлый раз, добавил нормальное отображение
текста и фотографии, теперь ссылка будет ссылкой, фотка фоткой и тд...

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

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

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

Ответы

Ответ 1

По одному классу 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("