Немножко тоже посмотрел. Добавлю и от себя Зачем одновременно держать как

Question

Немножко тоже посмотрел. Добавлю и от себя Зачем одновременно держать как

классы-юзкейсы, так и классы интеракторы?

https://github.com/ImangazalievM/Bubbble/blob/master/app/src/main/java/com/imangazalievm/bubbble/presentation/mvp/presenters/ShotDetailsPresenter.java#L34 и ему подобные. А вы уверены что презентер должен знать откуда ему зависимости приходят? Да и сам стиль внедрения странный - часть зависимостей по человечески в конструктор приходят, а часть он у компонента просит.

Не заметил чем отличаются данные внутри com.imangazalievm.bubbble.data.network.responses и com.imangazalievm.bubbble.domain.models (я, правда, не все смотрел). Если ничем, то зачем заводить дубликаты и мапперы?

Какое назначение у классов внутри com.imangazalievm.bubbble.data.repository.datastores? Так как сейчас они делают то, что должны делать репозитории, которые у вас тоже есть, но почемуто выполняют роль делегатов к датастору.

https://github.com/ImangazalievM/Bubbble/blob/master/app/src/main/java/com/imangazalievm/bubbble/presentation/ui/activities/ShotDetailsActivity.java#L97 и подобные. Вот вы вроде бы как и ДИ используете, а зачем-то зависимости через new сами создаете.

#it #programming #russian #software

0

26.08.2017

4 ответов

21 просмотр

Архитектор

Насчет компонента в конструкторе вы правы, тоже думал об этом. Чуть позже заменю

0

26.08.2017

Архитектор

Датасторы сделал, чтобы не пепегружать репозиторий кучей кода

0

26.08.2017

Архитектор

Респонсы и энтити разделил, как вы прнимаете, для независимости. Да они мало чем отличаются, но это пока. Возможно в будущем объединю их, если пойму, что это оверхед

0

26.08.2017

Архитектор · Accepted Answer

Архитектор

Насчет последнего вопроса про new: мне же нужно как-то передать туда аргументы.

0

26.08.2017

214 похожих чатов

Немножко тоже посмотрел. Добавлю и от себя Зачем одновременно держать как

4 ответов

Похожие вопросы