DEV Community

Cover image for Stop returning NULL components

Stop returning NULL components

Daniel Silva on May 26, 2021

Conditional rendering on React helps you build your apps avoiding unnecessary renders depending on some validations, and it can be used on tooltips...
Collapse
 
sergey profile image
sergeyv

The article desperately needs a test to prove the point being made. How much is "a lot of performance"? Half a percent? Fifty percent? Is it only measurable if you render thousands of widgets or just a few?

Collapse
 
sergey profile image
sergeyv • Edited

Ok, wrote a quick test app myself: when rendering 125.000 widgets the difference is 450ms for the "outside check" option and ~2000ms when doing the check inside the component, which is quite significant.

For 1000 widgets, it's ~60ms vs ~70ms and is within the error margin. Maybe just don't render 100k widgets at once :)

github.com/sergeyv/inside-outside


UPDATE: Note to self: never do performance testing on a development build :) On a production build the difference is much smaller: for 125K widgets it's 350ms "outside" versus 450ms "inside". I even went ahead and scaled it to 1.000.000 widgets, the results are ~3s vs ~5s.

I have a feeling that, in a real application, there's a very limited number of scenarios where the two approaches could show any measurable difference.

Collapse
 
asyrialak profile image
Andrew Syriala

Does it affect the lighthouse score in any measurable way?

Thread Thread
 
sergey profile image
sergeyv

Does your app render tens or hundreds of thousands of widgets conditionally at the same time? If it does then yes, it will affect the score. The common wisdom is to avoid rendering that many widgets at once though.

I quickly tested with 1.000.000 widgets and got 78 "inside" vs 95 "outside". With 125k widgets, however, both variants got 99.

Thread Thread
 
asyrialak profile image
Andrew Syriala

Stoked that you tried testing it out. It sounds like that test confirms its a bit of a premature optimization. Great tool in your toolbet if you ever need to render a millionish components at a time though.

Collapse
 
ucvdesh profile image
Daniel Silva

Well yeah, but that's for a really simple example app, now imagine a middle-high class app with a lot of states, fetching to the server, displaying other things...

Thread Thread
 
sergey profile image
sergeyv

For a "real app" the difference will likely to be much less, exactly for the reason that it does many other expensive things. The test app does almost nothing but creating hundreds of thousands of widgets, so the difference is exaggerated.

A real-life example: Imagine you have cheap nails for 1 cent each and more expensive at 10c each. A ton of cheap nails would cost, say, $10K and a ton of the expensive ones will be $100K, which is a huge difference.

But if you use the nails to build some nice furniture - you only need a few dollars worth of nails in either case and the cost of the nails in the final product's price will be minuscule in either case and maybe some other considerations may become more important.

Collapse
 
codyseibert profile image
Cody Seibert

Do you happen to have benchmarks that show how much this really affects performance?

Collapse
 
ucvdesh profile image
Daniel Silva

Hmmm, don't actually have a documented benchmark. But I've been seeing this a lot through multiple projects I worked on, specially on forms.

But I think is a great idea to add benchmarks, thanks! I will do it for sure!

Collapse
 
codyseibert profile image
Cody Seibert • Edited

Just wondering. I’ve never returned null in my react apps. I always just do {myBoolean && MyComponent()}. I can’t insert carot symbols but you get the point.

Thread Thread
 
ucvdesh profile image
Daniel Silva

Exactly! This is a correct conditional render, because MyComponent does not render unless myBoolean is true

Collapse
 
ucvdesh profile image
Daniel Silva

That's why I said that may potencially affect performance of the apps. I mean, if you have something really small or something that won't be changing a lot in the short term, you won't notice it, but still is something to work on because it's not a good practice tho...

Collapse
 
jackmellis profile image
Jack • Edited

While I agree in principle with your point - not rendering a component is preferable to rendering a null component - the reasoning is not to do with performance. React components are super cheap to instantiate, like super super cheap.
The reason you should avoid doing this is because you're not in control of the rendering behaviour of your child component, it means your pages not lay out in the way you expect it to, and it is harder to add more conditionals at the top level.
On the other hand however, there are some very valid reasons to render null. If the logic required to determine the show boolean is really tightly coupled to the rest of the child component's behaviour, it seems silly to have to calculate it twice. Also sometimes you want the child component to be totally encapsulated, sometimes you want to drop a component down and you dont care what it does under the hood, the implementation detail shouldn't be leaked if you can help it.
Finally I think your examples aren't quite right. show ?? <MyChildComponent/> will render the child when show is null, otherwise it will render the value of show...

Collapse
 
ucvdesh profile image
Daniel Silva

Well, you're right with the cheapness of React components but you're missing the re-rendering of every single on of the components. If the parent component have a state that changes a lot, your ChildComponent will re-render too, now let's imagine that we have something like a form component that on every keypress will set a state, that will make you loose performance...the same happen if you have something like a graph that will set a lot of states...

On the second pointer, you can have a null validation on the conditional rendering but not on a render from a component, because you will have a useless lifecycle running, so there's no reason why a return (null) will be valid. Although, I did a little disclosure about sometimes you will have something like a wrapper that will not be showing but may still be needed, but still that wrapper will not return just NULL, at least children to be there...

