В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

Kotlin Community

3275 membersпожаловаться на группу
2020 June 05

VS

Vladimir Sitnikov in Kotlin Community
Ayrat Hudaygulov
совсем микро репро:
fun bar(): Int = 1

fun foo(): Unit {
    bar()
}


как сделать это варнингом? пусть даже в линтере Идеи. Идея это пропускает, она умеет только варнинги на неиспользованные значения кидать, но не на результаты функций
javax.annotation.CheckReturnValue
источник
18:49пожаловаться#1

VP

Vladimir Petrakovich in Kotlin Community
Ayrat Hudaygulov
совсем микро репро:
fun bar(): Int = 1

fun foo(): Unit {
    bar()
}


как сделать это варнингом? пусть даже в линтере Идеи. Идея это пропускает, она умеет только варнинги на неиспользованные значения кидать, но не на результаты функций
Проблема в том, что никто не знает, есть ли у функции сайд-эффект, ради которого она и вызывалась. Если ругаться на всё, будет очень много ложных срабатываний.
В идее вроде есть инспекция, которая умеет ругаться на такое для некоторых стандартных функций и для помещенных заданной аннотацией. Но возможно это только для джавы.
источник
18:49пожаловаться#2

AH

Ayrat Hudaygulov in Kotlin Community
Vladimir Sitnikov
javax.annotation.CheckReturnValue
Спасибо, щас попробую
источник
18:50пожаловаться#3

AH

Ayrat Hudaygulov in Kotlin Community
Vladimir Petrakovich
Проблема в том, что никто не знает, есть ли у функции сайд-эффект, ради которого она и вызывалась. Если ругаться на всё, будет очень много ложных срабатываний.
В идее вроде есть инспекция, которая умеет ругаться на такое для некоторых стандартных функций и для помещенных заданной аннотацией. Но возможно это только для джавы.
Ну я-то знаю! Обмажу свои функции дополнительной аннотацией… ну хотя бы так
источник
18:50пожаловаться#4

VP

Vladimir Petrakovich in Kotlin Community
Ayrat Hudaygulov
Ну я-то знаю! Обмажу свои функции дополнительной аннотацией… ну хотя бы так
Надо поискать. А что у вас за баг, к которому приводит неиспользование возвращаемого значения? Обычно такое стреляет только со всякими Deferred.
источник
18:51пожаловаться#5

AH

Ayrat Hudaygulov in Kotlin Community
Vladimir Petrakovich
Надо поискать. А что у вас за баг, к которому приводит неиспользование возвращаемого значения? Обычно такое стреляет только со всякими Deferred.
источник
18:54пожаловаться#6

AH

Ayrat Hudaygulov in Kotlin Community
это ktor
источник
18:56пожаловаться#7

VP

Vladimir Petrakovich in Kotlin Community
Ayrat Hudaygulov
Ну если бы ответ был выделен в отдельную переменную, а не возвращался в трёх местах, было бы сложнее так ошибиться 🙂
источник
18:56пожаловаться#8

AH

Ayrat Hudaygulov in Kotlin Community
Vladimir Petrakovich
Ну если бы ответ был выделен в отдельную переменную, а не возвращался в трёх местах, было бы сложнее так ошибиться 🙂
Ну если бы компилятор мне дал по шапке за неиспользуемое значение if выражения я бы сразу всё понял)
источник
18:57пожаловаться#9

VP

Vladimir Petrakovich in Kotlin Community
Ayrat Hudaygulov
Ну если бы компилятор мне дал по шапке за неиспользуемое значение if выражения я бы сразу всё понял)
К сожалению, компилятор не может знать, нужно ли оно вам. А вот ktor позволяет не возвратить ничего)
источник
18:58пожаловаться#10

AH

Ayrat Hudaygulov in Kotlin Community
Vladimir Petrakovich
К сожалению, компилятор не может знать, нужно ли оно вам. А вот ktor позволяет не возвратить ничего)
поэтому я и искал, может это флагом включается..
источник
18:59пожаловаться#11

VP

Vladimir Petrakovich in Kotlin Community
Мне кажется, источник ошибки скорее в том, что обработчик запроса не обязан возвращать ответ
источник
18:59пожаловаться#12

AH

Ayrat Hudaygulov in Kotlin Community
неявный каст в Unit мне не нужен, я бы предпочёл явный
источник
18:59пожаловаться#13

VP

Vladimir Petrakovich in Kotlin Community
Похоже, что для котлина такой инспекции нет. Видимо, остаются только сторонние линтеры. Впрочем, они обычно гибче.
источник
19:02пожаловаться#14

VP

Vladimir Petrakovich in Kotlin Community
Ayrat Hudaygulov
неявный каст в Unit мне не нужен, я бы предпочёл явный
Мне кажется, слишком часто возвращаемое значение игнорируется, чтобы его не делать.
Взять хотя бы методы у коллекций - add/remove/put.
источник
19:03пожаловаться#15

AH

Ayrat Hudaygulov in Kotlin Community
Vladimir Petrakovich
Мне кажется, слишком часто возвращаемое значение игнорируется, чтобы его не делать.
Взять хотя бы методы у коллекций - add/remove/put.
таки да, но варнинг особенно яростно нужен для каста возвращаемого значения (последнего выражения в скоупе). Вот там неявный каст из T в Unit стреляет в ногу

fun foo(): Unit {
  map.put(…)
  map.put(…)
  map.put(…)
  map.put(…)
  map
}


вот тут ок для промежуточных экспрешнов не иметь такого варнинга но для возвращаемого точно стоит
источник
19:50пожаловаться#16

VP

Vladimir Petrakovich in Kotlin Community
Ayrat Hudaygulov
таки да, но варнинг особенно яростно нужен для каста возвращаемого значения (последнего выражения в скоупе). Вот там неявный каст из T в Unit стреляет в ногу

fun foo(): Unit {
  map.put(…)
  map.put(…)
  map.put(…)
  map.put(…)
  map
}


вот тут ок для промежуточных экспрешнов не иметь такого варнинга но для возвращаемого точно стоит
Ну а если последним выражением будет put(), то привет false-positive
источник
19:51пожаловаться#17

AH

Ayrat Hudaygulov in Kotlin Community
Vladimir Petrakovich
Ну а если последним выражением будет put(), то привет false-positive
ну написать в конце
map.put(…).ignore()

fun <T> T.ignore(): Unit = { .. //cast to unit explicitly }
источник
19:52пожаловаться#18

AH

Ayrat Hudaygulov in Kotlin Community
если такой варнинг и будет, он наверное будет за флагом
источник
19:53пожаловаться#19

VP

Vladimir Petrakovich in Kotlin Community
Вам бы наверное Rust понравился)
источник
19:53пожаловаться#20