Goodbye, useEffect - David Khourshid

Поділитися
Вставка
  • Опубліковано 15 лис 2022
  • From fetching data to fighting with imperative APIs, side effects are one of the biggest sources of frustration in web app development. And let’s be honest, putting everything in useEffect hooks doesn’t help much. Thankfully, there is a science (well, math) to side effects, formalized in state machines and statecharts, that can help us visually model and understand how to declaratively orchestrate effects, no matter how complex they get. In this talk, we’ll ditch the useEffect hook and discover how these computer science principles can be used to simplify effects in our React apps.
  • Наука та технологія

КОМЕНТАРІ • 447

  • @bepisdevs
    @bepisdevs Рік тому +555

    The answer: use libraries that use useEffect so you don’t have to see it.

  • @foobbar4457
    @foobbar4457 Рік тому +360

    Goodbye useEffect shouldn't be the name of the talk, all those examples, useQuery, useLoaderData are actually using useEffect under the hood, more like "Hello dependencies"

    • @anarchoyeasty3908
      @anarchoyeasty3908 Рік тому +29

      As always with talks like this, you are saying goodbye to using useEffect inside of your component code. The fact that useEffect is used elsewhere in the app doesn't change the fact that you are removing it from your component code itself. Abstracting away messy code is good.

    • @foobbar4457
      @foobbar4457 Рік тому +28

      ​@@anarchoyeasty3908 Then call it "Abstracting useEffect", everybody gets it, show an example of custom hooks and lead into libraries that are better maintained than your custom silly code. The talk/talks would be less confusing ;)

    • @Andreas-gh6is
      @Andreas-gh6is Рік тому +4

      I think he got the problem right: using useEffect to declare asynchronous logic is not scaleable. One solution can be to use things like useQuery which will somewhat simplify that stuff. When using Redux you can also use thunks and saga to avoid chains of useEffect calls.

    • @wlockuz4467
      @wlockuz4467 Рік тому +5

      ​@@anarchoyeasty3908 "Abstracting away messy code is good."
      Messy code will always be messy doesn't matter if its abstracted away.

    • @Andreas-gh6is
      @Andreas-gh6is Рік тому +3

      @@wlockuz4467 But it's not messy it's just a bit complicated or voluminous. Abstracting that away makes it less complicated and less voluminous.

  • @skylight5214
    @skylight5214 Рік тому +121

    Now this talk compared to the previous one is much more helpful. More examples, more time to explain the problem and solutions, and a lot less promotion of XState. Great talk, David, thank you!

    • @zigang93
      @zigang93 Рік тому +2

      agreed.. previous talk look like XState promo

    • @tanercoder1915
      @tanercoder1915 Рік тому +1

      and no ROUND looking brackets (that are actually [] square brackets ) like in the last talt

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

      XState is fantastic though. You really should try it out.

    • @matt-eu-poland
      @matt-eu-poland Рік тому +1

      this one looks like a promo talk as well.

  • @thejezzi5219
    @thejezzi5219 Рік тому +49

    My favorite useEffect is the one which won't show up but then miraculously appears. You know `Rendered more hooks than during the previous render`. Love that. Especially if you don't know why because there are no conditions which prevent a hook from executing.

    • @1OJosh
      @1OJosh Рік тому +13

      If you have a return statement before a hook, you'll get that error. Make sure all your return statements come after all hooks

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

      @@1OJosh Which would mean that I had a condition which decides if the return gets ecevuted or not but there is no conditon or return like I already mentioned in my comment above.

    • @1OJosh
      @1OJosh Рік тому +4

      @@thejezzi5219 Then maybe you've got a condition before a hook, like if(something) useHook()
      There has to be something like that somewhere. Are you able to consistently reproduce the error? Are you sure it's definitely happening at random?
      Could be an external package causing the error as well, I'd check them too

  • @muhammadmartinez2042
    @muhammadmartinez2042 Рік тому +5

    "It seems really cool. Terribly named. But, you know, that's the future."
    Sums up tech nicely

  • @laststone9672
    @laststone9672 Рік тому +1

    Yes useEffect calling 2 times. Finally I got the solution. Thank you sir🙏

  • @aleksamitic6655
    @aleksamitic6655 8 місяців тому

    Excellent one! Really usefull also since the new docs came out from react, first one I red was actually you might not need an effect, and it change my view of react completley!

  • @MatthewFerrin
    @MatthewFerrin Рік тому +14

    The problem with Suspense is that it’s not fine grained enough. For more vertically stabile less jumpy loading experiences you actually want nearly identical html with additional loading styles applied. Having a fallback should be an anti-pattern for pretty and impressive feeling things. It feels so good to transition from some placeholders to more fully loaded content with minimal transitional animation for only some text and some images.

  • @sakules
    @sakules Рік тому +5

    This was a great talk. I am glad state machines are making noise in the web technologies.

  • @codewithguillaume
    @codewithguillaume Рік тому +25

    Sooo interesting. It goes to my watch later playlist.

    • @DreamersLab
      @DreamersLab Рік тому +20

      which I won't watch anytime soon 😆😆

    • @codewithguillaume
      @codewithguillaume Рік тому +6

      @@DreamersLab 😂😂😂 I will I am very disciplined !

    • @codewithguillaume
      @codewithguillaume Рік тому +3

      I wasn’t expecting that my comment would make people 😂

    • @DreamersLab
      @DreamersLab Рік тому +2

      @@codewithguillaume I add videos to watch later all the time but hardly remember to watch them again. The only time I watch again is when I am stuck.

    • @codewithguillaume
      @codewithguillaume Рік тому +1

      @@DreamersLab interesting I think it’s how it works for a lot of people - for me I never have videos to watch - in even struggle to find new ones and get frustrated haha

  • @theindieprogrammer
    @theindieprogrammer Рік тому +416

    I find it hard to believe that I have to install another library to perform a simple fetch call because React dislike its own useEffect-based solution. Thanks, React

    • @dr.michaelmorbius2400
      @dr.michaelmorbius2400 Рік тому +29

      i knew that there was something wrong when that shady dude tried to sell me his javascript course for 15$ 3 years ago, i can't believe how much i've sacrificed for money 😭.

    • @dweblinveltz5035
      @dweblinveltz5035 Рік тому +8

      You can do whatever you want, but react-query (while yes, another library) is great for this purpose--even if there weren't anything wrong with useEffect.

    • @theindieprogrammer
      @theindieprogrammer Рік тому +56

      ​@@dweblinveltz5035 ​ It can be the best library ever, but the problem is not using react-query; The problem is React creating the useEffect hook to handle effects such as data fetching, then discouraging this use case without providing a React-only alternative.
      So, according to the new React "rules", the only viable way to perform a simple fetch call is by using another library.

    • @theindieprogrammer
      @theindieprogrammer Рік тому +5

      @@dweblinveltz5035 The use hook is the alternative, but unfortunately it’s not available yet =(
      If they had released the use hook before “deprecating” the useEffect it would have been great.

    • @khoinguyen-ft2ys
      @khoinguyen-ft2ys Рік тому +4

      @@theindieprogrammer For simple fetch, I usually create custom hook (with useEffect in side) to handle data fetching. But yeah, it is much simpler to use something like react-query to do that job for you.

  •  Рік тому +502

    So React introduced something and based on the documentation and examples they provided, people started to use it as recommended, then React devs figured out that this could be abused badly so they decided to introduce some "safeguards" (which people will ignore anyway).
    Well done again, React 👏

    • @DevMeloy
      @DevMeloy Рік тому +24

      Absolutely! I'm a full stack mostly FE dev in React and am not a fan of React and how complex apps need to be constructed. I've worked in other FE frameworks that seem to be better but are not as popular as React... hopefully that changes soon.

    •  Рік тому +11

      @@DevMeloy I can relate to this. Also, while React seems good in theory, I'm yet to see a project, which is not a mess (or tend towards being that). I don't know the reason of this, and maybe it's just my poor experience, so I don't want to generalize, ofc.

    • @mudaquetoca
      @mudaquetoca Рік тому +4

      "What matters is being popular" 😍😍😍😍

    •  Рік тому +3

      @@mudaquetoca 1 million flies can't be wrong, right

    •  Рік тому +6

      @@SourceHades This is the way! Yes, as I said React pretty much looks good on paper, but not 2 years into a rushed project with 30 devs 😭

  • @peterjhartvideo
    @peterjhartvideo Рік тому +12

    What an intuitive framework 😤 I learned a couple of things, but what I am hearing is using React to communicate with an API is wrong and I should be using React features that don't exist yet or another library

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

    Thank you David, your talk may help the future of components who have been through so much regarding this topic

  • @jazzymichael
    @jazzymichael Рік тому +26

    Regarding fetching data, there was no explanation for how third party libraries handle it. You just said let these other people abstract it for you so you don't mess up, without explaining the abstractions used in remix, next, query, etc

    • @jackhedaya571
      @jackhedaya571 Рік тому +9

      I’m with you. How could something as simple as fetching data not be possible without a 3rd party library

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

      @@jackhedaya571 React is still in beta, but we the “clever” hipsters use an unfinished beta in production

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

      spoiler alert: They all use useEffect

  • @ShubhBeard
    @ShubhBeard Рік тому +51

    import a big library, learn its syntax and internal working but do not use a simple useeffect. thats what the reacts devs wanted to do right.

  • @anazi
    @anazi Рік тому +85

    Oh my God when do you guys stop "innovating" and let us code with confidence.
    Everytime after learning something and feeling proud .. the next day I get a video says why whatever I have learned is now garbage and old.

    • @graficandorealidades7561
      @graficandorealidades7561 Рік тому +14

      front-end development in a nutshell

    • @zen-ventzi-marinov
      @zen-ventzi-marinov Рік тому +6

      Please edit your comment, you forgot the quote marks around "innovating".

    • @anazi
      @anazi Рік тому +1

      @@zen-ventzi-marinov Understandable, but the innovations in the last two years are basically are happening on branches of branches of a branch. Nothing Major, and they are just selling different (shovels in this startup golden rush.) It's just annoying that the same mentors and teachers we rely on them to help us are turning into sales men who wants to use you without thinking of your time, and money.

    • @zen-ventzi-marinov
      @zen-ventzi-marinov Рік тому +1

      @@anazi Couldn't have said it better myself. Very little that is meaningful, and the little meaningful changes taking place(please don't ask me to name them, I can't but for the sake of the argument) are being trumped by all the meaningless changes that we have to keep up with.

    • @anazi
      @anazi Рік тому +5

      @@zen-ventzi-marinov Quote marks implemented. 😂

  • @lucassouzasilva6842
    @lucassouzasilva6842 Рік тому +8

    React devs: we created a monster.

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

      "And now it's up to all of you guys to slay it"

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

    Thanks, very useful talk

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

    Amazing Talk. thank you!

  • @nichiyohane
    @nichiyohane Рік тому +9

    i felt the best answer is going back to class components when you WANT be sure didMount, didUpdate, didUnmount works as expected, not adding another lib that want to implement class components lifecycle control within functional components.

  • @josefsle97
    @josefsle97 Рік тому +16

    react really has kinda turned into a non standardized weird library comparing it to web standards. Nowadays i much more prefer SolidJS or any Library that does not get me to download 20 libraries to make a simple fetch or have virtual dom-magic maxxed out for simple change detection

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

    Really great talk

  • @collinmonahan3428
    @collinmonahan3428 Рік тому +8

    I think useMemo is important not only to memoize the cost of computing some value, but also to stabilize values as they may be passed down as props, to an unknown number of components of unknown implementation, and in general in React apps they can be more efficient when components inputs are not changing unnecessarily... If it's not a primitive then the usual shallow compare will fail when a new object is created on each render. useMemo should be used as good practice imo.
    Great talk, agree wholeheartedly on the main topic, useEffect, thank you!

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

    Amazing 🔥
    Big thanks :)

  • @eXquisiteBuddy
    @eXquisiteBuddy 7 місяців тому +3

    It feels like the solution with useEffect is basically using it with rxjs. For example the takeUntil operator is op for clean ups, I wonder why rxjs didn't pick up with react community.

  • @IStMl
    @IStMl 9 місяців тому +4

    the more i learn React the more i hate it
    the people making it dont even know what theyre doing
    how did this even become the standard

    • @smoothbeak
      @smoothbeak 3 місяці тому

      Amen to that :)

    • @smoothbeak
      @smoothbeak 3 місяці тому

      Became the standard because of Facebook and because people follow what is big and trendy

  • @jorgefelix4409
    @jorgefelix4409 Рік тому +1

    agg the EffectHell what a pain, very nice talk!!

  • @borislavborisov9063
    @borislavborisov9063 Рік тому +63

    Typical web development - hooks come out and everyone is like "wow this is so much better than class based components because we have no this to deal with we can combine stuff logically and no spread it across different lifecycle methods, etc. etc." which are definitely good points.
    The problem I see is that hooks and useEffect in particular were glorified like "this is the way", then a few years later - "useEffect can be really bad, even the name is confusing". Then we have another solution to the problems useEffect might bring. So now I just expect this solution to become problematic as well and people would have to find a solution to the solution. But I guess that's just how stuff is, we have numerous examples, like functional programming being an old paradigm and getting revived in recent years as the way to go. My conclusion - there is no one perfect way of doing anything especially in programming, what is modern today might be an anti-pattern tomorrow and it all comes down to personal preferences, marketing of the idea/approach, having good documentation, etc.

    • @ChillAutos
      @ChillAutos Рік тому +3

      From everything I've seen poor use of useeffects comes from devs never bothering to learn how react even works. Then they just complain about hooks. Learn how it works and you won't shoot yourself in the foot and write rubbish. The old docs were bad though, the new beta docs are good

    • @borislavborisov9063
      @borislavborisov9063 Рік тому +1

      @@ChillAutos I agree, for sure new and better ways come up. I was fine with class based components when they were the thing even though I have seen a few of their downside along the way. Same with hooks, so yeah whatever the technology, you should really have a good understanding of what is going on and how to use it, just as you said so you don't write rubbish or make videos like - why "something" is the wrong way.

    • @VortechBand
      @VortechBand Рік тому +2

      Well, it can also be called: progress. That's how we went from 8086 to Core i9 via small deprecative steps over the years.

    • @borislavborisov9063
      @borislavborisov9063 Рік тому +1

      ​@@VortechBand Absolutely! Going for example from class components to hooks and now having further solutions for the "downsides" of useEffect, this can be considered progress. I don't want us to stay with 1 way of doing things and stick to it forever and ever and not make any progress. What I am up against is how things get presented with a lot of hype in a marketing style like "this is the way" because it sounds contrary to as you said progress, it's like well "Eureka" we found the solution that is "THE Solution". I would like something more objective, like "we've seen how this played out and we see this and this issues, that can be remove/minimised if we alter out approach to this".

    • @sjfieksnd
      @sjfieksnd Рік тому +1

      Technology only eveolves, and That is not a problem. It is how everything is, especially in development.

  • @echobucket
    @echobucket Рік тому +26

    React has never been able to handle fetching data correctly. I don't know what the disconnect is.... I feel like React just wanted to build a rendering library and pretend that async/await, promises and fetching data didn't exist and it wasn't their responsibility.

    • @echobucket
      @echobucket Рік тому +3

      Leading to componentDidMount/componentDidUnmount, useEffect,and Redux Thunks. ALL of which are horrible hard to use designs... I just wish our components could be async functions.. but apparently they've coded themselves into a corner and can only do that on the server....

    • @VortechBand
      @VortechBand Рік тому +6

      Well, that's exactly what it is. React is the View in MVC. Fetching data is the responsibility of the Controller, so it checks out.

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

      @@echobucket nextjs now allows you to just use the await keyword

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

    That generated code is so cool !

  • @alexandrurazvandumitru5744
    @alexandrurazvandumitru5744 Рік тому +7

    people should read the design philosophy in react before saying stuff like useeffect is complicated. perhaps it s not the apps that we build that are not complicated enough for those that understand it, maybe it s just that some devs don t bother understanding how you should use this function and how it works internally and what it is supposed to be

    • @alyankhan806
      @alyankhan806 9 місяців тому

      exactly!

    • @5374seth
      @5374seth Місяць тому

      You're telling me the solution to a problem is to have more understanding of the tools you chose instead of adding more libraries/tools which you understand even less? BLASPHEMY!
      _installs entire npm repository into project_

  • @Gaijin101
    @Gaijin101 6 місяців тому +1

    Years later and the confusion is still not solved. React in a nutshell. For others, use preact signals or vue.

  • @Andreas-gh6is
    @Andreas-gh6is Рік тому +46

    I really don't struggle with useEffect because I don't use it when stuff gets complicated. I don't want to declare my async logic in JSX or with useEffect. That's why I'll use redux-thunk and redux-saga to remove all that logic from the components. So the components can be purely reactive.

    • @wlockuz4467
      @wlockuz4467 Рік тому +9

      Exactly! People don't understand useEffect properly but decide to use it anyways then run into problems and call it bad design. As devs its our job to understand the technologies we work closely with.

    • @ozgursulum4116
      @ozgursulum4116 Рік тому +1

      Sooooo true. I am using RTK Query.

    • @me-cz3wo
      @me-cz3wo Рік тому +2

      remove redux-saga bro

    • @tienlx97
      @tienlx97 Рік тому +3

      @@oleksandrfomin326 exactly, I don't known why we shoud use redux when UI state we have Context Api and server state we have react-query/swr

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

      @@tienlx97 I totally agree with you. I really don't understand why we need a single Redux store for multi-state management. They are standalone states in their own component. We are focusing on single-page rendering with minimal state management while Redux asks us to manage states and methods in one store.

  • @georgethenewbie
    @georgethenewbie Рік тому +10

    The mess and technical debt React has introduced to the world should be at some point be measured in time and resources lost . Companies that adopted React without the teams necessary to maintain it will end up spending thousands of $$$$ to re-write their applications in something else ( by the looks of it Svelte, or Vue, or anything reasonable)

  • @AndreiVasilcoi
    @AndreiVasilcoi Рік тому +3

    Most topics he presents start with "you remember when I said you should do this...well no quite" the entire talk is "not quite/most of the times/usually". Why even learn the framework if in 6 months it changes 180 degrees, just makes it a pain to maintain long running products.

  • @kksdf1
    @kksdf1 Рік тому +4

    Well Angular sounds fun now 🤣

  • @levinie5081
    @levinie5081 Рік тому +3

    xstate looks very powerfull and expressive👍

  • @anthonydugarte
    @anthonydugarte Рік тому +34

    People often miss reading the whole thing, useEffect runs twice only in development, not in a prod build

    • @abdulsalammohammed8045
      @abdulsalammohammed8045 Рік тому +12

      That is because of strict mode

    • @big_gui
      @big_gui Рік тому +3

      @@abdulsalammohammed8045 Exactly, not a big deal though, just remove the strict tag.

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

      At the moment, yes, but a future version of React might run useEffect some unknown number of times because the component could sometimes get unmounted and remounted automatically. The point is that you need to change the mental model of how you use useEffect so that you never depend on the component being (un)mounted.
      Here's a good explanation from a blog post I found: "Such a decision was made by the React team, as in the future, they’d like to add a feature that allows React to add and remove sections of the UI while preserving the state. As it requires components to be resilient to effects being mounted and destroyed multiple times, an additional check has been introduced."

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

      @@SourceHades makes CI/CD so simple and straightforward. /s

  • @gauravsingh1963
    @gauravsingh1963 Рік тому +3

    1- What if we want to call it once for every re-use? We can't have it outside.
    2- if one specific thing changes in prop do something and using custom hook is same thing

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

    seriously i need to save this video. watch twice or more .

  • @_dontlookup_4774
    @_dontlookup_4774 Рік тому +2

    I love this hook called useAngular xD

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

    Thank God

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

    really nice talk! It really is intuitive thing to understand useEffect

  • @axelvalles255
    @axelvalles255 Рік тому +22

    Hopefully vue or svelte will start to be used more

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

    Where I can find more about the "just use()" hook?

  • @vivekkaushik9508
    @vivekkaushik9508 Рік тому +1

    That ending was awesome. Perfect.

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

    Nice demo of your product ;)

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

    Could you tell me what tool you are using for the presentation?

  • @fricze
    @fricze Рік тому +12

    nobody in the right mind will use xstate as a primary state management solution. zustand/jotai/react query/recoil might be solutions. React should've stayed at the "simple view library" level and promote some other, sane, solution for state management. maybe reactive streams. as a person using React from its beginnings, I still think that maybe half of their development now is innovation and the rest is just complication.

  • @notappi3982
    @notappi3982 Рік тому +1

    most of it went over my head but i liked it

  • @orenkaizer6958
    @orenkaizer6958 Рік тому +5

    There will be time that people post a video on "Goodbye, useState"! I'm Sure:)

  • @adityamittal4357
    @adityamittal4357 Рік тому +9

    You could do and remember all this crap and install frameworks into a library just to do basic stuff, or just use Angular which is a stable framework, much more straightforward, class based, and just works.

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

      You don’t have to install another frameworks in a library, maybe another framework in this framework, because React is a framework disguised as a library for marketing (and adoption) purposes

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

      @@adrianvmois9426 That's not true, I've been using and watching since 2013. It's a library that's been trying to become a framework for years that's constantly been changing and is still lacking a lot that a framework needs to have. Despite the addition of hooks etc, people still add redux and saga and all kinds of things to be able to work with React properly. It's not a framework disguised as a library, it's a library trying to become a framework. It literally started as just a way to insert JSX into HTML and render that. That's all it used to do. And it still sucks at state management on its own without redux and managing async state without saga. And doesn't even begin to offer a lot of what the other frameworks offer. Some people even think Jquery is a framework, but it's not.

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

    Haww, I only useEffect and useState from last 2 years. I have never used others. One of my components has 10 useEffects. I was kinda proud of them. Even patted myself in the back and praised react for being such a nice framework.

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

    htmx and alpine! thank you.

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

    In 15:42, is the `setIsOpen(nextIsOpen)` safe? I always think that when you derive next state from current state, the updater function MUST be used.

  • @lorenzrosenthal119
    @lorenzrosenthal119 Рік тому +5

    side effects are the hardest thing in software engineering..... and the frontend is full of this.

  • @MinNyeAccount
    @MinNyeAccount Рік тому +60

    I like how we need external frameworks and state-machines to solve what is trivial problems in all other paradimes than react.

    • @dweblinveltz5035
      @dweblinveltz5035 Рік тому +9

      React is itself just a view library.

    • @ozgursulum4116
      @ozgursulum4116 Рік тому +4

      I mean dweb is right it is just a view library.

    • @adrianvmois9426
      @adrianvmois9426 Рік тому +2

      @@dweblinveltz5035 is a framework marketed as a view library

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

      The primary purpose of framework authors is to lock you into their framework.
      We were able to build systems before React, and one of the most important lessons to learn is that even the UI is just an I/O device and should be abstracted out of the truly important core logic.

  • @miskamyasa
    @miskamyasa Рік тому +2

    What will be next? Goodbye useCallback, useMemo? Goodbye React?

  • @timdowd6613
    @timdowd6613 Рік тому +2

    15:29 line 6 - state is asynchronous

  • @licoling1599
    @licoling1599 Рік тому +1

    Is there anything that can replace useEffect?

  • @ngochunglongnguyen4523
    @ngochunglongnguyen4523 Рік тому +5

    but why do React 18 run useEffect twice when has empty dependencies? Isn't it counter intuitive or is it has other application that React

    • @mattgreenberg3207
      @mattgreenberg3207 Рік тому +8

      It only happens when you turn on StrictMode... and then it only happens in development. Its not a terrible idea. This talk is kinda jank.

    • @ronaldgipa8731
      @ronaldgipa8731 Рік тому +5

      @@mattgreenberg3207 It's a terrible idea because your production build runs differently than development so you need to test your code in both environment to make sure that your app works as intended

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

      Is a rushed and not fully tested library that should be in beta, not in production.

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

    How to use add event listener?

  • @dawnpeace9122
    @dawnpeace9122 Рік тому +3

    I don't know what is right anymore

  • @user-fo4jm9tw9r
    @user-fo4jm9tw9r Рік тому +2

    I waited 28 minutes before David starts talking about Xstate)

  • @syubbanfakhriya971
    @syubbanfakhriya971 Рік тому +1

    this is good, is there an article version of this?

  • @hestiathegoddess5326
    @hestiathegoddess5326 8 місяців тому

    witch presentation program it he using?

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

    this is the reason why i migrated to flutter lol, react doing so many things and then left em, create new one and bullies if someone already comfortable with the old method

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

    What the app that he use to present?

  • @anti5895
    @anti5895 Рік тому +2

    Svelte.

  • @gipek
    @gipek Рік тому +20

    When you advice to use external libraries/frameworks to avoid doing something in a library, you know it's a bad one. I've been thinking that class components was a lot better. Also I am questioning why did I even switched to React, I would be fine and never think about all this if I was using Angular.

    • @wlockuz4467
      @wlockuz4467 Рік тому +14

      Its not that React is bad, its just that people don't understand how useEffect works and end up using it wrong and complain about it. I think one of the major reasons for it is the shift from class to function components.

    • @ishandiablo
      @ishandiablo Рік тому +4

      The problem is React is a library. It was meant to be a library to create basic UI. UI which works and changes itself based on state. All other stuff like async data fetching, SSR, bundling, routing, etc was not supposed to be in it. It was upto dev to add it however they want.
      But that of course became complicated, React team is still trying to figure out how to provide these capabilities, while still staying into the relam of library and not turning into a full blown framework. NextJS for e.g. is something I would classify as framework.

    • @lukafireman
      @lukafireman Рік тому +7

      @@ishandiablo No, the issue is how they included incorrect examples in the Docs and people started using it.
      I've literally searched useEffect in reactjs docs and found a "Bad Example of useEffect".
      No fam, it's just represented badly.

    • @gipek
      @gipek Рік тому +7

      @@wlockuz4467 Yes but if thousands of people are doing it wrong, I think the problem is somewhere else. I'm using React but I feel frustrated about this 'oh we didn't intended this when we introduced hooks' statements.

  • @mfpears
    @mfpears Рік тому +106

    As a developer who has never been in a very messy React codebase, I'm wondering what React got so wrong that someone is advocating for imperative code over declarative code. Like, something is seriously wrong here.

    • @PhilipAlexanderHassialis
      @PhilipAlexanderHassialis Рік тому +32

      Oh trust me, you haven't seen the half of it. Although I prefer hooks, I got to miss the simplicity of componentWillUnmount. Like, ok, the component is about to unmount, so I want to gather such and such information from such and such inner components through references, bundle it up and dispatch it to the state. Can I do that in a useEffect(()=> { .... return ()=>cleanup, []}? No, I cannot because my cleanup has dependencies and the return statement of useEffect(....,[]) will run on unmount only when the state is clean and there are no dependencies, oh, and the inner components are destroyed too. When you get to need something that will run just before the destroy cycle begins, you are left with horrible messy ideas that make your codebase atrocious, horribly atrocious.
      The React team should give a hook or something that performs exactly like a componentWillUnmount, so people can gather data, do any final stuff they want to do and then let the component go.

    • @Diamonddrake
      @Diamonddrake Рік тому +4

      React’s declarative data driven aspect has always been about what you see. Some things always have to be imperative, that’s how events work. The beauty of react is that there’s always an escape hatch to let you work around the limitations.

    • @mfpears
      @mfpears Рік тому +2

      @@Diamonddrake that is NOT how events work. In React I've noticed a lot of developers confuse the mechanics of how React has to work with best practices. Maybe it would be helpful for you to try out another framework sometime.

    • @Diamonddrake
      @Diamonddrake Рік тому +14

      @@mfpears you are being presumptuous, I am a professional software engineer and I maintain code bases across a variety of frameworks and technologies. Events are by definition imperative. Events hook into JavaScript through callbacks in the browser engine and they do so imperatively. React and react minded libraries hide those details as much as possible.

    • @Diamonddrake
      @Diamonddrake Рік тому +8

      @@mfpears I watched your video, I feel like the title and angle is a strange approach. None of your 5 examples are reasons why imperative programming is bad, they don't really draw any lines (you can derive state and keep data close to where its used in imperative programming and you the other points are all subjective). A better approach would be suggesting to be more reactive when possible. I like react because I do enjoy reactive programming. But If you limit yourself exclusively to abstractions that follow a paradigm your core language does not then you are forcing yourself to fight against their limitations. My point is there are times when an action can't be reactive, specifically an event. if you want to do something that doesn't depend on state, then to do it reactively you need to make up some state for it to depend on just to make it happen. If you wrap your events in an observerable, it doesn't just become reactive. If that's what you are thinking then I believe your mental model off. Something being imperative isn't bad, its just not always the best choice. When an event happens like a click and you change some state, that is an imperative action, even if you wrap it in a stream listener and use that to derive new state from existing sate. But you def don't want to manually crawl the browser DOM and update divs everywhere manually during an imperative action. Limit your imperative actions to permuting state and invoking effects that are not state based. You can jump though hoops to keep it declarative, or you can write good code that breaks pretend rules sometimes. Re-rendering an entire system because a number changed is a reactive code problem that can be difficult to fix if they are disjointed in the hierarchy and all your code isn't perfect, Just updating the single spot on the page is an imperative solution that its performant and can save the day. There's very few objective truths in programming.

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

    React is creating new problems, to solve old problems. Thats how I see this.

  • @loucadufault6549
    @loucadufault6549 Рік тому +3

    I can't tell you how much I don't want to import a state machine into front end code...
    Code gen that results in non-native code is just bad. If it at least resulted in a few boilerplate JS components I would be ok with using those as a starting point.

  • @9272fe
    @9272fe Рік тому

    this is sick :D

  • @alyankhan806
    @alyankhan806 9 місяців тому

    why react renders twice in strict mode? - This is because then you can see if your components is pure or not. I found it very useful and it has saved me a few times. What is so difficult to understand about this? people need to read the docs and not judge the framework, it's not reacts fault if you did not understand the reason or made no effort to read the docs

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

    I have only one problem with useEffect when adding object to dependency list, only that

  • @turbulantarchitect5286
    @turbulantarchitect5286 Рік тому +1

    At 13:13, how can the total get updated when items change without useeffect?

    • @hongyuchen708
      @hongyuchen708 Рік тому +2

      when items change, the component will re-render itself, so the line which is used to calculate the sum will be executed again.

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

      @@hongyuchen708 yea, when we want component to rerender our first choice is useEffect and forget that state update also makes component rerender.

  • @baststar
    @baststar Рік тому +1

    I used Angular and React and prefer Angular as a Framework and its RxJS

  • @IvanRandomDude
    @IvanRandomDude Рік тому +4

    Fellas made a rocket science out of calling GET method rofl

  • @maxwellcoding
    @maxwellcoding 6 місяців тому +1

    12:45
    If data comes from an outside source, then you have to use useEffect.

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

    What is alternative?

  • @sarunas8002
    @sarunas8002 8 місяців тому

    What the talk was about was "fetch your data outside the component". What people are saying "f*ck you React, give me encapsulation!"

  • @lucabaxter4002
    @lucabaxter4002 Рік тому +3

    I don't see a component rendering 2 as a big issue. I try to avoid useEffect as much as i can, but some times is really useful for specific cases other than side effects. I get it that "should" be used only for side effects, but i don't see why it is a problem to use it to set states. I think we are still overthinking as usual if we compared now days with back in the days where we were rendering a whole page with refresh and no one was complaining. Rendering a component twice with no bugs is not a huge deal.

  • @q8zahabi
    @q8zahabi Рік тому +1

    I always consider it to be used for "Side Effects" not "Effects" which might change the way how you are using it.

  • @nomadshiba
    @nomadshiba Рік тому +9

    react and its thousand different hooks and "features"
    glad that i never had to use it for too long

  • @noherczeg
    @noherczeg Рік тому +6

    I love how this sums up everything essentially wrong with react. You can have hundreds of conferences for years with people trying to explain how setting a value can go wrong. Can't wait for SolidJS to take the place of react. No surprises there....

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

    This video has more views then all videos combined from this channel

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

    Imagine my face seeing this thing after finishing my morning routine coding a website using Svelte

  • @moolipit
    @moolipit Рік тому +7

    I thought useeffect only run twice in dev mod...

    • @Speglritz
      @Speglritz Рік тому +6

      That's correct, strict mode is only for development mode.

  • @user-ct8dn5xj3o
    @user-ct8dn5xj3o Рік тому +2

    do we have such headache in Vue ?

    • @adrianvmois9426
      @adrianvmois9426 Рік тому +2

      No, Vue is polished and fully tested, not in beta like this garbage.

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

    What to do instead of writing setCounter() useState inside useEffect?

    • @lukafireman
      @lukafireman Рік тому +1

      Handle it in an event handler outside of the effect.

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

      @@lukafireman oh Okay got you thanks.

  • @manfyegoh
    @manfyegoh Рік тому +11

    aren't they making it more complicated?

    • @IStMl
      @IStMl 9 місяців тому

      It's React, what do you expect

  • @apostolgabriel7710
    @apostolgabriel7710 9 місяців тому +1

    When he got into libraries, vue here be like: "look what they need to do to mimic a fraction of our power"

  • @devonlamond
    @devonlamond Рік тому +2

    I like Svelte

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

    what about the 6 vulnerabilities on cra

  • @ThePandaGuitar
    @ThePandaGuitar Рік тому +2

    useEffect can get pretty nasty pretty quickly indeed. Beyond 5-6 dependencies shit really starts to hit the fan.

  • @ObiWanKenobi_IceNation
    @ObiWanKenobi_IceNation Рік тому +1

    The idea that I should handle side effects inside of event handlers sound soooo bad to me though

  • @brahimo4701
    @brahimo4701 Рік тому +25

    if u believe me, the reason I like reactjs more than other frameworks is because of this magically useEffect lol

  • @RyanWaite28
    @RyanWaite28 Рік тому +2

    I just use Angular 🤷🏾‍♂️