VP
да, но правда все равно не понятно, как отмену делать. Я бы сказал, что там вообще не должно быть никакого await, там надо вызывать onComplete у deferred, а еще лучше прямо внутри корутины
В смысле как? Вызвать cancel у
job.job.Size: a a a
2020 February 06
AN
В смысле как? Вызвать cancel у
Так если он сразу джойнится после запуска, где вызвать-то?
VP
Так если он сразу джойнится после запуска, где вызвать-то?
Судя по всему, где-то снаружи 🤷♂️
VP
Это же всё в launch завёрнуто
AN
Судя по всему, где-то снаружи 🤷♂️
Так он внаружу не дойдет. Он локально создается и сразу джойнится
VP
Очень похоже на AsyncTask на коленке
AN
Это же всё в launch завёрнуто
отменять-то хотя внутренний джоб, а не топ левел
AN
С топ левелом проблем нет, его отменяешь, все, что внутри - умирает. Но тогда, действительно, нужно withContext и все
AN
Очень похоже на AsyncTask на коленке
в асинктаске та же история, если он не выкидывается наружу, то отменить его не получится
VP
отменять-то хотя внутренний джоб, а не топ левел
Как я понял из контекста,
job - это не локальная переменная. И из-за этого пляски с apply.AN
Как я понял из контекста,
job - это не локальная переменная. И из-за этого пляски с apply.ааа, этот момент пропустил.
VP
в асинктаске та же история, если он не выкидывается наружу, то отменить его не получится
Да там нормальной отменой вообще не пахнет 😄
IS
Очень похоже на AsyncTask на коленке
Ну да похоже, а что плохого, просто сделал колбеки для удобства, там типо чтобы показать прогресс, потом скрыть, а не все в onComplete пихать
AN
Ну ладно, тогда да, делается launch/async, а потом к нему просто join/await без всяких apply
AN
чтоб не мусорить
IS
У меня до этого был launch, в нем Val result = withContext(background) и все
IS
Но решил, что лучше буду отменять конкретно то, что делается в другом контексте
AN
Но решил, что лучше буду отменять конкретно то, что делается в другом контексте
ну нормально. apply блок раздражает, поскольку не нужен и создает проблемы для чтения, в остальном смотрите по вашей обвязке
VP
Ну да похоже, а что плохого, просто сделал колбеки для удобства, там типо чтобы показать прогресс, потом скрыть, а не все в onComplete пихать
Если это удобно, то ничего плохого. Но танцы с коллбеками обычно не выглядят читабельнее, чем простое последовательное выполнение действий.
IS
ну нормально. apply блок раздражает, поскольку не нужен и создает проблемы для чтения, в остальном смотрите по вашей обвязке
Я его уже убрал, ввиду того что job интициализировался в конце