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

React is a minimal on the surface, but it’s actually a highly complex JavaScript UI library with many potential pitfalls. In this tutorial, we look at 10 antipatterns in React, along with tips and tricks to improve our code. fireship.io/courses/react
#react #js #CodeThisNotThat
🔥 Use this discount code to get 33% off Fireship PRO:
hh7F4Ezs
fireship.io/pro
🔗 Resources
React Docs reactjs.org
Full React Course fireship.io/courses/react
React in 100 Seconds • React in 100 Seconds
📚 Chapters
00:00 React Antipatterns
01:00 1. Big Components
01:55 2. Nesting Gotcha
02:35 3. Failure to Memoize
03:15 4. Useless Divs
03:44 5. Messy Files
04:40 6. Big Bundles
05:34 7. Prop Drilling
06:30 8. Prop Plowing
07:00 9. Try Some Curry
07:39 10. Code Smarter
🎨 My Editor Settings
- Atom One Dark
- vscode-icons
- Fira Code Font
🔖 Topics Covered
- React Pitfalls
- React Code Smell
- Problems with React.js
- React Best Practices
- Structure a react project
- Antipatterns in react to avoid
- React tips and tricks
- React interview tips

[@Mikenight120]

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]

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 :)

[@valtism]

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]

Thank you. This is great.

[@yevhenkozlov286]

And it always more flexible. For free.

[@Bobobratwurscht]

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

[@Dekatelon]

@@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]

@@Dekatelon I'm still not following.

[@Sakiifyable]

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!

[@Heisenberg-xc8ub]

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 🙏

[@amirhoseinhesami9336]

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]

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]

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

[@lahcencodery]

@@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]

​@@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]

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

[@DANJUMA9]

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

[@thevamsi]

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

[@user-ol6tq5hw7s]

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]

☝️☝️☝️Thanks for your feedback

[@kay_dev]

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]

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

[@xbutterguy4x]

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

[@akremgomri9085]

great notice thank you

[@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]

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

[@wizkid22]

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

[@dylanclarke9497]

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]

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

[@sighmanthethird]

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 😅

[@d3-in-10-minutes-or-less]

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

[@kwangsamyew8469]

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

[@ba8e]

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

[@darck5240]

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

[@SonAyoD]

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

[@mountainslopes]

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.

[@jakeysnaketube]

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.

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

[@binukads]

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

[@codinginflow]

Just installed glean. Thanks for the tip!

[@neard82]

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

[@justingolden21]

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

[@hanesmitter1469]

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

[@chatmeup8917]

☝️☝️☝️Thanks for your feedback

[@rhebucks_zh]

@@chatmeup8917 not actual firebase

[@heheboi812]

Many people forget to use useeffect's cleanup function!

[@FunctionGermany]

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]

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

[@austinevans9467]

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

[@invinciblemode]

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

[@paulwhiterabbit]

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

[@niklasstahl98]

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

[@austinevans9467]

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

[@RazKoller]

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

[@outis99]

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

[@theatypicaldeveloper]

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

[@SirVib]

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

[@BlurryBit]

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!

[@brianevans4]

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]

True, but this also comes at a cost of readability

[@madhououinkyoma]

Talk about antipatterns

[@igorswies5913]

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

[@ChristianHelms]

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]

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 :).

[@ChristianHelms]

@@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]

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

[@ChristianHelms]

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

[@hta218_]

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

[@Ferdziosz]

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

[@MarthinusBosman]

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

[@benwritescode]

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

[@phucnguyen0110]

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

[@FelipeMaffezzolli]

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

[@aurielklasovsky1435]

I love these. Thanks!

[@Pilosofia]

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

[@user-yy3ki9rl6i]

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]

@@user-yy3ki9rl6i 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]

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]

@@user-yy3ki9rl6i 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]

@@user-yy3ki9rl6i 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

[@FunctionGermany]

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]

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

[@FunctionGermany]

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

[@jameszhang1278]

When would you need currying?

[@FunctionGermany]

@@jameszhang1278 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]

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).

[@nro337]

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

[@nemeziz_prime]

Your course is surely gonna be awesome 🔥💪🏻

[@Kwuasimoto]

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)

[@BaileySimrell]

