redux reducer return 'defaultState'は安全ですか？

Question

質問リダンダントでdefaultStateを返すことは安全ですか？安全でない場合は、状態が突然変異する可能性がある場合、私に例を挙げてください。

const defaultState = { 
    test: 'test 
} 

function testReducer(state, action) { 
    switch action.type 
     case RETURN_DEFAULT_STATE: 
     return defaultState; // is this ok to do? or need {...defaultState} 
    default: 
     return state; 
} 

Answer 1

あなたはどこかdefaultStateを変異していないと、それはあなたが望む新しい状態を反映している場合、私はこの問題がどうなるかが表示されません。

あなたの例のコードに変更なしで：ここで

はdefaultStateが考えられる突然変異させることができる方法のいくつかの例がありますが、とにかく、すでにこれを実行すべきではない

store.dispatch({type: "RETURN_DEFAULT_STATE"}); 
store.getState().x = "x"; 

注意が問題を抱えていますあなたがいる場合。

今、あなたの例のコードの修正版：あなたがある場合

export const defaultState = { 
    test: 'test 
} 

function testReducer(state, action) { 
    switch action.type 
     case RETURN_DEFAULT_STATE: 
     return defaultState; // is this ok to do? or need {...defaultState} 
    default: 
     return someOtherReducer(state, action); 
} 

function someOtherReducer(state, action) { 
    state.x = "x"; 
    return state; 
} 

注すでにsomeOtherReducer()はとにかくで何をしてとすべきでないという問題があります。

import {defaultState} from "whatever"; 

defaultState.x = "x";  

store.dispatch({type: "RETURN_DEFAULT_STATE"}); 
store.dispatch({type: "UNKNOWN"}); 

Answer 2

それはとても長いコードを書くすべての開発者が入力パラメータを変異しないように接着しても安全です。私はまだ偶然の突然変異を防ぐためにObject.assignを行うことを提案します。

不良コードサンプル：

const defaultState = { 
    test: 'test' 
}; 

function testReducer(state, action) { 
    switch action.type 
     case RETURN_DEFAULT_STATE: 
     return defaultState; // is this ok to do? or need {...defaultState} 
     case DO_MALICIOS_MUTATION: 
     state.test='foo'; // this might potentially mutate original defaultState object 
     return defaultState; 
    default: 
     return state; 
} 

Answer 3

それは限り、あなたはどこにでもそれを変異していないと問題になることはありません。しかし、私はオブジェクトが偶然によって突然変異したために起こったバグを経験したことがあります。したがって、私はあなたの状態がデフォルトの状態オブジェクトをフリーズすることができ変異していないことを確認したい場合はお勧めします：

const defaultState = Object.freeze({ 
    test: 'test 
}); 

あなたは例外を取得strictモードで、このような凍結されたオブジェクトを変更しようとします。