Best Practices for useEffect by React Documentation

Поделиться
HTML-код
  • Опубликовано: 18 окт 2024

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

  • @ВячеславС-к1м
    @ВячеславС-к1м 3 года назад +41

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

    • @it-sin9k
      @it-sin9k  3 года назад +3

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

  • @АндрейПахомов-м7ъ
    @АндрейПахомов-м7ъ 3 года назад +17

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

    • @it-sin9k
      @it-sin9k  3 года назад +2

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

  • @АлексейФурсов-г2ш
    @АлексейФурсов-г2ш Год назад +2

    Хорошая работа! Но есть пару заметок ( Вижу, что видео старое, может мои заметки показаны в след. видео)
    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 ...
    }

  • @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 года назад

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

  • @zooyotz
    @zooyotz Год назад +1

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

    • @it-sin9k
      @it-sin9k  Год назад

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

  • @igormalykhin5528
    @igormalykhin5528 10 месяцев назад +1

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

    • @it-sin9k
      @it-sin9k  10 месяцев назад +1

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

  • @ArthurYelkin
    @ArthurYelkin 2 года назад +6

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

  • @andreyshevchenko4602
    @andreyshevchenko4602 3 года назад +6

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

    • @it-sin9k
      @it-sin9k  3 года назад

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

  • @serhiirumiantsev7736
    @serhiirumiantsev7736 3 года назад +15

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

    • @it-sin9k
      @it-sin9k  3 года назад

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

  • @solexofficial9597
    @solexofficial9597 3 года назад +2

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

  • @379sanek
    @379sanek 3 года назад +3

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

    • @it-sin9k
      @it-sin9k  3 года назад

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

    • @379sanek
      @379sanek 3 года назад

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

    • @it-sin9k
      @it-sin9k  3 года назад +1

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

  • @mikhailstepanischev8316
    @mikhailstepanischev8316 3 года назад +2

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

  • @marynakosiv3750
    @marynakosiv3750 3 года назад +1

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

  • @dmitrykarpovich186
    @dmitrykarpovich186 Год назад

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

  • @someChicoRy
    @someChicoRy Год назад

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

  • @igordzyabenko3712
    @igordzyabenko3712 3 года назад +2

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

    • @it-sin9k
      @it-sin9k  3 года назад +1

      Спасибо :)

  • @denydavydov9290
    @denydavydov9290 3 года назад +1

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

    • @it-sin9k
      @it-sin9k  3 года назад +1

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

  • @НикитаГончаров-е8ш

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

  • @illyakyrylov
    @illyakyrylov 3 года назад +1

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

  • @LRXAORLOV
    @LRXAORLOV 3 года назад +1

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

    • @it-sin9k
      @it-sin9k  3 года назад +2

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

  • @llwebstylell242
    @llwebstylell242 2 года назад +1

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

    • @it-sin9k
      @it-sin9k  2 года назад +1

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

  • @kostyakozlov5289
    @kostyakozlov5289 2 года назад

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

    • @it-sin9k
      @it-sin9k  2 года назад

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

  • @deanArtDev
    @deanArtDev 2 года назад

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

  • @aleksandrzelenskiy4000
    @aleksandrzelenskiy4000 3 года назад

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

    • @it-sin9k
      @it-sin9k  3 года назад +1

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

  • @muhammadkasymov4087
    @muhammadkasymov4087 2 года назад

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

  • @dmitrykorovin4356
    @dmitrykorovin4356 3 года назад

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

    • @it-sin9k
      @it-sin9k  3 года назад

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

  • @gubatenkov
    @gubatenkov Год назад

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

  • @ДмитрийЛуба
    @ДмитрийЛуба 2 года назад

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

  • @ДониёрАзизов-о8ц
    @ДониёрАзизов-о8ц 3 года назад

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

    • @it-sin9k
      @it-sin9k  3 года назад

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

  • @Devilmakrey
    @Devilmakrey 3 года назад

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

  • @СергейЛукашевский-п2щ

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

    • @se777en
      @se777en Год назад

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

  • @beholderrk
    @beholderrk 3 года назад +1

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

  • @diegoavenger7702
    @diegoavenger7702 3 года назад

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

  • @STELLS541
    @STELLS541 Год назад

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

  • @123456qwerty132
    @123456qwerty132 2 года назад

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

    • @it-sin9k
      @it-sin9k  2 года назад

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

  • @СергейУстинов-ф3щ
    @СергейУстинов-ф3щ 2 года назад

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

    • @it-sin9k
      @it-sin9k  2 года назад +1

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

  • @Илья-ж8ч8о
    @Илья-ж8ч8о 3 года назад

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

    • @it-sin9k
      @it-sin9k  3 года назад

      спасибо!)

  • @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 у вас отлично получается продолжайте в том же духе.

  • @Divanniy.Trader
    @Divanniy.Trader 3 года назад

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

  • @iratnovskaya
    @iratnovskaya 3 года назад

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

    • @it-sin9k
      @it-sin9k  3 года назад +1

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

  • @Сергей-й9т7ъ
    @Сергей-й9т7ъ 2 года назад +1

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

    • @it-sin9k
      @it-sin9k  2 года назад +1

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

  • @kirillpavlovskii8342
    @kirillpavlovskii8342 3 года назад

    хорошо

  • @vovapirotskiy5031
    @vovapirotskiy5031 Год назад

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

    • @it-sin9k
      @it-sin9k  Год назад +1

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

    • @someChicoRy
      @someChicoRy Год назад

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

    • @someChicoRy
      @someChicoRy Год назад

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

    • @it-sin9k
      @it-sin9k  Год назад

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

    • @someChicoRy
      @someChicoRy Год назад

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

  • @-X-Ray-
    @-X-Ray- 3 года назад

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

  • @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.

  • @evgenykoypish9398
    @evgenykoypish9398 3 года назад +1

    useEffect by React Documentation

  • @ReAgent003
    @ReAgent003 2 года назад

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

    • @it-sin9k
      @it-sin9k  2 года назад +1

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

    • @yauhen9607
      @yauhen9607 Год назад

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

    • @it-sin9k
      @it-sin9k  Год назад

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

  • @bur5153
    @bur5153 2 года назад

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

  • @fl1pp1x
    @fl1pp1x 3 года назад

    Geniously

  • @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 релиз, а на слуху так и не появился. Обычно хорошие либы быстро всплывают в разных статьях / видео / конфах.

  • @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 на коленках :)

  • @vladimirpolyakov4382
    @vladimirpolyakov4382 3 года назад

    Гооодится!

  • @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 теперь понял))

  • @ДониёрАзизов-о8ц
    @ДониёрАзизов-о8ц 3 года назад

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

  • @vargvarg7922
    @vargvarg7922 Год назад

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

    • @it-sin9k
      @it-sin9k  Год назад

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

    • @vargvarg7922
      @vargvarg7922 Год назад

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

    • @it-sin9k
      @it-sin9k  Год назад

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

  • @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 спасибо за ответ! :)

  • @romanmed9035
    @romanmed9035 9 месяцев назад

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

    • @it-sin9k
      @it-sin9k  9 месяцев назад

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

    • @romanmed9035
      @romanmed9035 9 месяцев назад

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

  • @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 года назад

      Они хранятся у конкретной ноды, я поэтому и написал "условно"))
      Я вот прямо сейчас расписываю новый выпуск и там как раз немного углубляюсь, про то как работают хуки) а пока рекомендую посмотреть это видео про исходники хуков - ruclips.net/video/kcHEWut-DUA/видео.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

  • @irynahudava9460
    @irynahudava9460 3 года назад +1

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

    • @it-sin9k
      @it-sin9k  3 года назад +2

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

    • @ДмитрийИванов-з8з2м
      @ДмитрийИванов-з8з2м 3 года назад

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

    • @it-sin9k
      @it-sin9k  3 года назад

      @@ДмитрийИванов-з8з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

  • @Сева-я4э
    @Сева-я4э 3 года назад

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

    • @it-sin9k
      @it-sin9k  3 года назад

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

    • @Сева-я4э
      @Сева-я4э 3 года назад

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

    • @Сева-я4э
      @Сева-я4э 3 года назад

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

    • @Сева-я4э
      @Сева-я4э 3 года назад

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

    • @it-sin9k
      @it-sin9k  3 года назад

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

  • @AndriiKuftachov
    @AndriiKuftachov 2 года назад

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

  • @frontman6849
    @frontman6849 3 года назад

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

  • @romanmed9035
    @romanmed9035 2 года назад +1

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

  • @alexandrmoskalev8642
    @alexandrmoskalev8642 3 года назад

    norm

  • @ДалерКамолов-г5т
    @ДалерКамолов-г5т 2 года назад +1

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

    • @it-sin9k
      @it-sin9k  2 года назад +1

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

  • @АртёмЕмельянов-х7ч
    @АртёмЕмельянов-х7ч 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), то потом код можно реально переиспользовать.

  • @Skar38rus
    @Skar38rus 2 года назад

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

  • @Алексей-т2х8й
    @Алексей-т2х8й 3 года назад

    +

  • @volodymyrduday5258
    @volodymyrduday5258 3 года назад

    700 likes

    • @it-sin9k
      @it-sin9k  3 года назад +1

      Спасибо!

  • @VeaceslavBARBARII
    @VeaceslavBARBARII 9 месяцев назад

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

  • @ДмитрийМарков-ь4э
    @ДмитрийМарков-ь4э 2 года назад

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

    • @it-sin9k
      @it-sin9k  2 года назад

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

  • @Артем-щ5у2и
    @Артем-щ5у2и 2 года назад

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

  • @marynakosiv3750
    @marynakosiv3750 3 года назад

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

    • @it-sin9k
      @it-sin9k  3 года назад

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