Size: a a a

2020 July 22

AO

Aleksandr Osipov in ☄️ effector
Поэтому в эффекторе везде опции идут обьектом так сказать с заделом на будущее
источник

m

makoven in ☄️ effector
Aleksandr Osipov
Ну вот когда параметров больше становится то сталкиваемся что такой подход нерасширяем
Но их не больше 3 в ваших апи
источник

AO

Aleksandr Osipov in ☄️ effector
источник

AO

Aleksandr Osipov in ☄️ effector
С течением времени там появлялись новые опциональные параметры
источник

AO

Aleksandr Osipov in ☄️ effector
И возможно ещё будут
источник

AO

Aleksandr Osipov in ☄️ effector
Если бы изначально там не использовался объектный синтаксис для опциональных параметров, то это было бы просто не масштабируемо
источник

AO

Aleksandr Osipov in ☄️ effector
makoven
Но их не больше 3 в ваших апи
Не всегда известно заранее как будет расти апи
источник

m

makoven in ☄️ effector
Aleksandr Osipov
С течением времени там появлялись новые опциональные параметры
Всякие скучные параметры можно сваливать в объект options последним аргументом. Все-равно они не используются в 99% случаев
источник

BB

Bugs Bunny in ☄️ effector
ну вот такой же логикой руководствовались в openapi-generator и получилось не очень
источник

BB

Bugs Bunny in ☄️ effector
мягко говоря
источник

BB

Bugs Bunny in ☄️ effector
vsc еще аргументы никак не именует и там просто подряд портянка из параметров
источник

m

makoven in ☄️ effector
Bugs Bunny
ну вот такой же логикой руководствовались в openapi-generator и получилось не очень
Не только. В ноде, в стандартной библиотеке встречается. Нормально
источник

ei

export default - зло... in ☄️ effector
Жесть, как можно оправдывать лапшу из аргументов
источник

AO

Aleksandr Osipov in ☄️ effector
makoven
Всякие скучные параметры можно сваливать в объект options последним аргументом. Все-равно они не используются в 99% случаев
ну а вслучае guard sample именованные параметры просто напросто улучшают читаемость кода на мой взгляд, потому что не надо помнить о порядке и сразу видно что есть clock target filter fn
источник

BB

Bugs Bunny in ☄️ effector
export default - зло is the side effect
Жесть, как можно оправдывать лапшу из аргументов
ну когда мало и знаешь, что будет мало на 100%, то по фигу
источник

ei

export default - зло... in ☄️ effector
export default - зло is the side effect
Жесть, как можно оправдывать лапшу из аргументов
Это же просто отвратительно, совершенно нет плюсов
источник

BB

Bugs Bunny in ☄️ effector
а когда это неизвестно, то тут уже объектный надо 100%
источник

ei

export default - зло... in ☄️ effector
Ну, я понимаю когда функция прямо на 1000% будет из 1-2 аргументов, то это да
источник

m

makoven in ☄️ effector
Aleksandr Osipov
ну а вслучае guard sample именованные параметры просто напросто улучшают читаемость кода на мой взгляд, потому что не надо помнить о порядке и сразу видно что есть clock target filter fn
Да я-то не против. Эффектор от этого хуже не стал. Я просто за чистоту. Появятся в стандарте именованные аргументы, тогда ок)
источник

🦜

🦜 in ☄️ effector
Van Pav
Парни, а так легально делать вообще?
const Component = () => {

 useEffect(() => {
   const unwatch = messageCreated.watch(handleScrollBottom);
   return unwatch;
 }, [])

 …
Я бы в модели это описал и сделал импорт в компонент

export const unwatch

import unwatch

useEffect(() => unwatch)
источник