GB
Т.е. что мне, так написать?
[state.selection, ...state.selection.map(i=>state.items[i])]
Size: a a a
2020 July 28
GB
И что при этом будет реакт делать?
AK
поставь себе плагин для еслинта на хуки, Дэн Абрамов говорил что нельзя массивы как зависимости указывать, реакт будет каждый раз твои массивы сравнивать
GB
поставь себе плагин для еслинта на хуки, Дэн Абрамов говорил что нельзя массивы как зависимости указывать, реакт будет каждый раз твои массивы сравнивать
ЧТО? А без ссылки на авторитеты, доказать сможете? )
GB
И у меня стоит плагин
GB
В этом и проблема, я писал выше: "И да, я знаю, что можно убрать зависимость от state.items, но тогда хваленые правила для хуков для eslint ругаются"
AK
зачем мне тебе что то доказывать?
GB
зачем мне тебе что то доказывать?
Я задал вопрос, если по делу посоветовать нечего, то ок. Я просто сразу это не понял. Думал, вы можете что-то подсказать )
DT
поставь себе плагин для еслинта на хуки, Дэн Абрамов говорил что нельзя массивы как зависимости указывать, реакт будет каждый раз твои массивы сравнивать
ссылки сравнит и только
AK
Т.е. что мне, так написать?
[state.selection, ...state.selection.map(i=>state.items[i])]
ты можешь написать функцию, которая будет сравнивать массивы, и записывать\возвращать куда-то тру или фолс, и ты это можешь в зависимости прокидывать, если нужно именно на изменения массива по длине смотреть, можешь написать [items.length]
BA
поставь себе плагин для еслинта на хуки, Дэн Абрамов говорил что нельзя массивы как зависимости указывать, реакт будет каждый раз твои массивы сравнивать
он сделал не верно, ему плагин и покажет массив совать туда
GB
ты можешь написать функцию, которая будет сравнивать массивы, и записывать\возвращать куда-то тру или фолс, и ты это можешь в зависимости прокидывать, если нужно именно на изменения массива по длине смотреть, можешь написать [items.length]
Я думал, что я действительно могу вытащить зависимости динамически. Но к сожалению react ругается, когда массив зависимостей динамически меняется.
const onSelectProp = props.onSelect;
const selected = useMemo(() => state.selection.map(i => state.items[i]), [
state.selection,
state.items,
]);
useEffect(() => {
onSelectProp(selected);
}, [onSelectProp, ...selected]);AK
если у тебя работает щас, задизабли строчку зависимостей и всё)
AK
ну, что бы плагин не орал
GB
если у тебя работает щас, задизабли строчку зависимостей и всё)
Тогда eslint ругается, потому и вопрос был, как это "красиво" сделать )
GB
Ну и понятно почему eslint ругается, это действительно не совсем корректно убирать целиком зависимость на items
AK
плагин иногда в зависимости сам предлагает поставить не корректные вещи)
BA
плагин иногда в зависимости сам предлагает поставить не корректные вещи)
он верно предлагает, используешь в хуке - суй в зависимости, глобальны епеременные не нужно
AK
типо говорит поставь в депы функцию, которая внутри и выполняется, если ее ставишь, у тебя циклически все обновляется постоянно)))0
GB
он верно предлагает, используешь в хуке - суй в зависимости, глобальны епеременные не нужно
Тогда как мне сунуть в зависимости динамический набор объектов? )