Skip to content

Conversation

@mpw
Copy link

@mpw mpw commented Jan 2, 2019

The mapping from the state/model is not a pure function, so remove the word "pure".

https://www.reddit.com/r/reactjs/comments/abj7hn/uis_are_not_pure_functions_of_the_model_reactjs/ed1h20x/

"To elaborate a bit, React components aren’t always “pure” in strict FP sense of the word. They’re also not always functions (although we’re adding a stateful function API as a preferred alternative to classes soon). Support for local state and side effects is absolutely a core feature of React components and not something we avoid for “purity”.
I admit it’s confusing to see it described as “pure function” and I suggest you to read the documentation (https://reactjs.org) to get a sense of how React components tend to look."

mpw added 2 commits January 2, 2019 15:48
https://www.reddit.com/r/reactjs/comments/abj7hn/uis_are_not_pure_functions_of_the_model_reactjs/ed1h20x/

"To elaborate a bit, React components aren’t always “pure” in strict FP sense of the word. They’re also not always functions (although we’re adding a stateful function API as a preferred alternative to classes soon). Support for local state and side effects is absolutely a core feature of React components and not something we avoid for “purity"
Update Readme:  remove "pure" from pure function
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant