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

Поділитися
Вставка
  • Опубліковано 14 чер 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  5 місяців тому +1

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

  • @arhitutorials
    @arhitutorials 5 місяців тому +45

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

    • @runapkin
      @runapkin 4 місяці тому

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

    • @umnikya7874
      @umnikya7874 4 місяці тому

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

    • @sweetcapitan5690
      @sweetcapitan5690 4 місяці тому

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

    • @sereekjan
      @sereekjan 2 місяці тому

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

  • @micsha123
    @micsha123 5 місяців тому +11

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

  • @vladsakun6904
    @vladsakun6904 4 місяці тому +1

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

  • @assylkhanassylbek7732
    @assylkhanassylbek7732 5 місяців тому +4

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

    • @AndroidBroadcast
      @AndroidBroadcast  5 місяців тому +4

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

    • @wetfloo
      @wetfloo 5 місяців тому +1

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

  • @artemovrost
    @artemovrost 5 місяців тому +2

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

    • @AndroidBroadcast
      @AndroidBroadcast  5 місяців тому

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

    • @artemovrost
      @artemovrost 5 місяців тому +1

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

    • @ninasemkina8811
      @ninasemkina8811 5 місяців тому +3

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

    • @artemovrost
      @artemovrost 5 місяців тому

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

    • @SergeyBakhtiarov
      @SergeyBakhtiarov 5 місяців тому

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

  • @OpossumbrO
    @OpossumbrO 5 місяців тому +3

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

  • @paulsoja2732
    @paulsoja2732 5 місяців тому +4

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

    • @eugenepopovich2264
      @eugenepopovich2264 5 місяців тому

      Посмотрите в сторону 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 5 місяців тому +1

    Спасибо.

  • @vish5138
    @vish5138 5 місяців тому

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

    • @koshelekteam
      @koshelekteam 5 місяців тому +1

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

  • @sergey30027
    @sergey30027 4 місяці тому

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

  • @dannyregster1120
    @dannyregster1120 5 місяців тому +1

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

  • @alexpro8317
    @alexpro8317 4 місяці тому

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

  • @user-cp5er1hg9l
    @user-cp5er1hg9l 5 місяців тому

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

    • @detribes228
      @detribes228 5 місяців тому +1

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

    • @user-cp5er1hg9l
      @user-cp5er1hg9l 5 місяців тому

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

    • @user-bj6kn4yo6k
      @user-bj6kn4yo6k 5 місяців тому

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

    • @user-qy4xm8wf3b
      @user-qy4xm8wf3b 5 місяців тому

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

    • @ninasemkina8811
      @ninasemkina8811 5 місяців тому

      прошу прощения опечатка 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 5 місяців тому

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

    • @ninasemkina8811
      @ninasemkina8811 4 місяці тому

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

  • @user-fe8hv9sn8e
    @user-fe8hv9sn8e 5 місяців тому

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

    • @AndroidBroadcast
      @AndroidBroadcast  5 місяців тому

      ua-cam.com/video/FGYbjBsCZ7c/v-deo.html

    • @AndroidBroadcast
      @AndroidBroadcast  5 місяців тому

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

    • @user-fe8hv9sn8e
      @user-fe8hv9sn8e 5 місяців тому

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

    • @daniil.pavlenko
      @daniil.pavlenko 5 місяців тому

      ​@@AndroidBroadcast 21:58

  • @wetfloo
    @wetfloo 5 місяців тому

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

    • @ninasemkina8811
      @ninasemkina8811 5 місяців тому

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

    • @sweetcapitan5690
      @sweetcapitan5690 4 місяці тому

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

  • @MrMonyag
    @MrMonyag 5 місяців тому +2

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

    • @ninasemkina8811
      @ninasemkina8811 5 місяців тому +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 5 місяців тому +1

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

  • @yura3882633
    @yura3882633 5 місяців тому +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 скиллы у человека, который разрабатывает приложение для психического здоровья.
      Оценивать чужое произношение вы не можете, потому что у вас оно мягко говоря неидеальное.