Телеграмм чат группы kotlin

Всегда. Число с плавающей точкой может выглядеть одинаково в виде строки, но внутри битик другой и равенства нет. Точное сравнение чисел с плавающей точкой - это очень опасная штука.

источник

10:59пожаловаться #7

I

Ivan in Kotlin Community

if (map[0] != null) {
    map[0]!!.add(o)
} else {
    map[0] = mutableListOf(o)
}

А вот не спроста же котлин во второй строчке требует ? или !!?

По идее же, я уже проверяю, что в мапе нулевой элемент не равен нуллу.

Получается, что кусок выше небезопасный?

источник

15:12пожаловаться #8

PE

Pavel Erokhin in Kotlin Community

безопасный

источник

15:12пожаловаться #9

PE

Pavel Erokhin in Kotlin Community

анализатор кода мб не учитывает че там в map[x] передаешь

источник

15:13пожаловаться #10

I

Ivan in Kotlin Community

выглядит это так

источник

15:13пожаловаться #11

PE

Pavel Erokhin in Kotlin Community

!! ставь и забей

источник

15:14пожаловаться #12

I

Ivan in Kotlin Community

Pavel Erokhin

!! ставь и забей

окей, так и сделал, спасибо

источник

15:14пожаловаться #13

КР

Кирилл Романенко... in Kotlin Community

Ivan

if (map[0] != null) {
    map[0]!!.add(o)
} else {
    map[0] = mutableListOf(o)
}

А вот не спроста же котлин во второй строчке требует ? или !!?

По идее же, я уже проверяю, что в мапе нулевой элемент не равен нуллу.

Получается, что кусок выше небезопасный?

Безопасный, но лучше переписать вот так:
map[0]?.add(o) ?: run { map[0] = mutableListOf(o) }

Или же
map[0]?.let { it.add(o) } ?: run { map[0] = mutableListOf(o) }

источник

15:14пожаловаться #14

PE

Pavel Erokhin in Kotlin Community

Кирилл Романенко

Безопасный, но лучше переписать вот так:
map[0]?.add(o) ?: run { map[0] = mutableListOf(o) }

Или же
map[0]?.let { it.add(o) } ?: run { map[0] = mutableListOf(o) }

+1

источник

15:15пожаловаться #15

SB

Sergey Barmin in Kotlin Community

ну оно конечно это не зря пишет, если map это нечто мутабельное и не потокобезопасное

источник

15:15пожаловаться #16

AS

Andrey Stepankov in Kotlin Community

Ivan

if (map[0] != null) {
    map[0]!!.add(o)
} else {
    map[0] = mutableListOf(o)
}

А вот не спроста же котлин во второй строчке требует ? или !!?

По идее же, я уже проверяю, что в мапе нулевой элемент не равен нуллу.

Получается, что кусок выше небезопасный?

Там же некая функция get, которая выдает nullable, откуда знать, что в реализации mutable map, get не вернет другое значение потом.

источник

15:15пожаловаться #17

I

Ivan in Kotlin Community

Кирилл Романенко

Безопасный, но лучше переписать вот так:
map[0]?.add(o) ?: run { map[0] = mutableListOf(o) }

Или же
map[0]?.let { it.add(o) } ?: run { map[0] = mutableListOf(o) }

угу, спасибо, были такие мысли

источник

15:17пожаловаться #18

AS

Andrey Stepankov in Kotlin Community

Ivan

if (map[0] != null) {
    map[0]!!.add(o)
} else {
    map[0] = mutableListOf(o)
}

А вот не спроста же котлин во второй строчке требует ? или !!?

По идее же, я уже проверяю, что в мапе нулевой элемент не равен нуллу.

Получается, что кусок выше небезопасный?

val list = map[0] ?: mutableListOf()
list.add(o)
map[0] = list

Можно по идее вот так

источник

15:18пожаловаться #19

I

Ivan in Kotlin Community

Andrey Stepankov

val list = map[0] ?: mutableListOf()
list.add(o)
map[0] = list

Можно по идее вот так

да думаю, вот такое вынести в extension

map[0]?.add(o) ?: run { map[0] = mutableListOf(o) }

источник

15:19пожаловаться #20