Домашнее задание по Flow.Федоров #129
Conversation
Поправил код после лекции Обработка ошибки Заменил лайвдату на стейт Поправил падение приложения
| @@ -4,21 +4,21 @@ import com.google.gson.annotations.SerializedName | |||
|
|
|||
| data class Fact( | |||
| @field:SerializedName("createdAt") | |||
| val createdAt: String, | |||
| val createdAt: String = "", | |||
There was a problem hiding this comment.
Зачем дефолтные значения?
There was a problem hiding this comment.
потому что я туповат и возвращал класс с дефолтными значениями когда делал приведение через as? понял свою ошибку и исправил
| .map { | ||
| when { | ||
| it % 15 == 0 -> { | ||
| emit("$it") |
There was a problem hiding this comment.
Можно вынести из when
There was a problem hiding this comment.
поправил, нашёл более подходящий оператор, для этого задания
| private val _catsFact = MutableStateFlow<Result>( | ||
| Success(catsModel = Fact()) | ||
| ) | ||
| val catsFact = _catsFact.asStateFlow() | ||
|
|
||
| init { | ||
| viewModelScope.launch { | ||
| withContext(Dispatchers.IO) { |
There was a problem hiding this comment.
убрал переключение на dispatchers io , потому что если я правильно понял, то ретрофит под капотом и так ходит на io поток
| ) : ViewModel() { | ||
|
|
||
| private val _catsLiveData = MutableLiveData<Fact>() | ||
| val catsLiveData: LiveData<Fact> = _catsLiveData | ||
| private val _catsFact = MutableStateFlow<Result?>(null) |
There was a problem hiding this comment.
Работать с нуллабл имхо не очень удобно. Добавь еще 1 наследник резалта, например, Initial
|
|
||
| sealed class Result{ | ||
| class Success(val catsModel: Fact) : Result() | ||
| class Error(val error: Throwable?) : Result() |
There was a problem hiding this comment.
А в каком случае ексепшен будет нулл?
Поправил код после лекции
Обработка ошибки
Заменил лайвдату на стейт
Поправил падение приложения