Ну да
Не особо. Не хватает верхнеуровнего кода
а так?
Servise.let
не смешно
В стейт должно готовое приходить
Код - пиздец каша как по мне
Норм, главное рефенсно-прозрачный 😌
Тут бы линзы помогли, но их надо написать или через arrow генерить (хотя не стоит они же вроде до сих пор через kapt генерят) Ну и в котлин ~2.0 может стат попроще, тк сахарку завезут для вложенным изменения readonly классов А так, я бы посоветовал избавиться от нулабильности, сразу код станет попроще (без ? ?.let и тд)
Не любитель лишний раз создавать переменные?)
ну есть вот такой вариант
Хоть и длиннее, но читабельнее. Легче будет потом разбираться.
Уже лучше. Приведи код в порядок по стилю
Название метода не соответствует поведению. Вариант как был... по аксиоме эскобара так и остался
Используешь экстеншены и при этом ставишь восклицательный знак. Плюс в стейт уже готовое должно приходить
Делай по 10 пробелов в одном методе будет эффективнее
У if ещё можно скобочки фигурные убрать, чтобы строку сэкономить)
И еще почитал названия переменных тихий ужас
Не понял, этот map инвертирует состояние isChecked, что ли? А его напрямую поменять у it нельзя? Тогда зачем через map, а не через forEach? Повсюду копирование, распределение памяти... Нужно напрягаться, чтобы понять. isXxxxxx() должно возвращать Boolean, либо называться по-другому. Пара строчек, а уже столько вопросов.
ну такое
Обсуждают сегодня