Redux: Это считается мутирующим состоянием?

Question

Использование Redux в Angular2 (NgRedux) для обновления/настройки записи заказа. Является ли это правильным для изменения прокси-переменной, а затем object.assign его в состояние?

В моем редукторе данных:

case DataActions.UPDATE_ORDER: 
    var updatedItem 
    var updatedState = state.orders 
    for(var i = 0; i < state.orders.length; ++i) { 
    if(state.orders[i]['key'] === action.payload.key) { 
     updatedItem = state.orders[i] 
     updatedItem.name = action.payload.name 
     updatedItem.items = action.payload.items 
    } 
    } 
    updatedState[i] = updatedItem 
    return Object.assign({}, state, { 
    orders: updatedState 
    }) 

Answer 1

Ваш «прокси-переменная» только одна ссылка на тот же объект, так что да, вы являются непосредственно мутирует элементов в этом массиве.

Для правильного внесения неизменных обновлений каждый уровень вложенности должен быть скопирован. В документах Redux приводятся примеры как распространенных ошибок, так и того, как это сделать правильно, в http://redux.js.org/docs/recipes/reducers/ImmutableUpdatePatterns.html.

Answer 2

Большое спасибо @markerikson за решение.

Я пошел с советами по документации и использовал библиотеку dotProp. Это сработало:

npm install dot-prop-immutable 
... 
var dotProp = require('dot-prop-immutable'); 
... 
case DataActions.UPDATE_ORDER: 
    for(var i = 0; i < state.orders.length; ++i) { 
    if(state.orders[i]['key'] === action.payload.key) { 
     state = dotProp.set(state, `orders.${i}.name`, action.payload.name) 
     state = dotProp.set(state, `orders.${i}.items`, action.payload.items) 
    } 
    } 
    return state