refactor: rename createFactory to functionFactory and improve memoization logic

Alejandro CR · Alejandro CR · commit 9f8522de103f · 2025-05-01T21:10:40.000-05:00
diff --git a/examples/browsers/libraries/reactjs/maker-fn.js b/examples/browsers/libraries/reactjs/maker-fn.js
@@ -5,15 +5,19 @@ const { createElement: ce, useState } = React;
 
 ReactDOMClient.createRoot(document.getElementById('root')).render(ce(App));
 
-function createFactory(fn) {
+/**
+ * @param {Function} fn - The function to be memoized.
+ * @returns {Function} - A memoized version of the function.
+ */
+function functionFactory(fn) {
   let fnCache = null;
   let depsCache = [];
 
   return (...args) => {
     if (
       fnCache === null ||
       depsCache.length !== args.length ||
-      args.some((dep, i) => dep !== depsCache[i])
+      !args.every((dep, i) => Object.is(dep, depsCache[i]))
     ) {
       fnCache = fn.apply(undefined, args);
       depsCache = args;
@@ -23,14 +27,14 @@ function createFactory(fn) {
   };
 }
 
-const OnClickFactory = createFactory(setCount => _event => setCount(prev => prev + 1));
+const onClickFactory = functionFactory(setCount => _event => setCount(prev => prev + 1));
 
 const stack = [];
 
 function App() {
   const [count, setCount] = useState(0);
 
-  const onClick = OnClickFactory(setCount);
+  const onClick = onClickFactory(setCount);
 
   stack.push(onClick);
 
