последовал рекомендации, и у меня возник новый вопрос.
Вкратце: у меня есть цепочка данных:
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 уровне, чтобы её юзали все слои. Это вроде как могло бы быть хорошим решением, но тогда получается что потенциально в этом классе может быть много внутренних классов, и это тоже вроде как громоздко.
Что мне нужно исправить?
Как минимум я вижу, что у тебя все ещё проблемы с инверсией зависимостей. Т.к. репозиторий работает с RepositoryResult, который находится в data. Но в целом, не понимаю зачем городить 10 разных типов резалтов. Достаточно одного класса в domain. Result или Either.
Обсуждают сегодня