Coroutines Homework by Diana #253
Conversation
| import retrofit2.http.GET | ||
|
|
||
| interface CatsService { | ||
|
|
||
| @GET("fact") | ||
| fun getCatFact() : Call<Fact> | ||
| suspend fun getCatFact(): Response<Fact> |
There was a problem hiding this comment.
Можно не писать Response. А просто
suspend fun getCatFact(): Response<Fact>
Будет проще потом обрабатывать ответ.
| interface CatsImageService { | ||
|
|
||
| @GET("v1/images/search") | ||
| suspend fun getCatImage(): Response<List<Image>> |
| fun cancel() { | ||
| job.cancel() | ||
| } |
There was a problem hiding this comment.
Думаю, что не самое удачное решение.
У скоупа есть экстеншн метод который называется также
public fun CoroutineScope.cancel(message: String, cause: Throwable? = null): Unit
И здесь как минимум вы путаете с ним.
Также, если вы вызываете
presenterScope.launch то там ведь создается новая джоба. А не которая ваша тут job. И кенселить надо по-хорошему её.
There was a problem hiding this comment.
В общем. CoroutineScope лучше отменять через уже существующий метод CoroutineScope.cancel.
А отменять джобы - это отменять джобы. Там где сделали launch, там и трекайте.
| override fun onStop() { | ||
| if (isFinishing) { | ||
| catsPresenter.detachView() | ||
| // catsPresenter = CatsPresenter(diContainer.catsService, diContainer.catsImageservice, this) |
There was a problem hiding this comment.
Закоментированный код лучше удалять, если он не несет какойго то вопроса.
| if (response.isSuccessful && response.body() != null) { | ||
| _catsView?.populate(response.body()!!) | ||
| presenterScope.coroutineContext.cancelChildren() | ||
| presenterScope.launch { |
There was a problem hiding this comment.
Вот здесь создается дочерняя джоба. Ее и надо трекать.
| override fun onResponse(call: Call<Fact>, response: Response<Fact>) { | ||
| if (response.isSuccessful && response.body() != null) { | ||
| _catsView?.populate(response.body()!!) | ||
| presenterScope.coroutineContext[Job]?.cancel() |
There was a problem hiding this comment.
Так отменять неправильно. Так мы отменяем родительскую джобу. А надо дочернюю.
Правильно было бы написать что-то типа такого:
private var loadJob: Job? = null
fun onInitComplete() {
loadJob?.cancel()
loadJob = presenterScope.launch {
…
}
}
No description provided.