i2gor87/baguni

Lightest react package for drop-in replacement of useState

Hi 👋, I'm Igor Hwang

Multilingual problem-solver from Korea

• 🔭 I’m currently working on baguni

• 🌱 I’m currently learning System Architecture

• 💬 Ask me about panini, fastapi

• 📫 Reach me at [email protected]

Connect with me:

Languages and Tools:

baguni's Issues

vas3k review

👋

baguni/src/index.js

Line 30 in bbab8bb

const reRender = () => set({});

React hook FAQ советует использовать следующий подход:

const [, reRender] = useReducer(x => x + 1, 0);

baguni/src/index.js

Lines 30 to 36 in bbab8bb

	const reRender = () => set({});
	useEffect(() => {
	bState.joinReRender(reRender);
	return () => {
	bState.cancelReRender(reRender);
	};
	}, [bState]);

Функция reRender создаётся новая на каждый рендер, а хук useEffect отрабатывает только на изменение bState, в результате чего в cancelReRender попадает другая функция reRender, чем была добавлена в joinReRender, то есть старые функции не удаляются, что вызывает утечку памяти. Если заюзать подход из п.1, проблема должна уйти, т.к. функция будет стабильная.

baguni/src/index.js

Lines 37 to 39 in bbab8bb

	function setState(newState) {
	bState.set(newState);
	}

Функция setState создаётся новая на каждый рендер, это поведение не аналогично хуку useState, где функция стабильная. Это может вызвать проблемы с рендерами, если пользователь не будет учитывать этот момент.

baguni/src/index.js

Line 24 in bbab8bb

return Object.freeze(Object.create(prototype));

Создание объекта через Object.freeze(Object.create(...)) выглядит как ненужное переусложнение на мой взгляд. Это выглядит как попытка защитить объект от изменений вручную, но такая «защита» без проблем обходится

PS
Я не пробовал код в работе, это просто визуальное ревью.
Замечу, что написание стейт менеджера это очень сложная задача, гораздо сложнее, чем кажется на первый взгляд.