Solution #2865

Neison32 · 2023-10-17T04:06:32Z

No description provided.

etojeDenys · 2023-10-17T06:51:48Z

src/transformStateWithClones.js

 }

+function transformStateWithClones(state, actions) {
+  const stateHistory = [cloneObject(state)];


Suggested change

const stateHistory = [cloneObject(state)];

const stateHistory = [];

and we can remove the slice at all

etojeDenys · 2023-10-17T06:52:10Z

src/transformStateWithClones.js

+
+    if (action.type === 'addProperties') {
+      // Add properties to the state
+      Object.assign(newState, action.extraData);
+    } else if (action.type === 'removeProperties') {
+      // Remove specified properties from the state
+      for (const key of action.keysToRemove) {
+        delete newState[key];
+      }
+    } else if (action.type === 'clear') {
+      // Create an empty state
+      newState = {};
+    }
+


let's use switch case instead of the multiple if else

Solution

32cdcab

etojeDenys suggested changes Oct 17, 2023

View reviewed changes

Neison32 closed this by deleting the head repository Sep 25, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Solution #2865

Solution #2865

Neison32 commented Oct 17, 2023

etojeDenys Oct 17, 2023

etojeDenys Oct 17, 2023

	const stateHistory = [cloneObject(state)];
	const stateHistory = [];

Solution #2865

Solution #2865

Conversation

Neison32 commented Oct 17, 2023

etojeDenys Oct 17, 2023

Choose a reason for hiding this comment

etojeDenys Oct 17, 2023

Choose a reason for hiding this comment