те же все квикфиксы

это не то?

студийный статический анализ из какого-нибудь пре-коммит хука или на CI не запустить

а у некоторых качество кода на поток поставлено

ээ

студийный статический анализ из какого-нибудь пре-коммит хука или на CI не запустить

мне казалось, что идея умеет работатаь в headless режиме и запускать инспекции из CLI на сервере

это просто тот самый линт, что градал делает

это просто тот самый линт, что градал делает

по-моему ты всё-таки что-то путаешь

gradlew lint, он отчеты генерит

и он и со студии тоже умеет

по-моему ты всё-таки что-то путаешь

скорее всего, вот только не понимаю что)

нет никакого gradlew lint

из коробки

я не знаю откуда у тебя этот таск

я не знаю откуда у тебя этот таск

Android же

а, там какой-то свой линтер?

но вообще я не думаю что студия использует этот линт

а, там какой-то свой линтер?

Да, Android Lint. Много чего умеет, но в основном специфичного для Android-проектов

лан, сорь тогда что ворвался в обсуждение тулинга для ондроеда, я в этом ничего не понимаю ?

но вообще я не думаю что студия использует этот линт

По-моему, использует. Вместе со встроенными в идею инспекциями java/kotlin кода.

я на котлине не под ондроед пишу, поэтому не понял сначала о каком линтере вообще речь идёт

прост там не только андроид специфик, но и чисто котлиновские правила

и стандартные идейные окошни

думал, что общее...

Ну так-то Kotlin != Android, так что упоминание его линтера весьма бесполезно в данном контексте. А идея для headless режима требует проект в своём формате, так что многовато возни на CI

Всем привет. Подскажите, как для переменных с lateinit отследить момент, когда эта переменная еще не инициализирована?

::varName.isInitialized

Это вообще нормальная практика? Эта переменная указывает на список объектов, которые получаю с сервера. Когда я их получил, инициализирую. Обращаясь к ней в следующий раз, достаю эти данные из поля класса.

::varName.isInitialized

Спасибо.

а какой профит в таких переменных кроме случаев с DI нпример когда филд енджекшен

почему не сделать нуллабл?

и правда.

ну вот раньше не было этого метода isInitialized

тогда вроде как понятно что упадет в рантайме

а щас еще проверку ставить

почему не сделать нуллабл?

А потом проверят 256 раз

А оно под капотом в nullable и прерващается.

ну а тут не 256 раз проверят?

студийный статический анализ из какого-нибудь пре-коммит хука или на CI не запустить

teamcity разве не умеет?

Бывает иницилизация нужна не сразу, но есть варят делегаты

А оно под капотом в nullable и прерващается.

Нада проверить

Просто если есть уверенность, что оно действительно проинициализируется где-то там в начале жизни объекта, но не совсем сразу, то можно скрасить себе жизнь отсутствием ?. по всему коду, взяв на себя ответственность за KotlinNPE.

Нада проверить

На прошлой неделе тут кто-то проверял вроде.

Просто если есть уверенность, что оно действительно проинициализируется где-то там в начале жизни объекта, но не совсем сразу, то можно скрасить себе жизнь отсутствием ?. по всему коду, взяв на себя ответственность за KotlinNPE.

Сценарий довольно простой: if (::data.isInitialized) { notifier.onDataFetched() } else { // fetch data. when fetched, call onDataFetched() }

ну часто либо лейтинит, либо налы, выбирай что больше не нравится

кстати, а насколько чревато использование isInitialized? это уже рефлексия или нет? надо бы наверное конвертнуть код в джаву и глянуть что там

нет, это сравнение с null в чистом виде

На прошлой неделе тут кто-то проверял вроде.

Я доверяю только себе, байт-коду и разрабам котлина

Да на здоровье, я ж не отговариваю.

какой ты крутой

Я доверяю только себе, байт-коду и разрабам котлина

И зеркалу

Яйца завидуют белой завистью ;)

Я доверяю только себе, байт-коду и разрабам котлина

> и разрабам котлина хи-хи-хи

Кеды?

ну иcходя из анотаций: String? lateinit var k: String , не совсем одно и тоже

Ноу щит, Шерлок, конечно String? и String -- это не одно и то же.

> и разрабам котлина хи-хи-хи

А что "хи-хи-хи", ты ж не над языком работаешь, насколько я понимаю :)

я нет

Ну всё, значит тебе тоже веры нет.

