Best Practices for useEffect by React Documentation

Поділитися
Вставка
  • Опубліковано 4 сер 2024
  • Как вы знаете код можно написать по разному и в обоих случаях интерфейс будет работать одинаково. В данном видео мы рассмотрим разное написание одной из самых бытовых задач и попробуем какой из вариантов написания более удачный
    Поддержать Айти Синяка можно здесь:
    UA-cam: / @it-sin9k
    boosty: boosty.to/sin9k
    Patreon: / itsin9k
    00:00 Анонс темы
    00:49 Пишем компонент Books
    02:45 Оцениваем читабельность кода
    03:48 Рациональность использования ресурсов
    05:00 Попробуем написать Books иначе
    05:39 Проводим анализ такого компонента
    06:32 Бонус
    08:20 Подписывайтесь!
    Подписаться на канал: / @it-sin9k
    Twitter: / it_sin9k
    -------------------------
    Данный канал создан для инициирования бесед на различные темы IT сферы (социальные / технические), а также для тех кому короткая видео выжимка статьи, выступления на конференции или же просто личных мыслей, являются более удобным форматом

КОМЕНТАРІ • 179

  • @user-zk6tw7rj1t
    @user-zk6tw7rj1t 3 роки тому +41

    мало мало очень мало такого хорошего контента ))) хочу больше React и с самого начала )))

    • @it-sin9k
      @it-sin9k  3 роки тому +3

      мы работаем над этим) как раз монтаж следующего выпуска активно идет)

  • @user-uu4gp9et2s
    @user-uu4gp9et2s 3 роки тому +17

    Мужииик, я просто обалдел, как ты крут и как полезен твой контент. Спасибо! Моя просьба продолжать бомбить про реакт и околореакт.

    • @it-sin9k
      @it-sin9k  3 роки тому +2

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

  • @user-yo5vl6ip2s
    @user-yo5vl6ip2s Рік тому +1

    Хорошая работа! Но есть пару заметок ( Вижу, что видео старое, может мои заметки показаны в след. видео)
    1. Почему бы нам не вынести функцию за пределы компонента? Зачастую методы для работы с АПИ продакшене выносятся в отдельные файлы.
    2. Лучше всего деструктуризацию проводить в теле функции. Понимаю, что в наглядном примере это смотреться адекватно и экономит нам строку. Но новичкам такой подход нравится и начинаю деструкторизировать по 10-15 свойств ( в шапке функции), этим самым усложняя читабельность кода :)
    const loadBooks = (categoryId) => {
    return api.getBooks({categoryId});
    }
    const Books = props => {
    const {categoryId} = props;
    const [books, setBooks] = useState([]);
    useEffect(()=> {
    loadBooks(categoryId).then(response => {
    setBooks(response.books);
    })
    }, [categoryId]);
    return ...
    }

  • @igormalykhin5528
    @igormalykhin5528 8 місяців тому +1

    Супер контент, продолжайте пожалуйста

    • @it-sin9k
      @it-sin9k  8 місяців тому +1

      Спасибо! скоро выйдет новый ролик) по секрету тема будет React Server Actions)

  • @sergeyglebko6715
    @sergeyglebko6715 3 роки тому +11

    Функция loadBooks вообще может быть вынесена из компонента. Она может принимать categoryId и возвращать response.books. А в useEffect ее вызываем (await loadBooks) и просто делаем setState() для результата. Тогда сама функция будет создана 1 раз.

    • @it-sin9k
      @it-sin9k  3 роки тому +4

      Как я в начале видео и говорил, способов написать один компонент очень много. Поэтому основная идея была, покрыть 1 рабочий кейс. Чтобы показать что можно делать и так тоже. Судя по комментариям, некоторым это пригодилось. Да я и сам писал сразу неправильно)

    • @dalnogirl
      @dalnogirl 2 роки тому +6

      Чтобы вызвать функции с помощью await, придется пометить функцию эффект как асинхронную, что автоматически приведет к возвращению промиса из нее, однако же реакт ожидает, что функция эффект будет возвращать другую (clean-up) функцию. Как следствие - ошибка

    • @kostyakozlov5289
      @kostyakozlov5289 2 роки тому

      действительно, хорошее примечание!

  • @illyakyrylov
    @illyakyrylov 3 роки тому +1

    Спасибо за видео.
    Как всегда познавательно.

  • @igordzyabenko3712
    @igordzyabenko3712 3 роки тому +2

    Спасибо, как всегда отличный материал и подача)

    • @it-sin9k
      @it-sin9k  3 роки тому +1

      Спасибо :)

  • @serhiirumiantsev7736
    @serhiirumiantsev7736 3 роки тому +15

    лайк и комент для алгоритма

    • @it-sin9k
      @it-sin9k  3 роки тому

      Спасибо!) Алгоритм работай!

  • @marynakosiv3750
    @marynakosiv3750 3 роки тому +1

    офигеть какой контент! такого еще не видела
    ++ за пояснения с графом

  • @andreyshevchenko4602
    @andreyshevchenko4602 3 роки тому +6

    Супер, спасибо за видео. Интересно наблюдать как можно по разному ипользовать каждодневные хуки.

    • @it-sin9k
      @it-sin9k  3 роки тому

      Рады быт полезными :)

  • @denydavydov9290
    @denydavydov9290 3 роки тому +1

    Спасибо, полезный материал!
    Первый варинт вполне удобен, если что то тоже пользуется этой же async функцией в рамкам компонента (или его дети).
    И второй удобен. Все как всегда зависит от задачи)

    • @it-sin9k
      @it-sin9k  3 роки тому +1

      Все верно :)
      главное оценить по 2-ум критериям, как выгоднее

  • @mikhailstepanischev8316
    @mikhailstepanischev8316 3 роки тому +2

    Очень познавательно,спасибо ! useEffect by React Documentation

  • @illiakyselov2232
    @illiakyselov2232 3 роки тому

    Спасибо, очень полезные видео!

    • @it-sin9k
      @it-sin9k  3 роки тому +1

      оставайтесь с нами) следующие видео будет крайне новым для русского сегмента)

  • @-X-Ray-
    @-X-Ray- 3 роки тому

    Спасибо огромное)

  • @solexofficial9597
    @solexofficial9597 2 роки тому +2

    Божественно, спасибо!) несправедливое количество просмотров и лайков, алгоритмы давайте шевелитесь

  • @diegoavenger7702
    @diegoavenger7702 3 роки тому

    Хорошое объяснение, лайк )

  • @379sanek
    @379sanek 3 роки тому +2

    Спасибо за труды, жаль мало подписчиков в отличие от всякого шлака. Подача и темы топовые!

    • @it-sin9k
      @it-sin9k  3 роки тому

      Это еще мы знатно выросли)
      первые 500 подписчиков вообще тяжело шли))
      Поэтому нам будет очень приятно, если вы расскажете о нас своим коллегам!

    • @379sanek
      @379sanek 3 роки тому

      @@it-sin9k да вроде стараемся, в дискорд rolling scopes вас кидал, вообщем думаю будет геометрическая прогрессия. Только не бросайте канал, если надо донаты, буду донатить, очень верю в вас

    • @it-sin9k
      @it-sin9k  3 роки тому +1

      Спасибо за поддержку!)
      Думаю в ближайшее время появится кнопка спонсировать канал)
      Бросать не планируем) обдумываем только варианты монетизации, думаю из пары экспериментов, что-нибудь получится)

  • @kirillpavlovskii8342
    @kirillpavlovskii8342 2 роки тому

    хорошо

  • @user-lv4mn2ey6p
    @user-lv4mn2ey6p 3 роки тому

    Спасибо большое за такой качественны и главное полезный контент!)

    • @it-sin9k
      @it-sin9k  3 роки тому

      Мы очень рады быть полезными) у нас еще много разных тем, которые мы хотим вам рассказать)
      Поэтому подписывайтесь и нажимайте колокольчик, ведь у нас нет спама)

  • @aleksandrzelenskiy4000
    @aleksandrzelenskiy4000 3 роки тому

    Нормально я зашел! По совету коллег

    • @it-sin9k
      @it-sin9k  3 роки тому +1

      Добро пожаловать!)

  • @Divanniy.Trader
    @Divanniy.Trader 3 роки тому

    Thanks for your lessons!!!!!!!!

  • @fl1pp1x
    @fl1pp1x 3 роки тому

    Geniously

  • @deanArtDev
    @deanArtDev Рік тому

    Можно еще сигнал (new AbortController) в api прокинуть и остановить запрос при unmount =)

  • @user-wn7sx5ut4o
    @user-wn7sx5ut4o 2 роки тому

    классное видео, спасибо большое!!!

  • @iratnovskaya
    @iratnovskaya 3 роки тому

    Спасибо, оч крутые видео! *_*

    • @it-sin9k
      @it-sin9k  3 роки тому +1

      Спасибо!) Лучшей благодарностью будет поделиться видео с коллегами)

  • @user-lk4mb3nw3d
    @user-lk4mb3nw3d 3 роки тому

    Контент кайф, продолжайте в том же духе

  • @dmitrykorovin4356
    @dmitrykorovin4356 2 роки тому

    очень полезный ролик, сразу подписался, спасибо за контент

    • @it-sin9k
      @it-sin9k  2 роки тому

      Можете заодно стать нашим спонсором)

  • @dmitrykarpovich186
    @dmitrykarpovich186 Рік тому

    Спасибо! Очень крутой контент!

  • @LRXAORLOV
    @LRXAORLOV 3 роки тому +1

    Может рассмотрим подробнее темы, связанные с уменьшением дублирующего кода, создание абстракций для работы с сущностями в проекта, какой-то слой сервисов для преобразования данных для отрисовки, чтобы не прям в верстке логика была)?

    • @it-sin9k
      @it-sin9k  3 роки тому +2

      Чтобы перейти к обсуждению более архитектурных вещей, хотелось бы сразу пройтись по базе, условно понимать хорошо как работает key или какой то хук. Имея такие видео, я могу на них ссылаться в будущих видео. Это как программа в школе, сначала надо научиться складывать и умножать прежде чем считать площадь квадрата. Но вообще я сейчас создал голосовалку, на то, что вы хотите увидеть после темы хуков - twitter.com/it_sin9k

  • @STELLS541
    @STELLS541 Рік тому

    Очень годно!!!

  • @gubatenkov
    @gubatenkov Рік тому

    Еще как альтернатива переменной и более нативный метод для отмены запросов существует встроенный класс, который возвращает инстанс контроллера со свойством signal

  • @user-nd5zd5kc5k
    @user-nd5zd5kc5k Рік тому

    Топ🔥🔥
    Получается, что callback из return эффекта срабатывает, если какая то из зависимостей меняется, либо происходит размонтирование компонента?
    И насчёт последнего примера. В классическом приложение есть состояния загрузки и пользователь все равно не увидит результат, но пример очень интересный

  • @someChicoRy
    @someChicoRy 10 місяців тому

    Круто, долго искал инфу, которую ты дал в качестве бонуса) спасибо за твой труд

  • @user-hn5ie9hx6q
    @user-hn5ie9hx6q 2 роки тому +1

    По поводу игнорирования ответа от сервера, рациональнее намного прерывать первый запрос и отправлять второй запрос

    • @it-sin9k
      @it-sin9k  2 роки тому +1

      да, отменять запрос дело хорошее :)

  • @user-lf8gj2lj2p
    @user-lf8gj2lj2p 2 роки тому

    Просто топ, почему так мало подписчиков ????

    • @it-sin9k
      @it-sin9k  2 роки тому +1

      Самое интересное, что я видел похожий комментарий, когда у нас было всего 60 подписчиков, и когда стало 9000)) Вероятно мы еще далеки от предела)

  • @vladimirpolyakov4382
    @vladimirpolyakov4382 3 роки тому

    Гооодится!

  • @Devilmakrey
    @Devilmakrey 2 роки тому

    лучший канал по react!

  • @kostyakozlov5289
    @kostyakozlov5289 2 роки тому

    Спасибо, я как то даже не задумывался об отмене запроса, если её не сделать то может последний ответ придти раньше, тогда старая категория перезапишет нужную пользователю

    • @it-sin9k
      @it-sin9k  2 роки тому

      Все верно, это называется race condition ;)

  • @llwebstylell242
    @llwebstylell242 2 роки тому +1

    вооо, вот это я понимаю - действительно полезный видос для меня, респект. Как раз таки сегодня хотел гуглить о том зачем ставят флаг в useEffect, и тут сразу ответ нашел)
    Недавно как раз писал dashboard и была проблема подобная и не знал как решить, ну теперь явно закреплю практикой всё это дело.

    • @it-sin9k
      @it-sin9k  2 роки тому +1

      Круто! Думаю на канале еще много полезных видео!)

  • @user-lv4mn2ey6p
    @user-lv4mn2ey6p 3 роки тому

    вау-вау-вау!

  • @beholderrk
    @beholderrk 3 роки тому +1

    Круто спасибо! По идее на стадии очистки эффекта можно ещё дополнительно абортировать запрос

  • @kirill537
    @kirill537 3 роки тому +1

    Здравствуйте. Отличное видио, спасибо большое. Можете пожалуйста сделать видео от том как вообще проектировать и разрабатывать компоненты в React вместе с Redux, так чтобы избегать пробросов пропрос в глубину через дочерние компоненты(и обходится без React Context) и делать компоненты более независимы и более переиспользуемыми? Стоит ли использовать хуки useSelector и useDispatch или использовать по старинке connect для Redux?

    • @it-sin9k
      @it-sin9k  3 роки тому +2

      Спасибо, да вполне могу такое видео сделать. Но тем очень много и все хотят разные темы услышать. Поэтому я организовал голосовалку и предложил разные темы, чтобы понять, что люди хотели бы услышать в первую очередь. И тема редакса там тоже есть. Здесь вы можете найти голосовалку - twitter.com/it_sin9k

    • @kirill537
      @kirill537 3 роки тому +1

      @@it-sin9k А книги какие можете посоветовать по React? Все что я находил 2016-18 годов.

    • @it-sin9k
      @it-sin9k  3 роки тому +4

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

    • @kirill537
      @kirill537 3 роки тому

      @@it-sin9k у вас отлично получается продолжайте в том же духе.

  • @user-sw4ed4gh9n
    @user-sw4ed4gh9n Рік тому +1

    А какой в принципе смысл добавлять loadBooks в зависимости useEffect в первом кейсе ? Если допустим у нас будет просто [ ] , то что-то изменится ? Loadbooks ведь может измениться ровно из-за изменения 1 зависимости ( сategoryId) , которая в свою очередь является props`ом , а изменение props`а влечёт за собой полный ререндер компонента . Это имело бы смысл , если бы мы как-то меняли этот id уже в самом компоненте , что конечно просто нонсенс. Я что-то упустил ?

    • @se777en
      @se777en Рік тому

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

  • @123456qwerty132
    @123456qwerty132 Рік тому

    А как лучше поступать, если функция loadBooks так же будет нужна внутри jsx. Например у нас есть кнопка refetch. Тогда использование use callback будет правильным решением или есть другой способ?

    • @it-sin9k
      @it-sin9k  Рік тому

      Да, тогда нужно использовать useCallback

  • @zooyotz
    @zooyotz Рік тому

    Редко такое бывает, но на этом канале хочется ставить лайк каждому видео)

    • @it-sin9k
      @it-sin9k  Рік тому

      Мы рады всем лайкам)

  • @hihoho1578
    @hihoho1578 3 роки тому

    Спасибо.
    Что думаете по поводу recoil от FB?

    • @it-sin9k
      @it-sin9k  3 роки тому

      Я не писал еще на recoil, даже на тестовых проектах, только читал доку.
      По первому впечатлению это redux нового поколения. Мы текущий проект на MobX пишем в крайне похожем стиле. Поэтому может организуем на работе прототип, чтобы оценить, возможно он нам больше подойдет

    • @hihoho1578
      @hihoho1578 3 роки тому

      @@it-sin9k а про effector что-нибудь слышали?)

    • @it-sin9k
      @it-sin9k  3 роки тому

      нет) но выглядит как будто они увлеклись набором абстракций)
      нужно стараться делать все проще и хотелось бы меньше черных ящиков, а больше чистого js. По крайней мере на больших проектах :)

    • @hihoho1578
      @hihoho1578 3 роки тому

      @@it-sin9k у меня было пару проектов с ним. Переписывали с редакса на эффектор. Мне понравилось. Советую посмотреть в его сторону))

    • @it-sin9k
      @it-sin9k  3 роки тому +2

      Может и попробую. Настораживает, только то что у них 21 релиз, а на слуху так и не появился. Обычно хорошие либы быстро всплывают в разных статьях / видео / конфах.

  • @vovapirotskiy5031
    @vovapirotskiy5031 11 місяців тому

    Очень крутые ролики спасибо за контент !❤
    Микро вопрос зачем создавать булавы переменные и затем инвертировать их
    If(! isIgnoreResponse) …
    Почему бы сразу не объявить переменную с отрицанием например
    shouldProcessResponse вместо isIgnoreResponse
    Или возможно есть какая то конвенция или бест практис
    Заранее благодарю 🙏🏽
    7:56

    • @it-sin9k
      @it-sin9k  11 місяців тому +1

      нет особой причины) видимо в тот момент мне показалось это более читабельным)

    • @someChicoRy
      @someChicoRy 10 місяців тому

      Слушай, а можешь объяснить, что ты имел ввиду? Если не булевые переменные, то что еще можно использовать?

    • @someChicoRy
      @someChicoRy 10 місяців тому

      Имею ввиду, что такое переменная отрицания?

    • @it-sin9k
      @it-sin9k  10 місяців тому

      @@someChicoRy Например можно создать переменную hasPermissions, а тебе нужно написать if где, если у человека нет пермишеннов, то сделать что то. В таком случае теоретически можно создать переменную hasNoPermission. И вот такие переменные я стараюсь не создавать. В таком случае либо подумать как вообще переименовать ее без No чтобы была, либо восклицательный знак использовать

    • @someChicoRy
      @someChicoRy 10 місяців тому

      спасибо. Плюс минус понял) @@it-sin9k

  • @evgenykoypish9398
    @evgenykoypish9398 3 роки тому +1

    useEffect by React Documentation

  • @imvasia13
    @imvasia13 3 роки тому +1

    Знаю, что такую проблему, что указана в бонусе, можно решить rxjs и оператором switchMap, оператор просто оменяет предыдущий запрос, когда тригерится новый.
    Я хочу уточнить, насколько корректно использовать rxjs в мире реакта?

    • @it-sin9k
      @it-sin9k  3 роки тому

      я периодически слышу, что где-то, кто-то использует rxjs, но никогда не видел своими глазами)

    • @AndriiKuftachov
      @AndriiKuftachov 2 роки тому

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

    • @AndriiKuftachov
      @AndriiKuftachov 2 роки тому

      Мне кажется (уверен процентов на 90), что раньше на сайте Redux в разделе FAQ был вопрос про RxJS и был ответ, что если вы используете RxJS, то вам нафиг не нужен Redux.

  • @ReAgent003
    @ReAgent003 Рік тому

    5:24 использование самовызывающейся асинхронной функции ничем не повредит, верно?
    7:32 или можно использовать AbortController

    • @it-sin9k
      @it-sin9k  Рік тому +1

      1) Не повредит, просто читать непривычно
      2) да, конечно, единственное, а что если запросов несколько или у тебя еще вычисления после запроса. И именно в этот момент анмаунтится страница, все равно есть гэп

    • @yauhen9607
      @yauhen9607 Рік тому

      @@it-sin9k 2) С его вы взяли, что сразу будет получен респонс 2, а не 3? Задержки сети, разная сложность запросов. На простом примере конечно оно всё красиво выглядит

    • @it-sin9k
      @it-sin9k  Рік тому

      @@yauhen9607 ну пусть придет 3-ий респонс раньше чем 2-ой. Какая разница? 2-ой респонз все равно проигнорируется, а 3-ий дождется и засетится

  • @bur5153
    @bur5153 2 роки тому

    я думаю, что многие к такому подходу пришли без чтение документации, через практику

  • @user-ge2qk4cm1j
    @user-ge2qk4cm1j 3 роки тому

    +

  • @muhammadkasymov4087
    @muhammadkasymov4087 2 роки тому

    недавно только гадал, как правильнее) спасибо ответ

  • @Zluka366
    @Zluka366 3 роки тому

    а если объявлять функцию как функциональное выражение это поможет оптимизации?

    • @it-sin9k
      @it-sin9k  3 роки тому

      а что такое функциональное выражение? (лучше пример с кодом)

    • @Zluka366
      @Zluka366 3 роки тому

      ​@@it-sin9k
      const Example = () => {
      *const exampleClick = () => {}*
      return Example
      }
      const Example = () => {
      *function exampleClick() {}*
      return Example
      }
      разное объявление функций влияет как-то на оптимизацию?)

    • @it-sin9k
      @it-sin9k  3 роки тому

      я понял о чем речь. В примерах в React документации, часто используют function вариант для методов. Даже при создании компонентов, они часто там пишут
      function Example() { return test }
      Но я думаю они это делают, для упрощения понимания примеров. Т.к. в create-react-app уже генерится компонент в виде стрелочной функции.
      Влияет ли это как на перфоманс, я думаю вряд ли. Это очередной подсчет спичек. Возможно стрелочные функции чуточку быстрее из-за того что не имеют имени и можно сразу в onClick передать и контекст работает иначе у стрелочной функции
      callSomeMethod(data)}>test
      Поэтому выберите просто более удобный синтаксис и не парьтесь)

    • @Zluka366
      @Zluka366 3 роки тому

      @@it-sin9k спасибо за ответ! :)

  • @irynahudava9460
    @irynahudava9460 3 роки тому +1

    Даже не знала, что некоторые люди пишут первым способом. Интересно)

    • @it-sin9k
      @it-sin9k  3 роки тому +2

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

    • @user-kp1ym2wl2j
      @user-kp1ym2wl2j 3 роки тому

      @@it-sin9k тоже писал первым способом, сейчас перепишу часть прода и задеплоюсь :) спасибо

    • @it-sin9k
      @it-sin9k  3 роки тому

      @@user-kp1ym2wl2j Рад, что данное видео оказалось полезным)

  • @user-kx3sz2bh5d
    @user-kx3sz2bh5d 3 роки тому

    не нужен await на 12 строке в бонусном примере?

    • @it-sin9k
      @it-sin9k  3 роки тому

      а какой смысл нам ждать конца выполнения этой функции?
      Более того наоборот, нужно скорее вернуть в return функцию, недожидаясь ответа от сервера, чтобы если deps изменяться, было что вызвать :)

    • @user-kx3sz2bh5d
      @user-kx3sz2bh5d 3 роки тому

      ​@@it-sin9k ну чтобы у нас true не проставился в игнор еще до того как с сервера пришли данные. ощущение что данный код никогда не добавит книгу
      хуки блочат рендер пока не будут завершены? await на сетевой запрос это уже браузер API, поэтому браузер может не блокировать основной поток нашим ожиданием промиса

    • @user-kx3sz2bh5d
      @user-kx3sz2bh5d 3 роки тому

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

    • @user-kx3sz2bh5d
      @user-kx3sz2bh5d 3 роки тому

      но возможно я не понимаю хуки, я с ними еще не работал, вот знакомлюсь)

    • @it-sin9k
      @it-sin9k  3 роки тому

      Я думаю вы упускаете несколько важный моментов:
      - в return мы возвращаем функция, вызов которого откладывается до изменения dependencies, т.е. isIgnoreResponse не изменяется до того как зависимости не изменятся
      - абсолютно для каждого вызова функции переданной в useEffect создается своя переменная isIgnoreResponse.
      Я вас точно уверяю, что код рабочий, вам стоит попробовать написать и поиграться с ним, чтобы вникнуть :)

  • @it-coding
    @it-coding 3 роки тому +1

    Я даже не припомню кейса , где бы нужно было в зависимость функцию передавать ,
    Я обычно создаю функцию вне эффекта , а в эффекте вызываю ее передав в нее параметр categoryID например

    • @it-sin9k
      @it-sin9k  3 роки тому

      Это значит, что ваш стек или ваши подходы к организации кода, полностью искореняют возможность такого кода. Например, мы сейчас пишем приложение используя mobX и там действительно нет никакой нужды делать такого рода конструкции, но в более простых pet проектах, я сталкивался с такой проблемой :)

    • @it-coding
      @it-coding 3 роки тому

      @@it-sin9k Ну на сколько я понял, задача стоит в том , что при новом categoryId мы вызываем функцию , getBooks , но в этой функции мы используем пропс переданный в компонент , то есть , при входе на страницу , мы вызовем при монтировани страницы useEffect вызовет функцию и мы получим книги , указав зависимость categoryId мы получаем тот же результат , и при создании функции внутри хука при монтировании и при зависимости от категории , тот же результат

    • @it-sin9k
      @it-sin9k  3 роки тому

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

    • @it-coding
      @it-coding 3 роки тому

      @@it-sin9k теперь понял))

  • @ovionlogis
    @ovionlogis Рік тому

    @it-sin9k в чем плюсы/минусы вообще не использовать useEffect в данном случае
    const Books = ({ categoryId }) => {
    const [category, setCategory] = useState(null);
    const [books, setBooks] = useState([]);
    const loadBooks = async (id) => {
    const response = await api.getBooks({ categoryId: id });
    setBooks(response.books);
    };
    if (categoryId !== category) {
    setCategory(categoryId);
    loadBooks();
    }
    return ( ... )
    }

    • @it-sin9k
      @it-sin9k  Рік тому

      Такой код немного опасен, для вашего приложения. Первая проблема, это вызов обновления стейта (setCategory) до, того как рендер закончится. Как реакт должен себя правильно вести себя, когда построение текущего виртуального дома не закончилось, а вы уже призываете к следуюзему рендеру?
      А вторая проблема, это вызов запросов во время рендера. Основная идея, в том, что у реакта есть свой жизненный цикл, когда стоит начать отправлять запросы (в useEffect). И если вызывать прямо в рендере, то реакт в некоторых ситуациях, не сможет гарантировать правильную работу приложения. Это будет на вашей совести, потому что пользуете фреймворком, не так как предпологают реакт разработчики

  • @monterio1234
    @monterio1234 3 роки тому +2

    А если не указывать loadBooks как зависимость и не идти на поводу у eslint? Тогда никакого лишнего кода, читаемость лучше чем с калбеками, а то что loadBooks каждый рендер создается пофиг, это вряд ли станет узким местом в программе

    • @it-sin9k
      @it-sin9k  3 роки тому

      Это будет вполне себе работать. И читабельность будет терпимой. Только придется смириться с несколькими вещами:
      1) отказаться от eslint правила следящего за зависимостями
      2) смириться с тем что вы создаете функцию до того как контент отрисовывается в браузере, а не прямо перед ее использованием (офк это экономия на спичках, не сильно тянет как весомый аргумент)
      А так да вполне рабочий вариант, я бы даже сказал, лучше чем с useCallback

    • @kostyakozlov5289
      @kostyakozlov5289 2 роки тому

      @@it-sin9k можно даже отмену запроса в эффекте реализовать, создать объект с полем отмены присваивания и просто передавать в функцию loadData, а в функции сброса менять это поле, useRef на коленках :)

  • @bogdanshelomanov5668
    @bogdanshelomanov5668 3 роки тому

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

    • @it-sin9k
      @it-sin9k  3 роки тому +1

      я расскажу свое видение из кода, который я прочел и обрывков фраз, которые я слушал на выступлениях на конференциях официальных от React Core team. "Hook" в переводе с английского языка звучит как "Крючок".
      Допустим у нас есть компонент у которого есть useState(6). И как вы знаете эта цифра 6 будем доступна при первом рендере, втором или любом другом рендере, пока вы на одном из рендеров ее не перезапишите. Как перезапишите будет другое значение хранится в state. Хотя ваш компонент это всего лишь функция, а у функции как вы знаете не существует this. Где тогда хранится эта цифра 6?
      Ответ прост, они сохраняют цепочку вызванных хуков условно в "глобальной переменной" и при новом рендере они крючком цепляются за туже "глобальную переменную".
      Т.е. хук это не что иное как просто придуманная абстракция с именем, которое React разработчики посчитали наиболее успешным. Таким же образом они могли назвать хуки - например Packages / BusinessLogic/ КотВМешке / КонопляНаБатуте

    • @bogdanshelomanov5668
      @bogdanshelomanov5668 3 роки тому

      @@it-sin9k я думал, что хуки подвязаны для каждого фибера, если они глобальным обьектом, как тогда реакт понимает, что у конкретной ноды - конкретные хуки?

    • @it-sin9k
      @it-sin9k  3 роки тому

      Они хранятся у конкретной ноды, я поэтому и написал "условно"))
      Я вот прямо сейчас расписываю новый выпуск и там как раз немного углубляюсь, про то как работают хуки) а пока рекомендую посмотреть это видео про исходники хуков - ua-cam.com/video/kcHEWut-DUA/v-deo.html

    • @bogdanshelomanov5668
      @bogdanshelomanov5668 3 роки тому

      @@it-sin9k там тоже сказано, что хуки в глобал обьекте, меня как раз это и сбило с толку

    • @it-sin9k
      @it-sin9k  3 роки тому

      по факту вот в переменную currentlyRenderingFiber присваивается текущая fiberNode.
      И внутрь него в виде linkedList складываются все хуки
      github.com/facebook/react/blob/v17.0.1/packages/react-reconciler/src/ReactFiberHooks.new.js#L557
      А при повторном рендере из currentlyRenderingFiber.alternate достается linked list из прошлого рендера
      github.com/facebook/react/blob/v17.0.1/packages/react-reconciler/src/ReactFiberHooks.new.js#L573
      И начинается создание нового linked list. И в этой строке они берут данные из прошлого хука и сохраняют его в текущий хук
      github.com/facebook/react/blob/v17.0.1/packages/react-reconciler/src/ReactFiberHooks.new.js#L606

  • @alexandrmoskalev8642
    @alexandrmoskalev8642 3 роки тому

    norm

  • @vargvarg7922
    @vargvarg7922 Рік тому

    7:10
    пам пам, и 3 пришел раньше чем 2, ибо асинхронное общение с сервером, и первый запрос не гарантирует первый ответ, а ты потом радуйся жизни, ловя плавающий баг

    • @it-sin9k
      @it-sin9k  Рік тому

      ну вроде в решении которое идет дальше, учтен этот момент :)

    • @vargvarg7922
      @vargvarg7922 Рік тому

      @@it-sin9k дальше да, я к тому что в этом моменте не моргание самая большая проблема

    • @it-sin9k
      @it-sin9k  Рік тому

      @@vargvarg7922 согласен)

  • @frontman6849
    @frontman6849 2 роки тому

    хера ты жесткий)

  • @Sylar7891
    @Sylar7891 2 роки тому

    5:28 а какой смысл выделять пямять чтобы дать функции название, если она сразу же вызывается? Можно просто создать самовызывающуюся анонимную стрелочную асинхронную функцию:
    (async () => { await ... })()

    • @it-sin9k
      @it-sin9k  2 роки тому +1

      Да можно и так сделать :)
      Я использую чаще всего функцию с именем, потому что самовызывающиеся функции в коде почти никогда не встречаю в коде. Соответственно людям сложнее читать код.

    • @gooseob
      @gooseob 2 роки тому

      @@it-sin9k, мб лучше просто передаваемую в useEffect функцию сделать асинхронной, а не в ней функцию создавать

    • @it-sin9k
      @it-sin9k  2 роки тому

      @@gooseob Мб что то изменилось в последних версиях, но раньше React в консоль ошибку плевал, если ему асинхронную функцию передаешь

    • @gooseob
      @gooseob 2 роки тому

      @@it-sin9k, а, понял, спасибо

    • @gooseob
      @gooseob 2 роки тому

      @@it-sin9k, да, сейчас можно вроде. Я сначала объявлял асинхронную функцию, а потом передавал её в useEffect

  • @volodymyrduday5258
    @volodymyrduday5258 2 роки тому

    700 likes

  • @romanmed9035
    @romanmed9035 6 місяців тому

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

    • @it-sin9k
      @it-sin9k  6 місяців тому

      сорри, что пропустил другой ваш вопрос.
      В данном вопросе вроде как размер функции не имеет значения. Вместо api.getBooks() будет какая то кастомная функция, но суть вроде как не меняется
      const books = getBooks(...) // some custom function
      setBooks(books)

    • @romanmed9035
      @romanmed9035 6 місяців тому

      @@it-sin9k спасибо за пояснения

  • @AndriiKuftachov
    @AndriiKuftachov 2 роки тому

    Я сходу не понял, нафига создавать функцию вместе того, чтобы потом ее отдельно вызывать.
    А потом понял, на какие костыли не приходится идти, если не используешь RxJS 🤣

  • @user-bx7yp9gy7f
    @user-bx7yp9gy7f Рік тому +1

    7:33 а нельзя ли использовать abort.controller?

    • @it-sin9k
      @it-sin9k  Рік тому +1

      Вполне себе можно :)

  • @user-kj8sw6js6y
    @user-kj8sw6js6y 3 роки тому

    Дока не пропагандирует подход, а предоставляет максимально простой и понятный код, для более быстрого и легкого понимания механизма и принципа работы.
    Самое рациональное решение - вызывать запрос в санке или саге. В хуке useEffect будет максимально понятное - вызов экшена и передача id в него и они же в зависимостях. Ты пропагандируешь лишнюю связанность кода. Компонент не должен знать о запросе.

    • @it-sin9k
      @it-sin9k  3 роки тому +3

      Так код, который я предоставил, это пример из реакт документации) тогда получается из этих слова "Ты пропагандируешь лишнюю связанность кода", не я пропогандирую, а документация все таки))
      Ну а если серьезно, то не одним редаксом едины. Сейчас уже не первый проект вижу тенденцию, что люди отказываются от того, чтобы всю асинхронщину совать в thunk / saga. Их мотивация заключается в том, что если ты используешь thunk, тогда тебе надо хранить информацию в redux. Если хранишь информацию в redux, а ты ее не планируешь глобально использовать, по разным причинам, например быстрой потери актуальности информации. Тогда тебе приходится еще чистить redux store, после анмаунта компонента. И люди решили, а зачем было вмешивать редакс в этот флоу? нельзя было просто в хуке сделать запрос показать лодинг и стейт сам очиститься после анмаунта? Особенно это актуально в RN проектах. Где тьма экранов, с мелкими запросами, и надо все равно приходится каждый раз перезапрашивать актуальные данные.
      В итоге, я все чаще вижу проекты, где люди прогоняют через redux, только те запросы и данные, которые действительно нужно хранить глобально. А если тебе не нужно хранить глобально, тогда вот такое написание кастомного хука часто можно встретить в проекте.
      paste.ubuntu.com/p/83zssmRhPR/
      Более того, если писать сейчас на mobX проект. Скорей всего будете использовать mobx-react-lite, у которого весь код будет в кастомных хуках обернутых внутри в useLocalObservable)
      Поэтому если, вы решили использовать redux по максимуму в своем проекте, а реакт только для отрисовки, это не значит, что ваш подход лучший и все должны его использовать. Это лишь один из возможных вариантов написания сайта.
      Более того, я писал раньше код, так как вы предлагаете и продвигал в массы, что при таком подходе все однообразно просто и понятно. Но на дистанции. Сначала я отказался от Саги, т.к. она начала давать больше сложностей, чем пользы. Потом от redux-form, т.к. сам создатель либы от нее отказался, все по этим же причинам, чтобы не сотрясать лишний раз редакс. И в итоге отказался от хранения всех данных в redux)
      Просто делюсь своим опытом)

    • @alexanderbunin7388
      @alexanderbunin7388 3 роки тому

      от твоего сообщения веет Димычем) к счастью, его подход - не панацея)

    • @AndriiKuftachov
      @AndriiKuftachov 2 роки тому

      @@it-sin9k по-моему, тех, кто хранит все данные в глобальном сторе нужно принудительно лечить (за редким исключением, когда это действительно нужно).

    • @AndriiKuftachov
      @AndriiKuftachov 2 роки тому

      А глобальное хранилище не должно знать о данных, которые нужны только в рамках компонента.
      Компонент должен звать какой-то слой отвечающий за запрос данных, а лучше ещё промежуточный слой, который отвечает за логику выполняя роль application из DDD.
      Но все равно, он будет асинхронным, поэтому нету смысла усложнять пример... как и использовать Redux для запросов.

    • @AndriiKuftachov
      @AndriiKuftachov 2 роки тому

      Как раз Redux для запросов и даёт лишнюю связанность кода, а если писать на чемоданчиком э человеческом TS (в крайнем случае JS), то потом код можно реально переиспользовать.

  • @romanmed9035
    @romanmed9035 2 роки тому +1

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

  • @user-dt1ev2td5c
    @user-dt1ev2td5c Рік тому

    Я создаю внутри useEffect самовызывающуюся функцию (async () => {...})(). В этом случае мне не приходится как то её называть и вызывать после создания. Нам ведь функция нужна для того , что бы сделать её асинхронной. Не знаю правда на сколько читабильно

    • @it-sin9k
      @it-sin9k  Рік тому

      Да, вопросик есть только в читабельности :)

  • @VeaceslavBARBARII
    @VeaceslavBARBARII 7 місяців тому

    7:40 isIgnoreResponse sounds unnatural.
    I'd better call it shouldIgnoreResponse.

  • @Skar38rus
    @Skar38rus 2 роки тому

    sin9k, 9000 грехов! 9 грех это ок!

  • @user-rt8ct7li5x
    @user-rt8ct7li5x 2 роки тому

    Можно и не использовать isIgnoreResponse. А просто прервать запрос. А промис в исключение улетит

  • @ArthurYelkin
    @ArthurYelkin 2 роки тому +6

    Спасибо большое за контент, давайте больше бест практисов, реально очень полезно знать)

  • @marynakosiv3750
    @marynakosiv3750 3 роки тому

    офигеть какой контент! такого еще не видела
    ++ за пояснения с графом

    • @it-sin9k
      @it-sin9k  3 роки тому

      Спасибо :) И добро пожаловать!)