Should I use arrow functions or class function style
This may be a very stupid question. But I was wondering if I should be using arrow function syntax or the classic function syntax for react components now or is this purely a style choice. I ask this purely as someone trying to work towards industry standards but have found a tremendous amount of mixed comments on it. Also is there any difference using typescript?
Other than the syntax, there is a very important functional difference between the two: function definitions are hoisted, consts are not. What this means in practice is that you can use a function you define later in the file if it's defined using the function f() { ... } syntax, but const f = () => { ... } functions can only be used after their definitions.
function f() { g() } // OK
function g() {}
const f = () => { g() } // Error
const g = () => {}
Personally, I like breaking up React components into smaller helper subcomponents and use them in a main component. I only export the main component, the helpers are private to the module. For better readability, I like the main component to be at the top of the file and then put the helpers in decreasing order of complexity. This style is only possible with classic function definitions, using consts forces you to use bottom-up instead of top-down order.
Dan Abramov made a convincing (to me) argument for using the named function style. Stack traces will give a more immediately useful information.
It’s not a deal breaker, so in a work context I would tend towards following the existing style as some people are really attached to anonymous functions
No diff with typescript, tho declared is more flexible because you can overload it. Declared (classic) functions are hoisted and can be used prior to their declaration. Not sure on perf differences
I was doing some research and it seems like arrow function preserves 'this' context. Might be a reason to either classic or arrow. The overloading seems like a valuable option too.