HW Sokolova_coroutines #156
Conversation
Sokolik411
force-pushed
the
coroutines_vsokolova
branch
from
2d783f3 to
9d47eed
Compare
Sokolik411
force-pushed
the
coroutines_vsokolova
branch
from
9d47eed to
fcdfbfb
Compare
Sokolik411
force-pushed
the
coroutines_vsokolova
branch
2 times, most recently
from
68f873a to
b074ef4
Compare
Sokolik411
force-pushed
the
coroutines_vsokolova
branch
from
b074ef4 to
796be51
Compare
| private val factService: CatsFactService, | ||
| private val imageService: CatsImageService | ||
| ) : ViewModel() { | ||
| val catModel = MutableLiveData<Result>() |
There was a problem hiding this comment.
Мутабельные стримы: лайвдата, стейтфлоу и тп делают приватными(обычно). Это для того чтобы у тебя в стрим мог эмитить только 1 класс, а не все у кого есть ссылка на CatViewModel.
| is SocketTimeoutException -> { | ||
| catModel.value = Result.Error(Throwable(Resources.getSystem().getString(R.string.error_connection))) | ||
| } | ||
| is CancellationException -> { |
There was a problem hiding this comment.
в CoroutineExceptionHandler этого можно не делать, справедливо только для обычного catch
|
|
||
| private val handler = CoroutineExceptionHandler { _, throwable -> | ||
| when (throwable) { | ||
| is SocketTimeoutException -> { |
There was a problem hiding this comment.
Я бы это перенес в try/catch. Тут оставил только необработанные
|
|
||
| override fun onCleared() { | ||
| super.onCleared() | ||
| viewModelScope.cancel() |
There was a problem hiding this comment.
viewModelScope сам будет очищен в onCleared
| interface CatsFactService { | ||
|
|
||
| @GET("fact") | ||
| suspend fun getCatFact() : Response<Fact> |
There was a problem hiding this comment.
Я бы просто Fact оставил
There was a problem hiding this comment.
d, для картинок сделала аналогично
| override fun onResponse(call: Call<Fact>, response: Response<Fact>) { | ||
| if (response.isSuccessful && response.body() != null) { | ||
| _catsView?.populate(response.body()!!) | ||
| runBlocking(Dispatchers.IO) { |
There was a problem hiding this comment.
Зачем здесь runBlocking? Его вообще кроме как в тестах использовать нигде не нужно
|
|
||
| fun onStop(){ | ||
| catsScope.cancel() |
There was a problem hiding this comment.
А почему это не перенести в detachView, ты же onStop вызываешь в onStop ЖЦ
No description provided.