Skip to content

Instantly share code, notes, and snippets.

@coryhouse

coryhouse/conciseObjectSpreadExample.js

Last active July 17, 2017 17:11
Show Gist options
  • Save coryhouse/9bd989ec8f89e3fcf058129b67a509ad to your computer and use it in GitHub Desktop.
Save coryhouse/9bd989ec8f89e3fcf058129b67a509ad to your computer and use it in GitHub Desktop.
Download ZIP
Consise example of object spread in React
Raw
conciseObjectSpreadExample.js
updateState({target}) {
this.setState({user: {...this.state.user, [target.name]: target.value}});
}
@donavon
Copy link

donavon commented Jun 14, 2017

but doesn't this.state.user break your rule then?

Yes it does, which is why I added "I know that it's not always avoidable". 'name and value are no brainers, getting the current user from state is less so. I'm always struggling with my devs to write Clean Code, but know "when to say when". I think the right answer is somewhere on this page, but I'm not sure which one it is. ;)

I'd be happy to take a look and give you my feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment