10 React Antipatterns to Avoid - Code This, Not That!

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

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

  • @Mikenight120
    @Mikenight120 2 года назад +449

    PLEASE MORE VIDEOS LIKE THIS!!! I love code reports and 100 seconds series but wish we got videos like this more often! Keep up the awesome job!!

  • @hodabi
    @hodabi 2 года назад +51

    Great video! When you work with react every day for a few years, you slowly realize exactly the things you showed here. It would've been nice to see this 2 years ago :)

  • @Sakiifyable
    @Sakiifyable 2 года назад +28

    This is by far the most efficient way to learn code and environment optimization I've seen. Every language needs a video like this, amazing work!

  • @valtism
    @valtism 2 года назад +161

    I think that when you have prop drilling issues, before reaching for context or redux, it's usually best to consider if you are using the `children` prop effectively. A lot of the time if you aren't using the drilled prop in the intermediate components, they can render `{children}` and that prop can be passed directly from the origin parent to the component that needs it.

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

      Thank you. This is great.

    • @yevhenkozlov286
      @yevhenkozlov286 2 года назад +2

      And it always more flexible. For free.

    • @Bobobratwurscht
      @Bobobratwurscht 2 года назад +4

      Sorry I don't understand this. Can you explain it a bit different?

    • @Dekatelon
      @Dekatelon 2 года назад +17

      @@Bobobratwurscht You have a parent component and a child component. The parent component accepts a bunch of props which are just used to pass (or drill) it to the child component. What you can do instead is to just accept a children prop in the parent component and render it. When you now render the parent component, you pass the child component as the children prop of the parent.

    • @nikilragav
      @nikilragav 2 года назад +2

      @@Dekatelon I'm still not following.

  • @kay_dev
    @kay_dev 2 года назад +121

    Very important thing with context - any item that consumes that context (i.e., the `const count = useContext(CountContext);` in the example) will ALWAYS rerender when the context value changes, regardless of if that component actually uses that value or not. A very common pitfall is to use Context as a sort of 'big store', where you've got a ton of values in the context that are consumed by components that only need one or two of said values. This means that every time the context component updates, EVERY SINGLE CHILD COMPONENT THAT RELIES ON THAT CONTEXT WILL ALSO RERENDER. It's a huge, huge performance issue in very large apps.
    Keep context to items that will change extremely rarely - themes, authentication states, etc - and never use them for data that changes somewhat frequently unless you're making a very tiny app that won't see the exponential performance loss of this.
    If you need something passed down but don't want to go through the whole heavy mess of Redux / MobX / etc, minimalist state management libraries like Recoil, Zustand, and Jotai are very easy, approachable, and lightweight ways to do this the 'right' way.

    • @feritperliare2890
      @feritperliare2890 2 года назад +10

      But why even have this problem in the first place why is sharing info in react so awful in every way possible

    • @xbutterguy4x
      @xbutterguy4x 2 года назад +5

      using useMemo to prevent re-renders is a much more minimal approach as you don't need to install any dependencies

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

      great notice thank you

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

      @@feritperliare2890 do you mean why react re-renders? if so, it is a way of keeping application UI updated as the state changes.

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

      @@adeleke5140 yes that’s how react does it meanwhile other frameworks only re render what needs to be re rendered when you call the info somewhere and not every spot that your store somehow reaches its a stupid expensive way to make sure your UI is updated

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

    9 months later and I can finally understand this video! First time I watched it, I had no idea what any of this was.

  • @amirhoseinhesami9336
    @amirhoseinhesami9336 2 года назад +630

    React's flexibility is its greatest strength, but if you don't know what you're doing, it can sometimes lead to weird things

    • @shapelessed
      @shapelessed 2 года назад +26

      That's why more opinionated frameworks like Vue exist, so you don't have to learn every way you can do stuff.
      I myself would honestly stick to Svelte for personal projects, it's one of few frameworks that don't actively piss you off on each step.
      (And yes, React is a lib - as long as you don't slap JSX on top of it)

    • @amirhoseinhesami9336
      @amirhoseinhesami9336 2 года назад +8

      @@shapelessed That's why other libs/frameworks like Vue, angular and svelte in the second, third and fourth place

    • @lahcencodery
      @lahcencodery 2 года назад +2

      @@amirhoseinhesami9336 react came first and that was huge advantage.. i would go with vueJS or svelte or even angular for complex Enterprise apps. The non opinionated react is not really my thing

    • @amirhoseinhesami9336
      @amirhoseinhesami9336 2 года назад +11

      ​@@lahcencodery vue and svelte in the enterprise world is not a real thing and also the funniest joke ever however the "angular" used to dominate the enterprise world however it is not relevant today because react is dominating the enterprise as well

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

      @@amirhoseinhesami9336 backed by Google and Facebook/meta 🤷🏻
      Any other reason which is better!?

  • @ЂорђеЈеленковић
    @ЂорђеЈеленковић 2 года назад +2

    Man you are a genius. Your channel and then 10 places are empty and then everyone else, it's amazing how original you are with content and just keep going! Big greetings from Serbia! :)

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

      ☝️☝️☝️Thanks for your feedback

  • @kwangsamyew8469
    @kwangsamyew8469 2 года назад +8

    I am using Svelte exclusively, but the concepts here in React still apply when it comes to componentizing. Super useful advice, thanks!

    • @ba8e
      @ba8e 2 года назад +5

      Svelte saved my fucking life. React is garbage in comparison.

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

    There's an instructor named John Smilga and his 14 hour long react video on yt covered all of these tips and tricks along with so many other things for free. I am glad that people like you guys exist.

  • @Heisenberg-xc8ub
    @Heisenberg-xc8ub 2 года назад +1

    I've been doing some stuff you said here without knowing it myself, I'm a self learner and have been following you for code quality improvements. You have once again upped my bar, thank you very much 🙏

  • @austinevans9467
    @austinevans9467 2 года назад +40

    One of these for Angular would be amazing! Great video as always Jeff

    • @invinciblemode
      @invinciblemode 2 года назад +19

      Angular is dead (This comment was paid for by Ben Awad)

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

      @@invinciblemode AngularJS to be exact, killed by google since December last year

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

      Eh, that would probably confuse most Javascript viewers who've never heard of types lol

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

      @@niklasstahl98 Not really. Most JS developers with a little bit of experience will be familiar with the fundamental types. It's only jarring for the first couple of days using them in a strict manner but after that it becomes natural.

  • @hanesmitter1469
    @hanesmitter1469 2 года назад +124

    Another anti pattern is doing component updates on unmounted component resulting to memory leaks. Majorly caused by asynchronous Javascript

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

      ☝️☝️☝️Thanks for your feedback

    • @rhebucks_zh
      @rhebucks_zh 2 года назад +11

      @@chatmeup8917 not actual firebase

    • @heheboi812
      @heheboi812 2 года назад +16

      Many people forget to use useeffect's cleanup function!

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

      when fetching data, the AbortController (native API) can be used, or a library that abstracts this away or avoids this issue can be used, like react-query

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

      calling an async function without setting the state would not lead to memory leaks right?

  • @FunctionGermany
    @FunctionGermany 2 года назад +16

    edit: apparently currying is still the correct term, i've just never ever seen it used for general function generation.
    regarding 9: in the example that _you show_ it's a curried function, but what _you describe_ is just a factory function.
    curried functions basically let you specify the multiple arguments of a function A in multiple steps where each step or function call returns you a function where that value is "stored" in the closure and you can specify the next argument with the next call on the returned function.

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

      Which is better in terms of implementation though? Factory or Curried? Because they're technically the same...

    • @FunctionGermany
      @FunctionGermany 2 года назад +5

      @@creatorsremose they're not. currying means the final function arguments get composed in this staged fashion. every curried function is a function factory, but function factories don't have to receive or provide any arguments up or down the chain. when working with react, you probably never need currying.

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

      When would you need currying?

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

      @@BuyHighSellLo it's a functional programming pattern for when you have a target function with more than one argument. if you look up currying you should find good examples.

    • @yonoseespanol
      @yonoseespanol 2 года назад +2

      He is correctly describing a curried function. Also, factory functions return objects, not functions (unless we're being strict with how we define a function in Javascript).

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

    Your videos are amazing man. Whenever I need a quick refresher you're the go to! Really appreciate all your hard work.

  • @brianevans4
    @brianevans4 2 года назад +25

    If you curry a javascript function in es6, you dont' need the braces and return statement, you can double up on the arrow syntax:
    const handleIt = v => e => console.log(e, v)
    equivalent function to the one shown at 7:31

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

      True, but this also comes at a cost of readability

    • @madhououinkyoma
      @madhououinkyoma 2 года назад +4

      Talk about antipatterns

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

      He uses TypeScript so if you want to define a type for the parameter you need to use parentheses

    • @youarethecssformyhtml
      @youarethecssformyhtml 8 месяцев назад +2

      I hate this syntax with a passion. I use regular function declarations for named functions and arrow functions for callbacks

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

    i love that you're explanations have no ummm's or aaaaa's, making them bearable and even entertaining to watch. love the edits

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

    This content was a gold mine. Started using React more in depth a few weeks ago to build a booking app, and I was just getting to the point of needing something like Suspense to conditionally render UI. Thanks!

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

      You don't REALLY need to use suspense to conditionnally render UI, it's not mandatory.

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

    This is such a good video. When you're new to React, it's flexibility and minimalism makes it unclear how you *should* be doing things, the only thing that is exposed obviously is how you *can* do things.

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

    I was just going through your react videos as I have a big interview coming up tomorrow. This is the best addition to my revision. Thanks a lot! You work in mysterious ways

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

    The 100 seconds course sounds fantastic. Small chunks of knowledge delivered without any bullshit. I hate when I look for a solution or example, and the article 10x times the length it could be if it skipped the obvious parts, like setting up the project

  • @xrr-1
    @xrr-1 2 года назад +27

    Another useful feature "functional state update"
    if the new state depends on the previous state, you can pass a callback to setState like
    `setState(previousCount => previousCount + 1)` instead of `setState(count + 1)`
    Using functional state update will help in the memoization of the event handler using `useCallback` as the dependencies can be an empty array
    and it will also help in avoiding stale state closures

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

      Always followed it to save myself from trouble

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

      i use for handleinput functions

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

      Yeap..it is a known issue. Everytime you need the current state value always use the callback. You can even end up in weird bugs if you the state value gets stale (like in an interval)

    • @wizkid22
      @wizkid22 2 года назад +2

      I remember seeing this in a PR and was very confused.... Now I use it all the time

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

      setState(count++) ; )

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

    I was trying to determine why your voice is so familiar. Then I realized I took a React Native course with you! Just wanted to let you know, it taught me enough that I ended up developing a full stack app using similar architecture!

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

    I stopped naming my files index.js because of that, not sure if I will switch to what you showed, it's amazing how you talk about stuff nobody anywhere talks about
    Also that vs code extension, HOLYY what a lifesaver

  • @Pilosofia
    @Pilosofia 2 года назад +27

    to the people who still use classes components: make sure to keep cleaning your caves, bad environment effect the productivity of the developer.

    • @blackkitty_42
      @blackkitty_42 2 года назад +5

      I just got accepted as a frontend dev on a startup company. A year ago, CEO hired some devs to create mobile app and web app using react native. Devs finished the contract and left.
      And this is where i get in. Imagine me, a react newbie, who have to fiddle with a messilly coded and stinky class based component react app. The worst part is, there's almost zero class based component tutorials on youtube.
      At least the pay is nice

    • @EnriqueDominguezProfile
      @EnriqueDominguezProfile 2 года назад +2

      @@blackkitty_42 I'd avise you to rely on the React docs, fortunately they have an excellent documentation (which is actually rare), and with plenty of class-based examples. And try sell your boss a progressive refactoring of your code.

    • @archmad
      @archmad 2 года назад +2

      i remember when my team first work on React before it got hooks but 3 months later, hooks got introduced and people didn't want to jump ship. tried convincing others to move all to hooks, but too late. we end up half of the code in hooks

    • @Pilosofia
      @Pilosofia 2 года назад +2

      @@blackkitty_42
      the majority of react developers stop using classes and try to avoid it. because functions give almost the same result and with more readable and clean code. classes still has some render cycle methods that are not exist in functions but you will not need them in 90% of the cases.

    • @scottfowler
      @scottfowler 2 года назад +2

      @@blackkitty_42 I set aside an hour every workday to refactor one or two class components into hooks-based function components. A week of that taught me more about the codebase than the previous two months, and after two weeks my boss took me aside and asked me to refactor everything. Refactoring to hooks is such a win in performance and complexity that you should try to make it happen however you can, everyone around you will notice the improvements quickly

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

    React's flexibility comes with great responsibility. I once worked on a React project where everything is memoized. We thought it would improve performance, but lately I realized it was a premature optimization

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

      Definitely a problem I see often. Using memo/useMemo/useCallback is a tradeoff between spending processing time and memory. JavaScript engines are really fast nowadays, and React isn't slow either. For most UI you won't notice any performance issues unless you do something really complicated; at which point you have a few options;
      If your component does something that results changes the DOM or needs to do offload rendering where it doesn't matter whether React can keep track of it, you can use imperative APIs; e.g. for animations. You can use refs to grab the context of underlying DOM nodes of React elements you want to access.
      In most cases, splitting out components into logical parts that can do their own work, such as subscribing to specific state or doing computationally expensive things that aren't easy to optimize otherwise, you can memoize it, optionally with a comparison function. The less props you pass those expensive components the easier it becomes to manage at the cost of less reusability-again a compromise you'll have to make.

  • @benwritescode
    @benwritescode Год назад +2

    In react everything depends on the developer’s knowledge, it can get to a complete mess or a solid structure. Great video 👍

  • @sighmanthethird
    @sighmanthethird 2 года назад +2

    Awesome, educational video! Really appreciate the effort (and also the humor) that went into this. Also, this is the first tutorial video I watch on default speed and not 1.5x 😅

  • @mountainslopes
    @mountainslopes 2 года назад +2

    Great video! It’s crazy to me though how many of these patterns are encouraged out of the box with something like SvelteKit. First principles approach with a clean slate has done the Svelte team wonders.

  • @BlurryBit
    @BlurryBit 2 года назад +2

    Ok, glean was a blessing from this video. I did not know it existed lol. I used rfc and rnfc, but this is next level. Thanks man!

  • @ChristianHelmsOther
    @ChristianHelmsOther 2 года назад +4

    Very helpful as always. As someone just getting back into coding after a break these help a ton. I'd love to see one over Tailwind CSS

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

      Why did you take a break if I may ask?, Was it due to a burnout or something like that?, I'm just curious so you don't have to answer if you don't feel like doing so :).

    • @ChristianHelmsOther
      @ChristianHelmsOther 2 года назад +2

      @@daumienebi I started a marketing and business development company. So for the majority of my time doing other things. I still built websites and coded it was just in WordPress more than anything so mostly code snippets and changing things in a theme. Recently, my company has grown to the point where I can spend some time doing new things and I picked up design and software development a lot more!

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

      @@ChristianHelmsOther ohhh okay, congratulations on your Company!!

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

      @@daumienebi Thank you very much! It's been a blast

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

    Terrific video! Every point is... well, on point. 🙂
    You have a new subscriber. 👍

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

    I'd love to be able to vote financially for what content you focus on, because I'd love more of these code this not that videos

  • @dekafmusic
    @dekafmusic 6 месяцев назад +1

    I love this guy's humor and explanation style😂😂

  • @TomDoesTech
    @TomDoesTech 2 года назад +7

    The "prop plowing" technique is great but I tend not to do it much because TypeScript won't validate the input against the component's input props

    • @teacul
      @teacul 2 года назад +2

      TypeScript, deservedly, gets a lot of praise, but it's really under-discussed how much you have to change the way you code in order to use it properly. I think it's lighthandedness is pretty overstated

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

      I believe TS validates spreaded objects `{...childProps}`, but it might only work if the object created was typed explicitly.

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

      @@teacul I personally don't try to fight it too much, it's an addition here to simplify our lives but if it ends up becoming cumbersome that's not a great deal anymore

  • @ncookiez
    @ncookiez 2 года назад +7

    This video just made me very happy that I work with Svelte...

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

      It is inevitable. React will die eventually. Long live Svelte!

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

      It's funny enough you are happy with Svelte but watching video about React patterns :D

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

      @@yevhenkozlov286 It's like watching videos of life in Africa, that make you appreciate what you have.

  • @nakuldhingra3682
    @nakuldhingra3682 2 года назад +5

    Many people are asking about vscode settings. According to me :
    Font : Fira Code
    Theme : One Dark Pro

  • @d3-in-10-minutes-or-less
    @d3-in-10-minutes-or-less 2 года назад +4

    A masterclass in both react and instructional videos. Thank you!

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

    "... that leaves us developers with plenty of room to screw things up with our own stupid ideas." So underrated.

  • @strategistaow3520
    @strategistaow3520 5 месяцев назад

    7:31, I couldn't think of that. I was at least familiar with some coding tips you have provided until 7:31 as i am watching, but i knew that i can return a function.

  • @davidalexander8786
    @davidalexander8786 2 года назад +2

    Context is not only "cool" to prevent prop drilling, is very helpful to separate state logic for a bunch of components

  • @wizkid22
    @wizkid22 2 года назад +11

    In number 9 while technically correct there is a pretty decent size caveat. React 16 and 17 will batch setStates as long as it not in an async function. If you are in an async function you can use unstable_batchedUpdates to batch setStates. The latter being pretty hacky in my mind, but can save you from a large refactor

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

    never heard about glean. Tested it this morning, it really has potential to speed up my work. Thanks for sharing!

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

    Two minutes in and there's a gem of a VSCode extension already. 👏🏽👏🏽👏🏽Can't miss!

  • @FunctionGermany
    @FunctionGermany 2 года назад +11

    for the context API i like to use this pattern where i have a single file that
    - creates the context with the default value undefined, but does not export it
    - creates and exports a provider component
    - creates and exports a custom hook that uses useContext and throws if the context value is undefined
    this way you always only import one thing (useMyContext hook) and you'll also always immediately know if you're using a component and there's no provider in it's parent hierarchy.

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

      Interesting….where can I learn this method?

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

      @@shawnc7381 on your own 🤣. Just understand the context API and custom hook deeply 💯

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

      @@shawnc7381 it's really nothing more than i described in the comment :D

    • @AdamKent755
      @AdamKent755 2 года назад +2

      @@shawnc7381 I can't post a link here, but google "kent c dodds how to use react context effectively" there's a post on his blog that covers this pattern.

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

    Dude, I like your content so much that I feel like I owe you money every single video you post. Great work Jeff!

  • @adirmugrabi
    @adirmugrabi 8 месяцев назад +1

    HOLY S**T glean is amazing!
    thank you for letting me know about this! this was a dream for me

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

    With great power comes great responsibility! Fantastic video, and I am definitely going to be using a few of these tips and tricks. Thank you.

  • @mentoriii3475
    @mentoriii3475 2 года назад +2

    5. That only works if you style your components with plain css, however most modern react applications are done with a framework or Sass, in that case i suggest splitting components with use case and create a index.js in the folder and export all components from that folder

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

      ☝️☝️☝️Thanks for your feedback

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

    Great video. As a beginner, i learned a lot in these 9 minutes. Thanks! 🔥 🚀

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

    Just installed glean. Thanks for the tip!

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

    I was in a trouble in today with react, and you uploaded a video about it. thank you very much

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

    Great video! I need this course for my start up project im currently working on.

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

    Thanks!

  • @albertcamus6611
    @albertcamus6611 6 месяцев назад +1

    i know i'm late to the party, but this is a lot to do a few simple things. I am coming from vue and needed to see habits for react development. no offence though, I think both get the job done, but react is more common in this region. That refactor markup -> component thing is really good.

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

    Even before this video I thought my event handlers were ugly. Now I'm going to rewrite them as curried functions. Thank you so much!

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

    Redux is not a heavy handed approach in my opinion. The reducer itself is a pretty simple function and Redux allows for a much better separation of concerns when couples with reselect and sagas.
    I’d argue that for any react app you’re being paid to make, react+redux+sagas+reselect is a must and the morally correct setup to use.

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

    Happy code this not that videos are back. Hey did Jeff switch away from angular and why???
    I guess Nextjs motivated him or something?🤣

  • @ethanneff9817
    @ethanneff9817 2 года назад +34

    Some of these are anti-patterns as well. Spreading props is an anti-pattern because it is not declarative and it can cause unneeded exporting of types between components. Additionally, your curried function should be nested within a useCallback() to prevent extra re-renders. If you want to teach against anti-patterns, it would be better if you taught your audience eslint recommended settings for common plugins.

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

      What are those eslint recommend settings?

    • @double-agent-ly
      @double-agent-ly 2 года назад +9

      I agree with the second half but not so much as the first. Spreading is a fine pattern if you as the developer are aware of what you are doing. Not to mention typescript ensures you use it properly

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

      @@_guru ESLint has a bunch of default rules, but if you really wanna give it a go, try the Airbnb rules. Those are opinionated, and I enjoy them very much.

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

      Spreading props isn't an anti-pattern, there's a major difference between personal code style and actual anti-patterns

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

      Callstack's eslint-config package is really nice to use. Not a big fan of AirBnb rules though.

  • @nathanadhitya
    @nathanadhitya 2 года назад +46

    Hey, I've actually thought of using curried functions, but my initial thoughts tell me that it is going to going to consume more memory as for each call, a new big function will be made again to be called. In contrast, the one without function currying I suppose would call the same function, not create a new one, but with different params, hence better perf and memory. I have yet to test if this is true or not. Is that how it works in JS? I'd like to know the results for this.

    • @CzajekTutorialowiec
      @CzajekTutorialowiec 2 года назад +32

      Doesn't matter. With curried function on 3 calls you create and return 3 functions. If you use 3 arrow functions, you created them as well, but manually, not programmatically.

    • @isqueirosbic
      @isqueirosbic 2 года назад +2

      I don't think that's the case. See, what you're thinking is that the onChange function runs the currying on every call, but what it does is that each call runs the returned function. The curry function only runs on re-render.

    • @CzajekTutorialowiec
      @CzajekTutorialowiec 2 года назад +5

      @@isqueirosbic Idk if you were responding to me or Nathan, but let me make it clear:
      In both cases on each render you create three functions that live in memory. In the first example you create them manually writing three arrow functions. In the second example you create those three functions by running the "creator"/"factory" function.
      In the second example you have "more calls" but that doesn't matter. The memory footprint is close to being exactly the same.

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

      @@CzajekTutorialowiec I was answering nathan ;)

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

      In either case, it creates 3 new functions on every render.
      If you *really* want to try to avoid this, you can create a useCallback function but this is most likely overkill over-optimisation.
      const handleClick = useCallback((param) => (event) => clickHandler(event, param, state), [state])
      Button text

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

    i didnt know there were this many antipatterns in react, i will be using all of them now! nice video

  • @dealloc
    @dealloc 2 года назад +13

    Rather than creating a custom hook just for storing related state, useReducer can be more ergonomic as it can also provide you with a single dispatch function that can update that state using actions.
    As for the second point about nesting; this also comes with a gotcha in that the parent component will update all its children when it updates state that may just be used for a few or a single child. It's useful if child components are reusable, but unnecessary otherwise. It's fine to co-locate state and event handlers in child components and is often easier to reason about; If you truly run into performance issues due to unnecessary re-renders, you can wrap child components in React.memo.

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

    I feel so good that I knew all of this and predicted all your solutions without even looking into the topic.

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

    Another good way to manage state is with the react-tracked library, it offers the flexibility of the context api, the advanced usage of reducers like redux, but the main goal is to avoid rerender of components when it's not needed. Check this library, it's a good one in my opinion 👍

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

      The JS solution for everything: just install another library and hope it still works when you deploy :D

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

    Suuuper pumped for the new React course on Fireship Pro

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

    Your course is surely gonna be awesome 🔥💪🏻

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

    Points 5 and 9 really help me, thanks!

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

    Thank you, Jeff! This might save me as a newbie-ish React developer for damn sure!

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

    curried functions won't prevent rerenders because when you call the function it returns a new function, so the reference is new each time. useCallback won't help here either, but if you set up your callbacks to target child component's return signature, then you can simply use useCallback

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

    Thanks! Was learning react today and this was like, not even let me choose the wrong path from start!

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

      you don't get better at something without sucking at it first

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

      @@mentoriii3475 ikr, still this is a good starting point

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

      ☝️☝️☝️Thanks for your feedback

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

      @@chatmeup8917 shut up impersonator

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

    Hey. One cool thing is you can also return multiple components in array, like return ([,]) instead of wrapping them in React.fragment

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

      Interesting, I think the ... Syntax is way nicer to read though.

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

    Bring it bruh!! I want that course

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

    I love 4:30!! Have been struggling with multiple index.tsx files for a while now 😂

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

    This is an amazing vid. Use react for work and a lot of the problems and solutions you mentioned apply to my everyday life.

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

    So hype for the react course!

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

    this was awesome. i love that last tip!

  • @islamibrahim8121
    @islamibrahim8121 2 года назад +2

    Tip #3 put a smile on my face, I was expecting useEffect and it being called out for being a major footgun but he actually highlighted useMemo in a completely novel way.

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

    Thanks, knew most but nice reminder as we often forget things if not practiced lately

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

    The vscode refactoring extension is a huge lifesaver.

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

    Thanks for the extension Glean. Saves my life

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

    Now I know why I messed up when first trying to build a thing on RN.
    This framework and Js project really need a simpler structure and easier to manage.

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

    I'm writing my first react app and prop drilling is the first thing I've noticed as not very elegant with my code. Now I can name it and think about solutions.

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

    I really enjoyed this. Good job 👍🏻

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

    Never heard of Glean before. Thanks a lot for sharing

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

    Thanks for the PRO discount! Just subscribed for life :)

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

    Can you go back three years in time and put this video out? Would have saved me a lot of time figuring these all out one by one...

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

    Not all heros wear capes, so much good nuggets in here. I use currying all the time but I didn't know about glean and now I know how to deal with the index.tsx index.tsx index.tsx issue in my vscode tab bar ☺

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

    Zustand in 100 seconds please. As always , great video. Thanks for your time.

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

    Have you done rxjs before? I think it's great to learn it since it has changed how I think and solve problems related to data handling. It feels like shaping a river with your own hands like a god.

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

    I love you. This was amazing. Please do other technologies in the same way. Angular, rxjs, firebase? :P

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

    Awesome explanation! Thank you so much for making this video!

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

    I really really like your style. Did you ever make public your "youtube stack" would be very interested. I want to do similar videos for the Data / Backend side.

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

    This is a super helpful video. Thank you for the great video!

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

    the curried function for event handlers is my new favorite piece of code.

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

    Thanks again for the tutorials Jeff your style of teaching is very straight forward and to the point. I am curious if anyone has seen a similar extension for JetBrains (glean)

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

    Hey! I just learned about antipatterns in my softwares engineering class today. What a coincidence 😮

  • @DavidDeCorso
    @DavidDeCorso 2 года назад +260

    React antipattern #1: AngularJS

    • @shapelessed
      @shapelessed 2 года назад +11

      Then goes Vue and Svelte...

    • @Suresh-br2zz
      @Suresh-br2zz 2 года назад +12

      Ben Awad wants to know your location

  • @guillemgarcia3630
    @guillemgarcia3630 2 года назад +2

    Thanks! With this video I appreciate more the simplicity that is rendering html with go html template

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

      ☝️☝️☝️Thanks for your feedback

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

    Really helpful video, thanks a lot!

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

    For the currying comment, rather than having a function outside of the component, use a useCallback or with every render it's producing a new function which then means when you're passing that as a prop it's seen as a prop change and causes a re-render for those components