Здравствуйте. В цитате - мой вопрос, который я ранее задавал. Я

Question

Здравствуйте. В цитате - мой вопрос, который я ранее задавал. Я

последовал рекомендации, и у меня возник новый вопрос.

Вкратце: у меня есть цепочка данных:
Repository -> Interactor -> VM -> View

Repository получает данные из внешних источников. Получение может завершиться с ошибкой. Ранее я спрашивал, как мне эту ошибку покорректнее пробросить от репозитория до View, чтобы оповестить юзера о том что конкретно случилось.
Мне посоветовали sealed класс, я про него почитал, штука классная. Заюзал у себя, всё работает. Только выглядит очень громозко, и я хотел спросить, правильно ли я вообще реализовал проброс?

Сейчас это работает так:
(Репозиторий: https://github.com/KirstenLy/TestTask/blob/develop/app/src/main/java/com/example/testtask/data/repository/EmployeeRepositoryImpl.kt)
У репозитория существует метод getData(), который возвращает данные(интерактору) в виде sealed класса:

sealed class RepositoryResult {
data class Error(val errorMessage: String) : RepositoryResult()

data class Data(val items: List<Employee>) : RepositoryResult()
}

На уровне репозитория всё красиво. Интерактор ловит этот RepositoryResult, и тут начинается шоу. Интерактору тоже нужно пробросить ошибку дальше, если ранее она пришла в RepositoryResult.Error. Поэтому я создаю ещё один sealed класс:

sealed class EmployeeInteractorResult {
data class Error(val error:String):EmployeeInteractorResult()

data class Data(val employees:List<Employee>):EmployeeInteractorResult()
}

И в интеракторе возвращаю наверх уже его. VM ловит этот класс, и через When распихивает данные или даёт команду на отображение ошибки.
Меня нарпягает громоздкость решения. Приложение маленькое, и я пока не могу увидеть, как оно мне в будущем акунется, как бы не пришлось выжигать потом это всё.
У меня была идея создать sealed класс для сущности на Presentation уровне, чтобы её юзали все слои. Это вроде как могло бы быть хорошим решением, но тогда получается что потенциально в этом классе может быть много внутренних классов, и это тоже вроде как громоздко.
Что мне нужно исправить?

#it #programming #russian #software

0

19.07.2019

1 ответов

11 просмотров

Konstantin Dovnar · Accepted Answer

Как минимум я вижу, что у тебя все ещё проблемы с инверсией зависимостей. Т.к. репозиторий работает с RepositoryResult, который находится в data. Но в целом, не понимаю зачем городить 10 разных типов резалтов. Достаточно одного класса в domain. Result или Either.

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

Здравствуйте. В цитате - мой вопрос, который я ранее задавал. Я

1 ответов

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