Skip to content

Instantly share code, notes, and snippets.

@dooman87
Last active November 26, 2018 10:39
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save dooman87/8e80bb266abf2d3e84ab7c7c7a156a0e to your computer and use it in GitHub Desktop.
Save dooman87/8e80bb266abf2d3e84ab7c7c7a156a0e to your computer and use it in GitHub Desktop.
FIX: TS2454: Variable 'newState' is used before being assigned.
export function apiKey(state: ApiKeyState, action: BaseAction) : ApiKeyState {
//Assigning variable here.
let newState: ApiKeyState = state;
switch (action.type) {
case 'ADD_API_KEY':
newState = Object.assign({}, state);
newState.data.apiKeys = [
...state.data.apiKeys,
action.payload.Key
] as string[];
break;
//More actions handling here
//...
}
//Using only newState in return.
return newState ? newState : API_KEY_INITIAL_STATE;
}
@jawys
Copy link

jawys commented Nov 26, 2018

You could even shorten line 18 to the following:

return newState || API_KEY_INITIAL_STATE;

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