DEV Community

loading...

Discussion on: 😲🤯The most outstanding new feature in Javascript you need to know about: Optional Chaining

chrisachard profile image
Chris Achard

Right, yep - if you're constantly checking the same thing for null or not, then maybe you need to abstract that to give you a better framework/way to access it.

Thread Thread
lampewebdev profile image
Michael "lampe" Lazarski Author

Like always thinking is the key here 😉

Thread Thread
qm3ster profile image
Mihail Malo

IMO it lends itself to patterns like:

useWidth(this.props.data?.myObject?.width||100)
alsoUseWidth(this.props.data?.myObject?.width||222)

Not only are you doing extra branching, but by having defaults at the usage site instead of defaults site you could have inconsistencies.

Instead, on construction of the this, you could have

this.data=_.merge(defaultData, incomingData)

and then access all the properties unconditionally.
(You could also only merge in properties that already exist in defaultData tree, and perform validation at this stage, creating a very predictable this)