Solution #2738
Solution #2738
Conversation
src/transformState.js
Outdated
| for (const action of actions) { | ||
| switch (action.type) { | ||
| case 'addProperties': | ||
| for (const data in action.extraData) { |
There was a problem hiding this comment.
The variable 'data' does not provide clarity on what it holds. Consider renaming it to something more meaningful like 'property'.
src/transformState.js
Outdated
| } | ||
| break; | ||
| case 'removeProperties': | ||
| for (const key of action.keysToRemove) { |
There was a problem hiding this comment.
The variable 'key' does not provide clarity on what it holds. Consider renaming it to something more meaningful like 'propertyName'.
| @@ -5,7 +5,27 @@ | |||
| * @param {Object[]} actions | |||
| */ | |||
| function transformState(state, actions) { | |||
| // write code here | |||
There was a problem hiding this comment.
Add a brief comment explaining what the function does, to improve code readability.
src/transformState.js
Outdated
| } | ||
| break; | ||
| case 'clear': | ||
| for (const key in state) { |
There was a problem hiding this comment.
The variable 'key' does not provide clarity on what it holds. Consider renaming it to something more meaningful like 'propertyName'.
src/transformState.js
Outdated
| switch (action.type) { | ||
| case 'addProperties': | ||
| for (const data in action.extraData) { | ||
| const prop = action.extraData[data]; |
There was a problem hiding this comment.
The variable 'prop' does not provide clarity on what it holds. Consider renaming it to something more meaningful like 'value'.
No description provided.