как считаете, такая каша — нормально, или лучше расписать? update.message?.text ?.let { commandRegex.find(it) } ?.groupValues ?.get(1) ?.let { commands[it.toLowerCase()] } ?.invoke(update)

Чат-бота пишешь? Лямбды в commands с сайд-эффектами? Если да, то лучше разделить парсин комманды в лямбду, от ее вызова.

Чат-бота пишешь? Лямбды в commands с сайд-эффектами? Если да, то лучше разделить парсин комманды в лямбду, от ее вызова.

да я уже в метод завернул, как выше посоветовали

а еще лучше завернуть это все в функцию update.message ?.text ?.let(::parseCommandName) ?.let { commands[it] } ?.invoke(update)

зы, можно написать commands::get

да я уже в метод завернул, как выше посоветовали

Я немножко о другом, можно вынести всю чистую логику в отдельную функцю, даже generic fun <T> findByCommand(update: Update, commands: Map<String, T>): T? = update.message ?.text ?.let { commandRegex.find(it) } ?.groupValues?.get(1)?.toLowerCase() ?.let(commands::get) Потом ты сможешь ее тестировать атомарно, простыми дата-тестами без моков, ведь они зависит только от аругментов.

Я немножко о другом, можно вынести всю чистую логику в отдельную функцю, даже generic fun <T> findByCommand(update: Update, commands: Map<String, T>): T? = update.message ?.text ?.let { commandRegex.find(it) } ?.groupValues?.get(1)?.toLowerCase() ?.let(commands::get) Потом ты сможешь ее тестировать атомарно, простыми дата-тестами без моков, ведь они зависит только от аругментов.

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

Ребят, как в koin сделать временный контекст, что бы View прокинуть в презентеру в конструктор? Все пример на сайте, через метод, уже после создания.

Ребят, как в koin сделать временный контекст, что бы View прокинуть в презентеру в конструктор? Все пример на сайте, через метод, уже после создания.

как выяснилось никак, поэтому, пришлось отказаться от него. koin не di контейнер, а вообще какой-то service locator

Зачем вообще инжектить вью/контекст в презентер? Это нарушение мвп и какой-то бэд пректис

Ребят, как в koin сделать временный контекст, что бы View прокинуть в презентеру в конструктор? Все пример на сайте, через метод, уже после создания.

Смысл презентера в мвп под андроид в том, что он существует дольше вьюхи. Вьюха при изменении конфигурации может пересоздаваться, а презентер - нет. Поэтому презентер при своем создании ничего не должен знать о вьюхе

Зачем вообще инжектить вью/контекст в презентер? Это нарушение мвп и какой-то бэд пректис

Не должен презентер ничего об андроиде знать.

Не должен презентер ничего об андроиде знать.

Если clean architecture

Зачем вообще инжектить вью/контекст в презентер? Это нарушение мвп и какой-то бэд пректис

Под View я имел ввиду интерфейс с методами, а не android.view.View

Для таких случаев у презентера есть методы takeView dropView

Но суть уже описали выше. Презентер живёт дольше вьюхи, значит в конструктор вьюху кидать нельзя

Если clean architecture

Ну да)

И не только clean

Но суть уже описали выше. Презентер живёт дольше вьюхи, значит в конструктор вьюху кидать нельзя

> вьюху кидать нельзя Это бред какой-то. Я создаю retain fragment, и назначаю его View - все его ЖС такой-же как у презентера. Я НЕ храню в презентере состояния, это просто прослойка для связывания.

как выяснилось никак, поэтому, пришлось отказаться от него. koin не di контейнер, а вообще какой-то service locator

В общем, я нашел способ. Через setProperty() > а вообще какой-то service locator Да какая разница, главное что у меня в коде 0 аннотаций, а все зависимости прокидываются через конструктор. Единственное место где виден koin - это создание дерева-зависимостей и презентеров.

И что это будет? Вы инжектите презентер во Вью и хотите ещё и Вью в презентер? Циклическая зависимость

И в чем проблема? У нас GC, а не ARC. Все зависимости идут на интерфейсы.

привет. я собрал проект, и вот какие классы попали в jar файл: https://gist.github.com/marshall-lee/7579fc2c21769cbfc91e3925b260b868 в зависимостях я разумеется явно их не указывал, у меня там только kotlin-stdlib-jdk8. они правда нужны котлиновскому рантайму, или у меня билд плохо настроен?

привет. я собрал проект, и вот какие классы попали в jar файл: https://gist.github.com/marshall-lee/7579fc2c21769cbfc91e3925b260b868 в зависимостях я разумеется явно их не указывал, у меня там только kotlin-stdlib-jdk8. они правда нужны котлиновскому рантайму, или у меня билд плохо настроен?

Это, скорее всего, org.jetbrains:annotations:13.0. Этот пакет находится в зависимостях стдлибы. http://central.maven.org/maven2/org/jetbrains/kotlin/kotlin-stdlib/1.2.21/kotlin-stdlib-1.2.21.pom

