AO
Поэтому в эффекторе везде опции идут обьектом так сказать с заделом на будущее
Size: a a a
2020 July 22
m
Aleksandr Osipov
Ну вот когда параметров больше становится то сталкиваемся что такой подход нерасширяем
Но их не больше 3 в ваших апи
AO
AO
С течением времени там появлялись новые опциональные параметры
AO
И возможно ещё будут
AO
Если бы изначально там не использовался объектный синтаксис для опциональных параметров, то это было бы просто не масштабируемо
AO
makoven
Но их не больше 3 в ваших апи
Не всегда известно заранее как будет расти апи
m
Aleksandr Osipov
С течением времени там появлялись новые опциональные параметры
Всякие скучные параметры можно сваливать в объект options последним аргументом. Все-равно они не используются в 99% случаев
BB
ну вот такой же логикой руководствовались в openapi-generator и получилось не очень
BB
мягко говоря
BB
vsc еще аргументы никак не именует и там просто подряд портянка из параметров
m
ну вот такой же логикой руководствовались в openapi-generator и получилось не очень
Не только. В ноде, в стандартной библиотеке встречается. Нормально
ei
Жесть, как можно оправдывать лапшу из аргументов
AO
makoven
Всякие скучные параметры можно сваливать в объект options последним аргументом. Все-равно они не используются в 99% случаев
ну а вслучае guard sample именованные параметры просто напросто улучшают читаемость кода на мой взгляд, потому что не надо помнить о порядке и сразу видно что есть clock target filter fn
BB
Жесть, как можно оправдывать лапшу из аргументов
ну когда мало и знаешь, что будет мало на 100%, то по фигу
ei
Жесть, как можно оправдывать лапшу из аргументов
Это же просто отвратительно, совершенно нет плюсов
BB
а когда это неизвестно, то тут уже объектный надо 100%
ei
Ну, я понимаю когда функция прямо на 1000% будет из 1-2 аргументов, то это да
m
Aleksandr Osipov
ну а вслучае guard sample именованные параметры просто напросто улучшают читаемость кода на мой взгляд, потому что не надо помнить о порядке и сразу видно что есть clock target filter fn
Да я-то не против. Эффектор от этого хуже не стал. Я просто за чистоту. Появятся в стандарте именованные аргументы, тогда ок)
🦜
Парни, а так легально делать вообще?
const Component = () => {
useEffect(() => {
const unwatch = messageCreated.watch(handleScrollBottom);
return unwatch;
}, [])
…Я бы в модели это описал и сделал импорт в компонент
export const unwatch
import unwatch
useEffect(() => unwatch)
export const unwatch
import unwatch
useEffect(() => unwatch)