On the error, yeah thanks! I totally miss that! I just fix it but not because the validation wasn't right but because the validation was returning false not null (also that's no good, but the example was explained it with null)

Collapse
 
ucvdesh profile image
Daniel Silva

It's a common practice but not a good practice, tho. And could have a little impact on performance depending on what you're doing, that's why I told that may potentially impact your performance. When managing big apps with a lot of functionalities, those extra re-renders will drop the performance of the app.

You can try it by using a ChildComponent and put a console.log('re-render') and you will see the multiple logs on the console. Now, imagine that you have it on a component like a form, that will set a new state on every key press and you have just 3 component that return null or false...It will do a lot of re-rendering just for 3 components with a bad conditional rendering.

Collapse
 
ivan_jrmc profile image
Ivan Jeremic • Edited

I return just an empty Fragment instead of null like this <></>

Collapse
 
loopmode profile image
Jovica Aleksic

Interesting. Besides the point of the author tho. He argues we shouldn't do that either - because the component will still go through lifecycle before returning empty fragment. I'd love to know perf difference to returning null tho. My feeling is that it must be. But slower tho.

Collapse
 
ivan_jrmc profile image
Ivan Jeremic • Edited

I'n not sure if Fragment also triggers a full lifecycle would be interesting to look into this a bit more.

Thread Thread
 
loopmode profile image
Jovica Aleksic

I guess so. In the end, render is performed and returns something. And execution of the render is part of the lifecycle.
The point is that react must first evaluate the empty fragment to figure out "it's nothing" as opposed it was filled with children.

Thread Thread
 
ivan_jrmc profile image
Ivan Jeremic

Yeah thinking more about it that makes sense. Need to go back and optimize 😫

Thread Thread
 
ucvdesh profile image
Daniel Silva

Yeah bro, a empty fragment also trigger a full lifecycle because of the render of the fragment, as soon as you have it, then the component exist ergo, have its own lifecycle

 
ucvdesh profile image
Daniel Silva

The first part is good is we do condition && , that's the right way. And the post was mean to talk about a separate component that return null, because that will trigger a whole lifecycle that won't be doing anything.

Collapse
 
lffate profile image
LFFATE

I think It's a messy way and it violates SRP (also an implementation hiding). Components are self-sufficient modules. What if I should check browser features? Do some calculations? Request api for determine should I render a component content? It's cannot be done on a higher level. it will cause a huge performance penalty when render conditions will trigger sibling code to execute.

Collapse
 
sevenzark profile image
Dave M

While this would make sense in a very small app, it seems as if it would make using data stores such as Redux or Mobx impossible. Components could not check the store to see if they have the data they need in order to render, but instead would be completely dependent, always, on props passed from a parent. This would require making the majority of components 'dumb' components that only render and do no logic of their own. In an enterprise-scale app this kind of prop drilling gets out of hand very quickly. It can also be difficult to maintain the codebase if I must always manually trace prop threads backwards through who knows how many layers, until I finally find where a value is being supplied or calculated.

Collapse
 
hayk1997 profile image
Hayk-1997

Hi :)).
Thanks to an article.
I have error when I'm using typescript with react.

{Its return type 'false | Element' is not a valid JSX element.
Type 'false' is not assignable to type 'Element | null'.}
This error appears when I return {false} instead of null

Collapse
 
asyrialak profile image
Andrew Syriala

When focusing on perf in a React app, I'd start by running a lighthouse audit, and seeing what your areas of improvement are. Address those, vs looking for micro-improvements than don't have a measurable impact.

Having the parent unnecessarily in charge of a child rendering can make it more difficult for a reader of your code to track down the state of the components. I also have never seen performance impacts (in a measurable way, which would make me consider refactoring) of having components returning null.

One thing being discounted here is readability. Lets assume you're working on a project with other developers, and your code gets handed off, what does this decision do to readability?

When concerns are separated, and components handle a single responsibility, they are definitely easier to debug. Think of it this way, if I am inheriting your code, and I am tracking down an issue related to a component, I am now going to have to touch more files to understand the state of the application. However, if components contain their own logic, and we've separated our concerns, then it's a one and done.

While you could establish patterns of having orchestration components that only manage conditional rendering, I think this design decision could be premature optimization at the cost of readability by sacrificing a separation of concerns.

 
ucvdesh profile image
Daniel Silva

Kinda hard not to have any conditional render on an app as we have different profiles, permissions and states. So as we can manage some ways to avoid some of those, we still gotta use good practices as we develop

Collapse
 
bharati21 profile image
Bharati Subramanian

Good article!
Wanted to know if there's an actual way to measure the performance when you use nullish coalescing/ &&/ return null ?

Also, I wonder why the docs have mentioned returning null, when it seems like a bad practice. Do you think it has any practical use cases?

Collapse
 
vikkio88 profile image
Vincenzo

I dont know if it is just me but this sentence

That is a mistake that can potentially increase a lot the performance of your application.

seems to mean exactly the opposite of what you mean?

Collapse
 
ucvdesh profile image
Daniel Silva

hahahahaha you're right! Sorry for the mistake!