Как улучшить код функциональным подходом || Android Studio КОДИНГ

Поделиться
HTML-код
  • Опубликовано: 11 июн 2024
  • Рефакторинг реальной задачи с помощью функционального подхода для улучшения архитектуры кода
    Эксперт - Нина Сёмкина / nina-semkina-9b595381 , Старший Android-разработчик в Кошельке. 10+ лет опыта в разработке.
    🔗 Код из видео github.com/NinaTverdohleb/Fun...
    🔗 Каналы "Android Broadcast" taplink.cc/android_broadcast
    ✉️ Написать Кирилл kirill@androidbroadcast.dev
    🔗 Boosty abdev.by/s0Z
    🔗 Android Broadcast Плюс abdev.by/q2A
    #кодинг #androiddev #android #программирование #kotlin #КириллРозов #андроид #androidbroadcast
    00:00 Вступление
    00:35 Про ограничения в разработке
    03:47 Базовая теория функциональной разработки
    11:53 Android Broadcast Plus
    13:30 Кодинг
    51:34 Заключение
    51:43 Спасибо Патронам!

Комментарии • 69

  • @AndroidBroadcast
    @AndroidBroadcast  4 месяца назад +1

    🔗 Boosty abdev.by/s0Z
    🔗 Android Broadcast Плюс abdev.by/q2A
    🔗 Каналы "Android Broadcast" taplink.cc/android_broadcast
    ✉ Написать Кирилл kirill@androidbroadcast.dev

  • @arhitutorials
    @arhitutorials 4 месяца назад +45

    Для тех кому лень смотреть, видео про то, что надо возвращать Either из методов внутри которых может выбрасываться исключение.

    • @runapkin
      @runapkin 4 месяца назад

      Братан, ты из лиги лени?

    • @umnikya7874
      @umnikya7874 4 месяца назад

      То что ты спец многие новички знают, и это видео для новичков будет полезно.

    • @sweetcapitan5690
      @sweetcapitan5690 4 месяца назад

      ​@@umnikya7874нет не будет.

    • @sereekjan
      @sereekjan 2 месяца назад

      Также она подсветила плюсы использования референсов на методы, это тоже из функционального программирования

  • @micsha123
    @micsha123 4 месяца назад +11

    Работал с Ниной еще в Яндекс Деньгах - профессионал своего дела и очень приятная коллега!
    Я знаю, что уже тогда она изучала вопрос применения такого подхода применительно к android-разработке. Дико приятно видеть, что часть этого опыта она решила выложить в формате такого доклада - уверен, для большого числа людей если и не даст новых знаний, то освежит имеющиеся. Очень круто!

  • @vladsakun6904
    @vladsakun6904 4 месяца назад +1

    Нина, спасибо огромное за выпуск!

  • @assylkhanassylbek7732
    @assylkhanassylbek7732 4 месяца назад +4

    Спасибо большое за выпуск, в очередной раз понял что тупой и учиться меня еще лет 10)))

    • @AndroidBroadcast
      @AndroidBroadcast  4 месяца назад +4

      Учиться есть всем специалистам, независимо от опыта. Не стоит так самокритично, особенно в начале. Хвалите себя чтобы не терять мотивацию

    • @wetfloo
      @wetfloo 4 месяца назад +1

      стоит за пределы андроида выйти, если не пробовали, мне rust и c помогли, вас может заинтересовать что-то ещё - много нового узнаете и кругозор в целом будет более широкий

  • @artemovrost
    @artemovrost 4 месяца назад +2

    Спасибо за видео. Интересно ваше мнение о том, как организовать cancellation у корутин в таком кейсе? Ведь CancellationException мы не должны перехватывать.

    • @AndroidBroadcast
      @AndroidBroadcast  4 месяца назад

      Ну так оно просто игнорируется в преобразовании в Result и выбрасывается дальше

    • @artemovrost
      @artemovrost 4 месяца назад +1

      @@AndroidBroadcast а не будет ли это нарушать тот функциональный концепт за существование которого боролись на протяжении всего видео? Ведь exception, по словам Нины, это сайдэффект.

    • @ninasemkina8811
      @ninasemkina8811 4 месяца назад +3

      Вы правы в том что грубо говоря из-за CancellationException у нас не чистая ситуация, и я делаю вид что как бы все хорошо))) Тут приходится идти на копромис добиваясь честно говоря чистых функций в контексте логики проекта а не логики работы приложеня в целом. Ох уж эти компромисные компромисы только в теории все красиво практика все портит))

    • @artemovrost
      @artemovrost 4 месяца назад

      @@ninasemkina8811обычное дело) демократия, чистая архитектура, функциональный подход - все хорошо лишь на бумаге)

    • @SergeyBakhtiarov
      @SergeyBakhtiarov 4 месяца назад

      ​@@AndroidBroadcast Внутри у runCatching try-catch, который ловит CancellationException и не даёт родительской корутине его обработать. Это нарушает cancellation flow.

  • @OpossumbrO
    @OpossumbrO 4 месяца назад +3

    Эксперт, легенда и просто хороший человек. Наконец-то я понял как улучшить архитектуру своего кода

  • @paulsoja2732
    @paulsoja2732 4 месяца назад +4

    сам юзаю Result уже не один год, удобно и нравится он мне. Но вот всё больше сталкиваюсь с тем, что гугловые библиотеки (типа пейджинга или биллинга) возвращают Flow. А оборачивать Flow с Result - выглядит уже монструозно. Перешел сейчас полностью на Flow, в принципе там тоже есть результат и эксепшин, но всё же немного это не то. Есть ли идеи как подружить это всё дело?

    • @eugenepopovich2264
      @eugenepopovich2264 4 месяца назад

      Посмотрите в сторону cqrs. Идея, что метод, который возвращает флоу только даёт доступ к данным но никак их не модифицирует. Если данные нужно изменить, то это делается другими методами. А вот те методов уже могут вернуть Result но без самих данных. За данными по прежнему нужно идти к flow.

    • @ninasemkina8811
      @ninasemkina8811 4 месяца назад +4

      Спасибо большое вам за комментарий, заставил меня очень призадуматься)) Работа с потоком данных, не кажется мне не естественной, для подхода, в котором мы говорим, что операция над данными - чистая функция. Ведь в целом Flow - это наш источник данных и он дает нам возможность применить операции (которые уже и будут чистыми) над этими данными. Тут есть два стула) если мы пишем код так что наши преобразования над потоком данных или генерация его, может вызывать исключения, вот тогда нам нужно оборачивать collect в try catch и там уже решать, что делать при ошибке с источником данных дальше. Или мы можем не вызывать исключений). По моему личному мнению в первом подходе как-то несимпатично все выглядит. Мне как разработчику хочется иметь стабильный источник данных, который в зависимости от ситуации либо возвращал мне только результаты и не валился почем зря, либо при возникновении ошибки кидал мне в поток данные которые для меня означают ошибку. Таким образом совмещая то, что я говорила в видео, в логике я хочу Flow к примеру резалтов(оставляем возможность возвращать флоу состояний как на ui, это если мы хотим устроить себе прям стейт машину для логики забыть про эксепшены и оперировать везде только состояниями), а на ui я хочу Flow состояний (State). И я все еще держу требование быть чистыми мои маленькие функции преобразований данных и работаю с потоками данных

    • @paulsoja2732
      @paulsoja2732 4 месяца назад

      @@ninasemkina8811 collect не обязательно оборачивать в try..catch. У Flow когда в emit будет ошибка, то она ловится где-то во вьюмодели в блоке catch (главное не забыть его вызвать). Да, у Flow есть все эти операторы - map, flatMap и еще много чего. Возможно про Result стоит уже и забыть. В общем я пока на переходной стадии - от Result не хочется отказываться, но видимо прийдется

  • @Filinhat
    @Filinhat 4 месяца назад +1

    Спасибо.

  • @vish5138
    @vish5138 4 месяца назад

    Пушка, бомба, где такие умницы обитают?

    • @koshelekteam
      @koshelekteam 4 месяца назад +1

      У нас в команде 🤩🙃

  • @sergey30027
    @sergey30027 4 месяца назад

    Жду продолжение

  • @dannyregster1120
    @dannyregster1120 4 месяца назад +1

    господа эксперты вам не кажется, что Throwable плохая задача возвращать, тк в наслениках у Throwable как Exception так и Error

  • @alexpro8317
    @alexpro8317 4 месяца назад

    Хороший доклад, сам уже юзаю Котлин Result двольно долго на своих проектах. Очень не хватает в нем стандартной flatMap😢
    Из небольшого минуса отмечу только комбинацию его работы с WorkManager, ибо там работа должна возвращать свой Result, и это немного режет глаз. Но это больше претензия к самому WorkManager-y)

  • @user-cp5er1hg9l
    @user-cp5er1hg9l 4 месяца назад

    кто-нибудь может объяснить код из начала видео fun g(s: String): Int = {x.length}, откуда х? где s? и почему тип Int?

    • @detribes228
      @detribes228 4 месяца назад +1

      Опечатка x на s поменяй и получишь функциональную ф-ию по получению длины строки

    • @user-cp5er1hg9l
      @user-cp5er1hg9l 4 месяца назад

      @@detribes228 ну то есть надо так? fun g(s: String): () -> Int = {s.length}

    • @user-bj6kn4yo6k
      @user-bj6kn4yo6k 4 месяца назад

      @@user-cp5er1hg9l Правильно fun g(s: String): Int = { s.length }. Есть книжка "Paul Chiusano Functional Programming in Scala", там очень подробно объяснено что такое чистые функции и сайд эффекты.

    • @user-qy4xm8wf3b
      @user-qy4xm8wf3b 4 месяца назад

      Скорее всего всё же
      fun g(s: String): Int = s.length
      Иначе val h = f(g(s)) не скомпилится

    • @ninasemkina8811
      @ninasemkina8811 4 месяца назад

      прошу прощения опечатка fun g(s: String): Int = {x.length} => fun g(s: String): Int = {s.length} ну а тип инт потому что length у CharSequence это Int

  • @Hash551
    @Hash551 4 месяца назад

    Нина, как андроид разработчик что вы можете сказать, по поводу постоянных "вредных обновлений ПО"? Под этим я подразумеваю ситуацию когда программа или оболочка андроида обновляется, и начинает тормозить на мощном железе. Хотя ничего нового с обновлением привнесено не было. Такая ситуация достоверно зафиксирована на ios и андроид. К примеру игра fruit ninja без каких либо изменений графики безбожно тормозит на ipad 3, но если путем определённых манипуляций поставить старую версию, и о чудо всё прекрасно работает! И так во всём ПО начиная от ютуба и дубльгиса, и заканчивая программами онлайн магазинов.

    • @ninasemkina8811
      @ninasemkina8811 4 месяца назад +2

      Не могу поспорить с тем что можно наблюдать ухудшения работы после обновлений, так как порой технологии которые в теории должны все ускорить наоборот тормозят, далеко ходить не надо просто вспоминая ситуацию с dagger и ksp. Но я не склонна верить в заговор о том что производители ПО намеренно ухудшают работу на старых телефонах, чтобы люди напокупали новых. Приложения имеют баги, могут быть неоптимизированны под новые версии ОС, обновления ОС могут иметь баги на конкретном железе, это возможно, делать из этого выводы о вредном намерение я считаю несколько опрометчивым

  • @user-sg1ip3du6q
    @user-sg1ip3du6q 4 месяца назад

    Result конечно хорош, но у него есть 1 большой, на мой взгляд минус - ошибки возвращаются только типа Throwable. а очень хочется, особенно для сетевых запросов, возвращать уже свой собственный тип ошибок.
    а видео полезное и интересное!)

    • @AndroidBroadcast
      @AndroidBroadcast  4 месяца назад

      Так можно улучшить введением Generic

    • @user-sg1ip3du6q
      @user-sg1ip3du6q 4 месяца назад

      @@AndroidBroadcast можно и свой собственный selead class написать. но придется либо дописывать красивые map, fold и тд, либо без них обходится. я очень надеюсь, что это доработается. раньше ведь Result вообще из методов нельзя было возвращать, а теперь на них архитектурную логику строят.
      хотя мне все чаще попадаются старички, воевавшие на RxJava, которые на Result смотрят сомнительно и любящие отлавливать ошибки во ViewModel.
      надеюсь, это видео даст толчок в массы и в сторону Result и его кастомных вариантов

  • @user-zu6fl2ug5o
    @user-zu6fl2ug5o 4 месяца назад

    За "pusser" лайк)

  • @umnikya7874
    @umnikya7874 4 месяца назад +2

    чётко, без воды

  • @Igor-yh4gl
    @Igor-yh4gl 4 месяца назад

    А почему гитхаб такой пустой 🤔 Не основной что-ли

    • @ninasemkina8811
      @ninasemkina8811 4 месяца назад

      Да это не основной))) это грустная для меня история как гитхаб заблочил мне аккаунт личный из за санкций для компании)))

  • @user-fe8hv9sn8e
    @user-fe8hv9sn8e 4 месяца назад

    Вот бы ещё ссылку на видео с result в retrofit

    • @AndroidBroadcast
      @AndroidBroadcast  4 месяца назад

      ruclips.net/video/FGYbjBsCZ7c/видео.html

    • @AndroidBroadcast
      @AndroidBroadcast  4 месяца назад

      Подскажи таймкод где оно там, пожалуйста

    • @user-fe8hv9sn8e
      @user-fe8hv9sn8e 4 месяца назад

      Походу везде. Или ошибка или пилите через dynamic proxy

    • @daniil.pavlenko
      @daniil.pavlenko 4 месяца назад

      ​@@AndroidBroadcast 21:58

  • @wetfloo
    @wetfloo 4 месяца назад

    kotlin-result более подходящая, имхо, чем то что предполагает стандартная библиотека
    Но вообще после того как я толкала в разные проекты проекты, сейчас не знаю, как к этому относиться. Тут проблема гвоздя и молотка, и если знаешь про монады и чистые функции, хочется везде их толкать, а kotlin всё же на исключения больше настроен. Стандартная библиотека не использует стандартный же Result, о чём тут говорить
    Наверное, если язык изначально не был спроектирован с парадигмой errors as values, её навешивать не очень удачная идея

    • @ninasemkina8811
      @ninasemkina8811 4 месяца назад

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

    • @sweetcapitan5690
      @sweetcapitan5690 4 месяца назад

      Result по факту костыль, появившийся для смягчения жжения в садалищной зоне при обработке ошибок корутин.

  • @MrMonyag
    @MrMonyag 4 месяца назад +2

    еще некоторые синьористые синьоры уверены, что функциональное программирование - это длиннющие цепочки трансформаций. в результате получается нечто неподдерживаемое: три-четыре экрана из map flatmap foreach и прочего, в которых всё, от загрузки данных из апи и их трансформаций до... и нет, я не шучу про три-четыре экрана, я прямо сейчас в такое смотрю и подумываю об увольнении.

    • @ninasemkina8811
      @ninasemkina8811 4 месяца назад +2

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

    • @sylvanbird8788
      @sylvanbird8788 4 месяца назад

      Дезертировать нельзя рефакторить. Каждый сам ставит запятую там, где считает нужным.

    • @ninasemkina8811
      @ninasemkina8811 4 месяца назад

      @@sylvanbird8788 )))) остро

  • @user-jf3iu8vi7k
    @user-jf3iu8vi7k 4 месяца назад +4

    Блин она так красива,

    • @AndroidBroadcast
      @AndroidBroadcast  4 месяца назад +1

      Парни всегда так искренне пишут про девушек, а вот девушки так парням в IT тематиках не пишут 😁

  • @bettereverytime435
    @bettereverytime435 4 месяца назад

    Сложна

  • @eugenepopovich2264
    @eugenepopovich2264 4 месяца назад +1

    Весь доклад предвкушал чистые функции, а по итогу чистый только маппер. Все остальные сущности имеют сайд эффекты

  • @yura3882633
    @yura3882633 4 месяца назад +2

    Над произношением надо поработать, не в обиду.
    Пока нету текста, понять о чем речь очень сложно.
    И очень много времени потрачено на ручной рефакторинг, есть же сочетания клавиш студии для рефакторинга

    • @ninasemkina8811
      @ninasemkina8811 4 месяца назад +8

      Хотела оставить возможность людям ускорять видео на столько на сколько им хочется) и хоткеи могут внести неясность на видео того что происходит

    • @user-um4sg6tu9i
      @user-um4sg6tu9i 4 месяца назад +4

      А вам над слухом и тактичностью.

    • @yura3882633
      @yura3882633 4 месяца назад +3

      ​@@user-um4sg6tu9iвот так вумен момент.
      Разве я что- то не по делу сказал?
      Для хоткеев есть презентационный режим, показывает нажатия тостами. Не пришлось бы размазывать рефакторинг на 50 минут

    • @MrNickolasFlamel
      @MrNickolasFlamel 4 месяца назад

      @@yura3882633 «вуман момент» - как маркер обнуления любой ценности всего пойнта 😂

  • @acras11
    @acras11 4 месяца назад

    Все было хорошо, пока я не услышал "саксефул". Ну блин. Такой серьезный разговор и это

  • @Nek_12
    @Nek_12 4 месяца назад

    Очень плохо - мое мнение.
    Теория объяснена плохо, галопом по европом и в видео абсолютно никак не применяется.
    Определения неправильные даны, видно непонимание самих концепций которые же и объясняются
    Каррирование называется currying правильно
    Код вначале видоса не компилирующийся вообще - перепутаны возвращаемые типы, параметры, использования, переменные. Еще и пример на f, g, h, что путает новичков (по моему скромному мнению)
    Советы даны плохие, Result мало кто уже использует, или kotlin-result, или ApiResult, или Arrow. Конечно готового метода flatmap нет - надо нормальные фреймворки использовать, где он есть :)
    Throwable нельзя ловить
    Either непонятно зачем был введен с учетом того, что про Result каждый второй джун знает, и назвать это "функциональным программированием" сложно, а речи про него вначале вообще не было
    Плохой английский очень, слушать тяжело с перепутанными понятиями и произношением
    Раздражает медленная скорость печати вручную, без горячих клавиш, смотрел на 2.0
    Судя по использованию интерактора и его неправильному определению, сначала нужно выучить клин арч
    29:32 Мы не создаем новую функцию - потому что функция инлайн...
    Ни от одного сайд-эффекта не избавились, все функции как были, так и остались "грязными", только не кидаются и не написаны наигранно плохо были...
    41:47 - Как вводим функциональное программирование - удаляем весь неудобный код, потому что он "плохой".
    44:41 - надо начать с правильного разделения на функции код и названий переменных, а "добавить в меня все друзья с имя" как-то не очень выглядит.
    45:23 - давайте возвращать результат от инта, потому что мы даже не проверим ошибки которые произошли и просто выкинем в этом и всем предыдущем коде всю обработку ошибок. accumulate, partial существуют?
    Эксепшны никак не относятся к функциональщине...

    • @user-um4sg6tu9i
      @user-um4sg6tu9i 4 месяца назад +1

      Очень плохо - это такие soft скиллы у человека, который разрабатывает приложение для психического здоровья.
      Оценивать чужое произношение вы не можете, потому что у вас оно мягко говоря неидеальное.