просто пару раз было забавно "хм, да не может же в компиляторе баги быть? это точно я что-то не так делаю" а потом "хм, оказалось что таки в компиляторе"

Ноу щит, Шерлок, конечно String? и String -- это не одно и то же.

я сказал про lateinit String, говорили что он под капотом налл

просто пару раз было забавно "хм, да не может же в компиляторе баги быть? это точно я что-то не так делаю" а потом "хм, оказалось что таки в компиляторе"

Спойлеры!

Хех, бывает. Я, благо, пока не натыкался :)

Спойлеры!

о, а я тебя знаю! :D

о, а я тебя знаю! :D

Да, привет, Кирилл. :)

вот и познакомились

О, сделали чтобы можно было котлин запускать в scratch файлах

я сказал про lateinit String, говорили что он под капотом налл

Ты же только что собственноручно показал проверку на null. @Nullable / @NonNull -- контракты типа, соответствуют String / String?, они ведь не гарантируют, что именно будет в переменных лежать, только обещают.

Ты же только что собственноручно показал проверку на null. @Nullable / @NonNull -- контракты типа, соответствуют String / String?, они ведь не гарантируют, что именно будет в переменных лежать, только обещают.

ну это да, еще isInitialized генерит дополнительный класс

и проверки на нул в гетерах

так-что обычные nullable типы, могут быть лучшим решением

Исключительно для того, чтобы вмест KNPE кидать "property uninitialized". Я уже даже не говорю о том, что бранч предикшн на lateinit-ах должен успешно греться и не ошибаться, нивелируя проверку на налл вообще. А вот касательно дополнительного класса -- не видел, да.

Исключительно для того, чтобы вмест KNPE кидать "property uninitialized". Я уже даже не говорю о том, что бранч предикшн на lateinit-ах должен успешно греться и не ошибаться, нивелируя проверку на налл вообще. А вот касательно дополнительного класса -- не видел, да.

походу генерится два

а нет что-то бавил и появился

вроде hotspot делает хитрую проверку на налл, инструкции для проверки не генерятся. если null то падает в sigsegv, которая обрабатывается как тело if

так что до бранч придикшена дело не дойдет )

Хм, занятно. И в обрабочике сигнала оно потом свой стек разматывает наряду с нативным?

Логично, в целом, эта оптимизация на пользовательский код не должна влиять.

http://hg.openjdk.java.net/jdk8/jdk8/hotspot/file/87ee5ee27509/src/os_cpu/linux_x86/vm/os_linux_x86.cpp#l396

но это нужно оптимизировать, а до оптимизации дойти должно

Даже не хочу спрашивать, что ты этой фразой имел ввиду.

вроде hotspot делает хитрую проверку на налл, инструкции для проверки не генерятся. если null то падает в sigsegv, которая обрабатывается как тело if

исходя из этого , или я не понял?

насколько помню, эта оптимизация включается если в тело проверки попадаем в 0.01% случаев

вернее меньше чем 0.01%

насколько помню, эта оптимизация включается если в тело проверки попадаем в 0.01% случаев

а не много платим за один if? проверка, анализ и изменения в райнтайме

хотя тут хз

Я не понимаю, к чему весь этот разговор. У тебя либо наллабл-поле и ты расставляешь все проверки сам, либо lateinit и за тебя это делает компилятор. Даже для платформ-типа, превращённого в нонналл, будут эти проверки. Какое к этому отношение имеет цена проверки?

Я не понимаю, к чему весь этот разговор. У тебя либо наллабл-поле и ты расставляешь все проверки сам, либо lateinit и за тебя это делает компилятор. Даже для платформ-типа, превращённого в нонналл, будут эти проверки. Какое к этому отношение имеет цена проверки?

не будем оффтопить, просто паньше думал что lateinit неплохое решение, а получилось что даже хуже

Я всё ещё не увидел, чем хуже.

Я всё ещё не увидел, чем хуже.

оверхед, за то что мы саи ручаемся за нот налл

А почему нельзя сделать inline suspend функцию, которая вызывает suspend лямбды?

А почему нельзя сделать inline suspend функцию, которая вызывает suspend лямбды?

инлайнить несколько стейт машин придётся. это сложно

А почему нельзя сделать inline suspend функцию, которая вызывает suspend лямбды?

механика не позволяет

Те нет способа написать свою функцию, которая просто будет вызвать launch(UI), передавая туда лямбду? А то переодически забываю UI передавать ?