Is it possible to write that `useLeaderstats` hook better than now?

It's just that I'm not that good with react.
export function useLeaderstats(player: Player) {
const [leaderstats, setLeaderstats] = useState<Leaderstats>()

const listener = () => {
const leaderstats = player.FindFirstChild("leaderstats")
if (!isLeaderstats(leaderstats)) return
setLeaderstats(leaderstats)
}
useMountEffect(listener)
useEventListener(player.ChildAdded, listener)

return leaderstats
}
export function useLeaderstats(player: Player) {
const [leaderstats, setLeaderstats] = useState<Leaderstats>()

const listener = () => {
const leaderstats = player.FindFirstChild("leaderstats")
if (!isLeaderstats(leaderstats)) return
setLeaderstats(leaderstats)
}
useMountEffect(listener)
useEventListener(player.ChildAdded, listener)

return leaderstats
}
useMountEffect and useEventListener is from pretty-react-hooks
2 Replies
Tester
Tester5mo ago
why do you want this hook? i mean, usually leaderstats are only visual and it's better not to use them for data storing
Mirrox
MirroxOP5mo ago
true, I just don't feel like replicating other's people state

Did you find this page helpful?