EO-746: refactor data saving on ProfileScreen #321
Conversation
| is Either.Success -> { | ||
| user = userResponse.data | ||
| emit( | ||
| book(user = user!!, creatingBookModel = creatingBookModel) |
There was a problem hiding this comment.
давай уберем форсированный not null через let
| val bookings = api | ||
| .getBookingsByUser( | ||
| userId = userResponse.data.id, | ||
| beginDate = localDateTimeToUnix(beginDate)!!, |
There was a problem hiding this comment.
давай избегать форсированного not null
| beginDate = localDateTimeToUnix(beginDate)!!, | ||
| endDate = localDateTimeToUnix(endDate)!! | ||
| ) | ||
| .convert( |
There was a problem hiding this comment.
не нравится вот эта функция. Не понятно во что она конвертит.
вообще код усложнен кучей параметров как будто.
| // TODO add implemetation for error | ||
| } | ||
| is Either.Error -> { | ||
| // TODO add implemetation for error |
There was a problem hiding this comment.
может добавим какую-нибудь ошибку )?
| @@ -47,6 +50,10 @@ class ProfileRepositoryImpl( | |||
| ) | |||
| ) | |||
|
|
|||
| init { | |||
| updateUserFlow.tryEmit(Unit) | |||
| ) | ||
| ) | ||
| override suspend fun refresh(): Either<ErrorWithData<User>, User> { | ||
| val cashedUser = bdSource.getCurrentUserInfo() |
There was a problem hiding this comment.
можно не исправлять, но как будто можно функцию refresh еще на несколько.
}
| emit(dateForEmit) | ||
| ) | ||
| } else { | ||
| val requestResult = api.getUser(cashedUser.id).convert(lastResponse.value) |
There was a problem hiding this comment.
название переменной можно улучшить, слишком абстрактно.
Функция convert не дает понимания во что конвертится, давай явно укажим тип переменной
| } | ||
|
|
||
| bdSource.getCurrentUserInfo().packageEither(requestResult) |
There was a problem hiding this comment.
.packageEither - давай придумаем название получше
| error = user.error.error, | ||
| saveData = user.error.saveData?.formatToUI() | ||
| ) | ||
| fun execute() = flow { |
There was a problem hiding this comment.
эта функция точно нужна в паблик? может сделаем private?
|
|
||
| fun executeInFormat() = execute().map { user -> | ||
| when (user) { | ||
| is Either.Success -> Either.Success(user.data.formatToUI()) |
There was a problem hiding this comment.
нарушено разделение слоев, usecase - это домаин уровень и должен возвращать domain - модель
| } | ||
| } | ||
|
|
||
| private fun User.formatToUI() = |
There was a problem hiding this comment.
User маппится в UI , но при этом модельки называются одинаково либо это одна и та же моделька. Давай исправим
| @@ -67,6 +67,7 @@ internal class ProfileStoreFactory( | |||
|
|
|||
| private fun fetchUserInfo() { | |||
| scope.launch(Dispatchers.IO) { | |||
| getUserUseCase | |||
No description provided.