Это, скорее всего, org.jetbrains:annotations:13.0. Этот пакет находится в зависимостях стдлибы. http://central.maven.org/maven2/org/jetbrains/kotlin/kotlin-stdlib/1.2.21/kotlin-stdlib-1.2.21.pom

то есть какие-то штуки из intellij нужны котлину в рантайме? Оо

то есть какие-то штуки из intellij нужны котлину в рантайме? Оо

Не «какие-то штуки из intellij», а библиотека с аннотациями.

ок)

вроде бы они собираются убрать эту зависимость, если не ошибаюсь где то писали об этом

Ребята, подскажите, как можно у data class сделать приватными setters у переменных?

Ребята, подскажите, как можно у data class сделать приватными setters у переменных?

никак, скорее всего. Только ручками поле объявлять

может кто-нибудь подсказать что за зверь такой Function Literals with Receiver, и для чего он нужен. прочитал доку на kotlinlang и пока что-то не сильно догоняю полезность сих функций

может кто-нибудь подсказать что за зверь такой Function Literals with Receiver, и для чего он нужен. прочитал доку на kotlinlang и пока что-то не сильно догоняю полезность сих функций

В основном эта фича юзается что бы делать DSL, если ты их не пишешь, можешь пока обойти стороной

типа для декларативного объявления чего-либо?

А не подскажете, что за зверь этот Detekt? И чем он лучше обычного линта, что в студию встроен?

по-хорошему линтер не должен зависеть от IDE, я считаю

ну он вроде и не зависит, те же градлотаски

емнип

просто в чем суть, хочу хоть какой-то линт подрубить. и в растерянности: есть стандартный, если всякие сторонние, что есть что и как быть?

ну сторонние это только detekt мб

и ктлинт вродь

ktlint точно сырой

detekt хз

типа для декларативного объявления чего-либо?

Ну да, тот же Anko на нем построен и позволяет весь UI описывать в коде, еще новый kotlin-gradle-DSL или html DSL из ktor. Если хочешь сделать красивый DSL, как в groovy (только стат. проверяемый), то это для тебя.

про линт: и не знаете, есть в стандартном варнинг про hardcoded strings? или какое-то уже готовое кем-то написанное правило?

@angmarr пока что эта штука взрывает мне мозг =) спасибо ☕️☕️

линт джавовский ругает, котлиновский нет

никак, скорее всего. Только ручками поле объявлять

понятно, спасибо!

В чем сырость ктлинта?

Ребята, подскажите, как можно у data class сделать приватными setters у переменных?

Нежную может уже ответили, после проперти пишешь прайвет в сет. В гуглани

может кто-нибудь подсказать что за зверь такой Function Literals with Receiver, и для чего он нужен. прочитал доку на kotlinlang и пока что-то не сильно догоняю полезность сих функций

все просто, это функция первым аргументом которой является reciver и к нему можно обращаться по this

Нежную может уже ответили, после проперти пишешь прайвет в сет. В гуглани

в конструкторе не выйдет так: data class MyData(var title: String = "")

В чем сырость ктлинта?

Он вроде ведь только кодстайл проверяет

Да

А детект?

А детект?

а детект - это статический анализатор

всякие правила типа "ветка else никогда не будет задействована" и прочие вещи

в конструкторе не выйдет так: data class MyData(var title: String = "")

Пробуй написать его в теле, еще один и может идея подскажет. Я пока с компа зайду

сделать только сеттер приватным не получится, можно только все проперти целиком, а в теле добавить публичный геттер

data class MyData(private var _title: String = "") { val title get() = _title }

сделать только сеттер приватным не получится, можно только все проперти целиком, а в теле добавить публичный геттер

вот оно как. понятно - спасибо!

data class MyData(private var _title: String = "") { val title get() = _title }

может лутше в конструкторе булиный вал а внутри приватный вар?

в этом случае код разве не скмопилируется в ява класс с двумя полями? (если отдельно делать var)

если бы можно было убрать val из констркутора, то да. но в дата классе все параметры констркутра должны быть val/var

в этом случае код разве не скмопилируется в ява класс с двумя полями? (если отдельно делать var)

он и так скомпилится

в моем примере будет только одно поле

val title get() = _title скомпилируется в String getTitle() { return _title; }

да таки

декомпилировал

но смотрится страннл

ну, похожий пример есть в документации https://kotlinlang.org/docs/reference/properties.html#backing-properties

да детект мощнее намного чем ктллинт) спасиб

ну, похожий пример есть в документации https://kotlinlang.org/docs/reference/properties.html#backing-properties

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

да детект мощнее намного чем ктллинт) спасиб

так а все же, зачем он, если есть стандартный?

обычный линт?

он разве поддерживает котлин? вроде только начинает