I wonder if I did something wrong because as far as I know: Object.assign and spread operator will create new object, therefore we can avoid mutate the internal state with is a bad practice. But in my project those two don't seem to work.
First, in my constructor, I setup the state as below:
this.state = {
article: {
title: "",
alias: "",
category: "JAVA",
steps: [
{
stepId: 1,
title: "Dummy title",
description: "The quick brown fox jump over the lazy dog",
length: 30
}
]
},
description: "",
newStep: {
title: "",
length: 0
}
}
Later, I add an addStep() function:
addStep() {
console.log(this.state.article.steps);
let article = { ...this.state.article };
article.steps.push({
stepId: _.last(this.state.article.steps).stepId + 1,
title: this.state.newStep.title,
length: this.state.newStep.length,
description: "",
});
console.log(this.state.article.steps);
}
And here the result:
As you can see this.state.article.steps has been mutated. The same goes for Object.assign. Finally I decide to use and that solved my problem.
let article = JSON.parse(JSON.stringify(this.state.article));
Result:
Object.assign{}
and object spread won't deep clone an object.
It will mutate steps
.
It is discussed in the following thread:
How do I correctly clone a JavaScript object?
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With