(no title)
shroompasta | 3 years ago
yeah, making it into a custom hook lol.
When did something that was originally worth 5 LOCs turn into 20 LOCs
shroompasta | 3 years ago
yeah, making it into a custom hook lol.
When did something that was originally worth 5 LOCs turn into 20 LOCs
javcasas|3 years ago
When you want it to work properly in all the cases. That includes when new data is being fetched and you don't want to show the old data. Or when you want to not refetch data if the conditions for refetching are not met. Or when you want to cancel an in-progress data fetch if the data shall no longer be shown after fetching it because the user closed the component tasked with showing the data.
I bet your 5 LOC can't do some of these things.
shroompasta|3 years ago
This only really happens when you have race conditions
> Or when you want to not refetch data if the conditions for refetching are not met.
This only happens when you fetch based on state update on the same component
> Or when you want to cancel an in-progress data fetch if the data shall no longer be shown after fetching it because the user closed the component tasked with showing the data.
you can do this with certain conditionals and a cleanup function
over 3 quarters of my data fetching is just a simple fetch on onComponentDidMount
do you mean to tell me I have to bring in a library just for that?