Suuuper pumped for the new React course on Fireship Pro

[@heroe1486]

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

[@hofmeinmer]

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!

[@REDnWHITEnGREY]

this was awesome. i love that last tip!

[@hoaxygen]

Thank you, this confirms some of my hunches! Very informative

[@coolicz]

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

[@ryanfleming1996]

I really enjoyed this. Good job 👍🏻

[@xrr-1]

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]

Always followed it to save myself from trouble

[@pt_trainer9244]

i use for handleinput functions

[@victorlongon]

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]

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

[@TheMikhaylov]

setState(count++) ; )

[@rohitbhujbal7333]

That's a great video!! Any thoughts on Recoil for state management instead of using Redux?

[@ShiNijuuAKL]

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

[@ncookiez]

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

[@ba8e]

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

[@yevhenkozlov286]

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

[@ba8e]

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

[@davidalexander8786]

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

[@thepriestofvaranasi]

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.

[@jaymanx4life]

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

[@codeaperture]

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

[@sounak2009]

Never heard of Glean before. Thanks a lot for sharing

[@albertnahas4619]

Very helpful video, amazing content as usual

[@camschroedes]

So hype for the react course!

[@everenjohn]

Very helpful, thank you!

[@MartinOckovsky]

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

[@adelineiscoding]

Thank you soooo much for glean, didn't know it !! :)

[@nakuldhingra3682]

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

[@gustavoh5143]

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

[@gatocode316]

Points 5 and 9 really help me, thanks!

[@carshoesch]

Really helpful video, thanks a lot!

[@mentoriii3475]

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]

☝️☝️☝️Thanks for your feedback

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

[@nodirbekvositov]

Thanks for the extension Glean. Saves my life

[@phantasyphotography3813]

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 ☺

[@amittraspal]

Just amazing, like always, I am going to bookmark this video until I've used most of these points 😅😅

[@Masus04]

Great Vid! Could you do something similar for Flutter? I reckon much of what you mentioned still applies.

[@mehrandvm7343]

Amazing video. Top notch tips

[@shadmerhi]

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

[@santiagosuarez8479]

Bring it bruh!! I want that course

[@TheGodSaw]

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.

[@lexsemenenko7044]

The code in the end. Thank you. I'm in

[@lookwhoneedsahobbie]

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!

[@apuravela]

Really helpfull video, thanks!

[@metenrog]

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

[@FrancescoOrsi]

lovely video, just I'm confused about the last topic "10. Code Smarter".. regarding useMetadata() could I pass setTitle, setTheme and setTs as well on return?

[@michaelhinojosa9665]

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

[@herrnamenlos1238]

Great video! What about a video about SCRUM? I think SCRUM in 100 seconds would be quite interesting

[@detaaditya6237]

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]

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.

[@oliverhughes169]

Thank you for the awesome videos ❤️

[@samerallahham2182]

07:31 to my knowledge, the call of a function with params without the arrow function syntax causes it to run immediately regardless of the event firing, doesn't it?

[@hugot8226]

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]

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

[@MaximilianKaden]

Amazing ! Thanks !

[@solumyt]

Great video, quick question - what is the advantage of using the context api w/ a custom hook to export global state vs making a simple custom hook like the one you made at 8:40?

[@heroe1486]

Do you mean using a custom hook and calling it from a parent and a child component then expecting state to be updated on the child component when changed in the parent ? That wouldn't work and hou would still need to pass the props and do prop drilling. I made a codesandbox but KZbin doesn't allow me to post it even with a destructured link thanks to spammers.

[@RovelStars]

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

[@mentoriii3475]

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

[@RovelStars]

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

[@chatmeup8917]

☝️☝️☝️Thanks for your feedback

[@RovelStars]

@@chatmeup8917 shut up impersonator

[@TomDoesTech]

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]

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]

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

[@heroe1486]

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

[@neihed]

Great content!

[@julianstorm7722]

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

[@bogdanalexandruburca3579]

Nice video! Can you do a "Code This, Not That!" video for Angular?

[@paulwhiterabbit]

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.

[@Elaoz]

absolutely amazing, thank you

[@chatmeup8917]

☝️☝️☝️Thanks for your feedback