Size: a a a

Kotlin Community

2019 December 02

BP

Bogdan Panchenko in Kotlin Community
Ilmir
Как человек, писавший парсеры и лексеры (а также постоянно смотрящий в сгенерированный байткод для корутин), я не нашел в коде чего-нибудь нечитаемого. На мой вгляд вполне читаемо и поддерживаемо.
😅 ждем отзыва человека писавшем на асме
источник

ᵛᵉⁿᵈᵉˡⁱᵉᵘ in Kotlin Community
Vadim Morozov
вот финалочка
        startActivity(Intent(Intent.ACTION_VIEW).apply { data = Uri.parse(url) })
val intent = Intent(Intent.ACTION_VIEW)
.let {
it.data = Uri.parse(url)
startActivity(it)
}
источник

ᵛᵉⁿᵈᵉˡⁱᵉᵘ in Kotlin Community
🤷🏻‍♂️ может поздновато
источник

ᵛᵉⁿᵈᵉˡⁱᵉᵘ in Kotlin Community
если в коде нигде больше переменная intent не нужна можно еще упростить

Intent(Intent.ACTION_VIEW)
.let {
it.data = Uri.parse(url)
startActivity(it)
}
источник

PB

Pavel Bayburtyan in Kotlin Community
ᵛᵉⁿᵈᵉˡⁱᵉᵘ
val intent = Intent(Intent.ACTION_VIEW)
.let {
it.data = Uri.parse(url)
startActivity(it)
}
Разве startActivity вернёт что-то? let возвращает результат последнего выражения. В таком случае also надо
источник

ᵛᵉⁿᵈᵉˡⁱᵉᵘ in Kotlin Community
Pavel Bayburtyan
Разве startActivity вернёт что-то? let возвращает результат последнего выражения. В таком случае also надо
return@let
выезжает за вас на стрелки
источник

SB

Sergey Barmin in Kotlin Community
Intent().apply{ data = Uri.parse }.run(::startActivity)
источник

А

Артёмка in Kotlin Community
Sergey Barmin
Intent().apply{ data = Uri.parse }.run(::startActivity)
Похоже на использование фич котлина ради использования фич котлина. Имхо не очень хорошо выглядящий кусок кода. apply, в котором одна строка, ну такое
источник

AN

Alexander Nozik in Kotlin Community
Артёмка
Похоже на использование фич котлина ради использования фич котлина. Имхо не очень хорошо выглядящий кусок кода. apply, в котором одна строка, ну такое
а чем плох apply с одной строкой?
источник

SB

Sergey Barmin in Kotlin Community
Артёмка
Похоже на использование фич котлина ради использования фич котлина. Имхо не очень хорошо выглядящий кусок кода. apply, в котором одна строка, ну такое
Лучше конечно завести переменную, и следующей строке присвоить атрибут
источник

AN

Alexander Nozik in Kotlin Community
я бы конечно написал intent().apply{data = Uri.parse; startActivity()}.
источник

I

Ivansuper in Kotlin Community
Сделайте отдельную экстеншн функцию, которая принимает дату и стартует интент. Будет вам одна строка, но не такая пестристая
источник

SB

Sergey Barmin in Kotlin Community
Alexander Nozik
я бы конечно написал intent().apply{data = Uri.parse; startActivity()}.
Мне просто нравится структурно отделять что в одном блоке мы сеттим данные, и если надо будет добавить еще сеттеров, либо в блоке запуска добавить действий, то не так наляпано будет
источник

AN

Alexander Nozik in Kotlin Community
Sergey Barmin
Мне просто нравится структурно отделять что в одном блоке мы сеттим данные, и если надо будет добавить еще сеттеров, либо в блоке запуска добавить действий, то не так наляпано будет
Ну идейно разницы особой нет, там инлайн
источник

LC

Lungu Cristian in Kotlin Community
.map { it as List<InfoMessages> }

Почему тут unchecked cast если it это List<InfoMessages.Content>

P.S. InfoMessages это sealed class
источник

AN

Alexander Nozik in Kotlin Community
Lungu Cristian
.map { it as List<InfoMessages> }

Почему тут unchecked cast если it это List<InfoMessages.Content>

P.S. InfoMessages это sealed class
Неправильно прочитал. Ну это так устроено. А зачем там вообще каст?
источник

VP

Vladimir Petrakovich in Kotlin Community
А по-моему, должно работать 🤔
источник

LC

Lungu Cristian in Kotlin Community
Vladimir Petrakovich
А по-моему, должно работать 🤔
работает но жалуется
источник

AN

Alexander Nozik in Kotlin Community
Vladimir Petrakovich
А по-моему, должно работать 🤔
Оно где-то работает, где-то нет. Так сложно понять по тексту
источник

VP

Vladimir Petrakovich in Kotlin Community
Lungu Cristian
.map { it as List<InfoMessages> }

Почему тут unchecked cast если it это List<InfoMessages.Content>

P.S. InfoMessages это sealed class
Нужно больше контекста
https://pl.kotl.in/PRA1HbfjO
источник