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

Подскажите. У меня есть репозиторий, который может возвращать либо данные из

бд, либо данные с сервера, либо кешированные данные из этих источников, либо ошибку. Для данных из бд - одна модель, для данных с сервера - другая. По клину(вроде как), интерактор должен получить эти модели, смаппить их и отправить дальше.

Наружу у меня смотрит один метод: условно getData(), клиент не знает о том что там внутри репозитория происходит. И у меня затык, помогите разобраться.

1) Раньше репозиторий не умел тянуть/кешировать данные из бд, и всё было классно. Но сейчас он это умеет, и получается что внутри репозитория данные могут быть представлены двумя моделями: бд и ретрофитовскими. Метод getData() формально может возвращать одну из двух моделей в зависимости от того откуда он стянул данные. У меня есть мапперы, которые перегоняют модель бд и модель retrofit в "итоговую" модель, которая юзается интерактором и передаётся дальше во View, и у меня есть большой соблазн маппить прямо в репозитории, чтобы возвращать в getData() сразу готовую "итоговую" модель, чтобы интерактор не мучался с пониманием того из какого источника данных эта модель,с бд или сети. И вот тут есть нюанс. Маппить в репозитории - вроде как грех, т.к Data слой узнаёт о моделях Domian слоя. С другой стороны, интерактор не должен знать из какого источника получены данные. Но если интерактор хочет маппить, ему придётся понять, какая модель пришла в getData(), что тоже плохо. Как правильно решить этот вопрос?

Код вот репозитория, но там сейчас разруха, я запутался в нулину)
https://github.com/KirstenLy/TestTask/blob/develop/app/src/main/java/com/example/testtask/data/repository/employee/EmployeeRepositoryImpl.kt

3 ответов

13 просмотров

Наоборот, Data слой и должен знать о Domain слое, вы можете спокойно маппить в репозитории

Почему вы не заведете 2 репозитория RepositoryDB и RepositoryNet ?

Александр-Куделько Автор вопроса

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

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

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

Добрый вечер. Есть вопрос, а может и предложение. Был у меня диалог в другой группе о делфи и я задался вопросом: "А нельзя ли в делфи цвет //коментария и {комментария} сде...
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
Карта сайта