Friday, May 10, 2024
 Popular · Latest · Hot · Upcoming
153
rated 0 times [  155] [ 2]  / answers: 1 / hits: 38464  / 9 Years ago, thu, october 15, 2015, 12:00:00

Bear with me here as this question pertains to my first test app using either React, Redux or react-redux. Docs have gotten me far and I have a mock banking app that mostly works. My state object looks roughly like this:



{
activePageId: checking,
accounts: [
checking: {
balance: 123,
transactions: [
{date, amount, description, balance}
]
}
]
}


I have just two actions:



1. CHANGE_HASH (as in url hash). This action always works as expected and all the reducer does is update the state.activePageId (yes, I'm cloning the state object and not modifying it). After the action, I can see the state has changed in the Redux store and I can see that React has updated.



function changeHash(id) {
return {
type: CHANGE_HASH,
id: id
}
}


2. ADD_TRANSACTION (form submission). This action never updates React, but it always updates the Redux store. The reducer for this action is updating state.accounts[0].balance and it's adding a transaction object to the array state.accounts[0].transactions. I don't receive any errors, React just doesn't update. HOWEVER, if I dispatch a CHANGE_HASH action React will catch up and display all of the ADD_TRANSACTION state updates properly.



function addTransaction(transaction, balance, account) {
return {
type: ADD_TRANSACTION,
payload: {
transaction: transaction,
balance: balance,
account: account
}
}
}


My reducer...



    function bankApp(state, action) {
switch(action.type) {
case CHANGE_HASH:
return Object.assign({}, state, {
activePageId: action.id
});
case ADD_TRANSACTION:
// get a ref to the account
for (var i = 0; i < state.accounts.length; i++) {
if (state.accounts[i].name == action.payload.account) {
var accountIndex = i;
break;
}
}

// is something wrong?
if (accountIndex == undefined) {
console.error(could not determine account for transaction);
return state;
}

// clone the state
var newState = Object.assign({}, state);

// add the new transaction
newState.accounts[accountIndex].transactions.unshift(action.payload.transaction);

// update account balance
newState.accounts[accountIndex].balance = action.payload.balance;

return newState;
default:
return state;
}


My mapStateToProps



    function select(state) {
return state;
}


What am I missing here? I'm under the impression that React is supposed to update as the Redux storeis updated.



Github repo:



Deployment bank demo



p.s. I lied about not having any errors. I do have a number of warnings




Warning: Each child in an array or iterator should have a unique key prop...




I'm already giving them a key prop set to it's index. I doubt that has anything to do with my issue though.


More From » reactjs

 Answers
32

The problem is in this piece of code:



// clone the state
var newState = Object.assign({}, state);

// add the new transaction

newState.accounts[accountIndex].transactions.unshift(action.payload.transaction);

// update account balance
newState.accounts[accountIndex].balance = action.payload.balance;


Cloning the state object doesn't mean you can mutate the objects it is referring to. I suggest you to read more about immutability because this isn't how it works.



This problem and solution to it are described in detail in Redux “Troubleshooting” docs so I suggest you to read them.



https://redux.js.org/troubleshooting



I also suggest you to take a look at Shopping Card example in Flux Comparison for Redux because it shows how to update nested objects without mutating them in a similar way to what you are asking.



https://github.com/voronianski/flux-comparison/tree/master/redux


[#64733] Monday, October 12, 2015, 9 Years  [reply] [flag answer]
Only authorized users can answer the question. Please sign in first, or register a free account.
kaitlynnb

Total Points: 402
Total Questions: 96
Total Answers: 109

Location: Trinidad and Tobago
Member since Fri, May 8, 2020
4 Years ago
kaitlynnb questions
;