yv
да, мы про разные вещи
Size: a a a
2020 June 08
AO
кароче не используй fx в useEffect реакта
Можно более радикальную позицию принять, не держать эффекты в качестве публичного апи
🦜
Aleksandr Osipov
Можно более радикальную позицию принять, не держать эффекты в качестве публичного апи
+
🦜
у меня так и есть
🦜
я раньше думал
Оо, это меньше юнитов, можно эффект сюда, эффект туда
Оо, это меньше юнитов, можно эффект сюда, эффект туда
🦜
все работает
DS
если хочешь что-то сделать на маунт компонента используй ивент или гейт
Дак че случится я чет и не понял толком
🦜
все работает
но это неудобно
🦜
Дак че случится я чет и не понял толком
кто не понял, тот поймет
DS
кто не понял, тот поймет
😒
P
Aleksandr Osipov
Можно более радикальную позицию принять, не держать эффекты в качестве публичного апи
эм, так как лучше то? )
P
я сейчас наоборот, вытаскиваю, меньше юнитов…
AO
эм, так как лучше то? )
Каждый решает сам, я за то чтобы в публичном апи были только сторы и события
P
только ради этого с редакса перехожу, а то писать эвенты на каждый эффект это тоже самое что писать action creator-ы по бойлерплейту
🦜
только ради этого с редакса перехожу, а то писать эвенты на каждый эффект это тоже самое что писать action creator-ы по бойлерплейту
не на каждый
AO
только ради этого с редакса перехожу, а то писать эвенты на каждый эффект это тоже самое что писать action creator-ы по бойлерплейту
Не слушайте никого, делайте как удобно, позже решите походит выбранный вариант вам или нет
R
я про
fx = createEffect({handler: () => await fetch()})
useEffect(() => {fx()})
fx = createEffect({handler: () => await fetch()})
useEffect(() => {fx()})
не вижу ошибок, а вот если раскомментить reject, то валится
https://codesandbox.io/s/admiring-christian-xm8vz?file=/src/index.js
https://codesandbox.io/s/admiring-christian-xm8vz?file=/src/index.js
🦜
не вижу ошибок, а вот если раскомментить reject, то валится
https://codesandbox.io/s/admiring-christian-xm8vz?file=/src/index.js
https://codesandbox.io/s/admiring-christian-xm8vz?file=/src/index.js
ошибок не будет, будет просто в каждом вызове useEffect новый промис
P
Aleksandr Osipov
Не слушайте никого, делайте как удобно, позже решите походит выбранный вариант вам или нет
я изначально как переписывать начал и добавлял эвенты, которые триггерят эффекты, но сейчас выпиливаю
P
так что пожалуй пока оставлю эффекты публичным апи