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

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

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

Вкратце: у меня есть цепочка данных:
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 уровне, чтобы её юзали все слои. Это вроде как могло бы быть хорошим решением, но тогда получается что потенциально в этом классе может быть много внутренних классов, и это тоже вроде как громоздко.
Что мне нужно исправить?

1 ответов

9 просмотров

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

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

Обсуждают сегодня

Добрый вечер. Есть вопрос, а может и предложение. Был у меня диалог в другой группе о делфи и я задался вопросом: "А нельзя ли в делфи цвет //коментария и {комментария} сде...
Kraszx
24
Всем привет! Подскажи, пожалуйста, как передать в TComboBox сразу значение и id записи. На Delphi я делал так: ComboBox1.Items.AddObject('Какое-то значение', Pointer(id запис...
Евгений
13
Мдя, прикол, боевая сборка запускается (именно под отладчиком) после F9 примерно полторы минуты (97 секунд если быть точным). Начал копать - проблема детектится сразу - зависа...
Александр (Rouse_) Багель
38
Здравствуйте, вопрос по структурам данных. Были у вас случаи, когда пришлось писать деревья или двунаправленные списки?
/ /
50
Товарищи, кто работа с iphelper? Или может я в самой логике ошибки фигачу, не пойму.... var ifTable : PMIB_IFTABLE; size, corSize: DWORD; Buffer ...
Warfarellen
4
я так понимаю, я так подозреваю, что создание такого плагина для человека, кто умеет писать плагины для делфи потребует минут 5-10 времени. но это мое подозрение. хотелось бы ...
Kraszx
7
Коллеги, добрый вечер. Создаю коллекцию от TFPGMap, ключ - перечисление, значение - целое. Нужно отсортировать коллекцию по значению. Как это можно сделать?
Kirill Filippenok
11
Скажи а ты когда этот канал создавал ты уже дельфи не любил, или это со временем пришло?
Роман Лях (rgreat)
18
Ребята, всем привет. Подскажите, пожалуйста, можно ли как-то через бота понять, что этого бота добавили в группу\канал и выдали ему права администратора?
Artem Stormageddon
9
Привет, такой вопросик появился кажется ли вам что Rust слишком сложный/строгий для высокоуровневого программирования и слишком "безопасный"/строгий для низкоуровневого?
Крокант
10
Карта сайта