HW_1_Kuzmin #174
HW_1_Kuzmin #174
Conversation
antonkazakov
left a comment
antonkazakov
left a comment
There was a problem hiding this comment.
Привет. Оставил пару комментов во Вьюмодели, добавь еще, пожалуйста, решение через презентер
| } | ||
|
|
||
| fun onInitComplete() { | ||
| val deferredFact = viewModelScope.async { catsService.getCatFact() } |
There was a problem hiding this comment.
А почему ты их вынес за пределы launch?
| private val _result = MutableLiveData<Result>() | ||
| val result: LiveData<Result> get() = _result | ||
|
|
||
| private val exceptionHandler = CoroutineExceptionHandler { _, throwable -> |
There was a problem hiding this comment.
Предлагаю оставить CoroutineExceptionHandler длдя необработанных исключений, а socketexction обработать через try/catch или runCatching
|
|
||
| viewModelScope.launch(exceptionHandler) { | ||
| val catModel = CatModel(deferredFact.await(), deferredImage.await().first()) | ||
| _result.postValue(Result.Success(catModel)) |
There was a problem hiding this comment.
Ты тут уже на главном потоке, можно setValue вызвать
added_presenter_logic|fixed_e_handling|and_stuff
added_presenter_logic|fixed_e_handling|and_stuff
|
Антон, привет. Внес корректировки. |
antonkazakov
left a comment
antonkazakov
left a comment
There was a problem hiding this comment.
Привет. Нужно распараллелить запросы
| fun onInitComplete() { | ||
| viewModelScope.launch(exceptionHandler) { | ||
|
|
||
| val deferredFact = withContext(Dispatchers.IO) { |
There was a problem hiding this comment.
Так, а тут же не поправил, у тебя последовательные запросы
|
Распараллелил запросы. |
| } | ||
| } | ||
|
|
||
| val deferredImage = async(Dispatchers.IO) { |
There was a problem hiding this comment.
Можно не переключать диспатчер для таких библиотек как рум и ретрофит и любых других что из коробки сапортят саспенд функции
2 participants
No description provided.