А зачем преобразовывать? Почему не просто тип?

Окей, значит toMap/toList только для защиты от человеческого фактора? Если мы предполагаем, что это не либа и партизанить никто не будет - вызывать это не обязательно?

0

04.02.2021

Mr Fox

Lev Shagalov
Окей, значит toMap/toList только для защиты от чел...

Если каста не боитесь, можете не вызывать. В обратном случае преобразовать придётся.

0

04.02.2021

Alexander Nozik

Alexandr ∨∧‾ Emelyanov
ибо могут скастить и поменять

Я думаю, что закладывваться на любителей играть в рефлексию - так себе идея. Если захотят, все равно сломают

0

04.02.2021

Alexander Nozik

Alexandr ∨∧‾ Emelyanov
ну контекст не известен) может это публичная либа?...

Публичная либа имеет публичное API. Если кто-то ее хачит и ломает API, это проблема того, кто хачит

0

04.02.2021

Alexander Nozik

Lev Shagalov
Окей, значит toMap/toList только для защиты от чел...

да

0

04.02.2021

Alexandr ∨∧‾ Emelyanov

Lev Shagalov
Окей, значит toMap/toList только для защиты от чел...

боюсь toMap и toList вернут те же мутабельные коллекции скастенные в иммутабл

0

04.02.2021

Alexander Nozik

Alexandr ∨∧‾ Emelyanov
боюсь toMap и toList вернут те же мутабельные колл...

У меня тоже такое ощущение. Вообще, protective copy надо применять только когда оно реально нужно.

0

04.02.2021

Alexandr ∨∧‾ Emelyanov

Alexander Nozik
У меня тоже такое ощущение. Вообще, protective cop...

тут согласен, да

0

04.02.2021

Lev Shagalov Автор вопроса

Alexander Nozik
У меня тоже такое ощущение. Вообще, protective cop...

Ну вот а когда оно реально нужно?

0

04.02.2021

Alexander Nozik

Lev Shagalov
Ну вот а когда оно реально нужно?

Например когда ты передаешь именно изменяемое состояние и хочешь его потом менять, но независимым образом. Или когда ты хочешь гарантировать, что результат будет неизменен. Например у тебя есть быстро меняющийся список, а в UI ты хочешь отдать снапшот

0

04.02.2021

Lev Shagalov Автор вопроса

Alexander Nozik
Например когда ты передаешь именно изменяемое сост...

А... Оно ж по ссылке будет меняться... Да...

0

04.02.2021

Alexander Nozik

Lev Shagalov
А... Оно ж по ссылке будет меняться... Да...

Да. Но обычно это не страшно, потому что результат консистентен всегда. Я в последнее время стараюсь где это надо на выход отдавать не коллекцию, а Iterator/sequence/flow, который дает тебе по сути снапшот без оверхеда на защитное копирование. Его можно прочитать только один раз.

0

04.02.2021

Mr Fox

Alexander Nozik
Да. Но обычно это не страшно, потому что результат...

Так flow же отдает постепенно данные, или это не принципиально? Насчёт sequence - хорошая идея

0

04.02.2021

Alexander Nozik

Mr Fox
Так flow же отдает постепенно данные, или это не п...

если это List.toFlow, то почти ничем не отличается от Sequence

0

04.02.2021