использую такой костыль ```when (...) {...} ?: Uni...

# russian

[Eugen Martynov]

использую такой костыль

when (...) {...} ?: Unit

[ghedeon]

типа exhaustive when нужен? Вопрос как-то сумбурно составлен.
https://stackoverflow.com/a/44383076/858448
https://proandroiddev.com/til-when-is-when-exhaustive-31d69f630a8b

[gildor]

Из коробки ничего, только костыли пока

[gildor]

Есть фича реквест про это
https://youtrack.jetbrains.com/issue/KT-12380

[Eugen Martynov]

о, супер!

[Eugen Martynov]

спасибо

[Dmitry Kandalov]

Не уверен, что я понял вопрос, но возможно вот это пойдёт
val Unit.exhaustive get() = this

[Dmitry Kandalov]

when (result) {
  ...
}.exhaustive

[gildor]

Да, это один из этих самых хаков
В такой имплементации проблема, что when может возвращать не только Unit

[gildor]

Если сделать расширением Any, то засоряет автокомплит всех типов

[gildor]

Я ещё использую иногда toString(), не засоряет автокомплит, но довольно сильно путает, если кто-то этот заказ не видел

[Dmitry Kandalov]

Если не Unit, то when к чему-нибудь присваивается (или возвращается из функции) и будет exhaustive сам по себе

[gildor]

Fixed

[gildor]

Если не Unit, то when к чему-нибудь присваивается

Это же не так, один из бранчей может возвращать значение которое мы не используем, только не будет Unit и что бы это исправить каждый бранч который возвращает не Unit придется каким то образом передать в Unit
Вот к примеру

enum class Operation { Log, Delete }
fun log(id: String): Unit = TODO()
fun delete(id: String): Boolean = TODO()

when (case) {
    Operation.Log -> log("foo")
    Operation.Delete -> delete("bar")
}.exhaustive // Не работает так как типом этого экспрешена будет Any, а не Unit

[gildor]

И что бы это заработало надо передлать на нечто подобное:

when (case) {
    Operation.Log -> log("foo")
    Operation.Delete -> {
        delete("bar")
        Unit
    }
}.exhaustive

[gildor]

не уверен что это лучше

[Dmitry Kandalov]

Если не Unit, то when к чему-нибудь присваивается

Я имел ввиду, что если when не используется для экшенов, то он обычно присваивается и будет требовать всех бранчей. Если используется для экшенов с разными типами, то да, есть проблема с Any, но в этом случае можно спорить, что надо писать код так, чтобы все экшены возвращали Unit или не игнорить результат вызова функции.

[gildor]

Я согласен, но это сильно зависит от функции, к примеру есть куча валидных кейсов когда результат вполне нормально игнорировать, к примеру один из экшенов запускает корутины через launch, то в большинстве случаев Job игнорируется, понятно можно вынести в функцию, но это как раз тот доп код которого хочется избежать