void over curly braces
Gunar Gessner
Apr 1, 2021
I disencourage returning values that aren't supposed to be returned:
// Don't do this onCopy={() => setCopied(true)}
The function above returns the result of setCopied(), which in the case of React is most likely undefined and shouldn't create problems. However, what if the code behind onCopy changes to expect certain things to be returned?
Better to only return expected values:
onCopy={() => { setCopied(true); }}
The code above explicitly returns undefined, even if setCopied ever changes.
And you can get rid of the curly braces by using the void keyword:
onCopy={() => void setCopied(true)}