Size: a a a

2020 March 23

🦉⁣

🦉 ⁣ in ☄️ effector
yumaa verdin
вообще в данном случае по моему самый норм вариант — это обычный тернарник. и быстро, и понятно, и ничего лишнего не создаётся.
надо только преттиер и линтер отключать :)
вложенные тернарки стараюсь не юзать
ибо их парсить взглядом тот ещё геморрой

да и отключать преттиер тож так себе затея
источник

АБ

Александр Бакиматов in ☄️ effector
🦉 ⁣
вложенные тернарки стараюсь не юзать
ибо их парсить взглядом тот ещё геморрой

да и отключать преттиер тож так себе затея
+++
источник

yv

yumaa verdin in ☄️ effector
🦉 ⁣
вложенные тернарки стараюсь не юзать
ибо их парсить взглядом тот ещё геморрой

да и отключать преттиер тож так себе затея
смотря как отформатировать код)
источник

🦉⁣

🦉 ⁣ in ☄️ effector
я тернарки пишу так:

const foo =
 condition
 ? first
 : second
источник

yv

yumaa verdin in ☄️ effector
мой вариант с функциями клёвый, но в проде я бы не стал такое юзать (если только не вся кодовая база прямо обмазана функциональщиной).
насколько я знаю (могу ошибаться), ещё ни в один JS движок не завезли оптимизацию хвостового вызова, это когнитивно напрягает в таком использовании :)
источник

Ti

Time is the side effect in ☄️ effector
yumaa verdin
вообще в данном случае по моему самый норм вариант — это обычный тернарник. и быстро, и понятно, и ничего лишнего не создаётся.
надо только преттиер и линтер отключать :)
Хз, я бы наверное что-то такое сделал (если конечно есть тип девайса до преобразования в булеан)

const deviceToRange = {
 'tablet': 5,
 'mobile': 3,
 'mobile-sm': 1,
 'default': 7
}

const range = deviceToRange[deviceType] ?? deviceToRange.default
источник

DS

Dmitriy Shuleshov in ☄️ effector
Time is the side effect
Хз, я бы наверное что-то такое сделал (если конечно есть тип девайса до преобразования в булеан)

const deviceToRange = {
 'tablet': 5,
 'mobile': 3,
 'mobile-sm': 1,
 'default': 7
}

const range = deviceToRange[deviceType] ?? deviceToRange.default
это не то
источник

Ti

Time is the side effect in ☄️ effector
Dmitriy Shuleshov
это не то
Поч
источник

DS

Dmitriy Shuleshov in ☄️ effector
шо такое deviceType ?
источник

🦉⁣

🦉 ⁣ in ☄️ effector
Dmitriy Shuleshov
шо такое deviceType ?
deviceType: 'tablet' | 'mobile' | 'mobile-sm'
источник

Ti

Time is the side effect in ☄️ effector
Dmitriy Shuleshov
шо такое deviceType ?
У меня например адаптивность через жс сделана так, что сначала ширина и высота экрана  мапятся в соответствующий deviceType (mobile, tablet, desktop), а потом от этого выводятся соответствующие сторы isMobile, isTablet и isDesktop
источник

Ti

Time is the side effect in ☄️ effector
Ну а если нет ничего такого промежуточного, там реально без тернарника только пачка ифов друг за другом) Или switch (true), что вообще жесть
источник

DS

Dmitriy Shuleshov in ☄️ effector
🦉 ⁣
deviceType: 'tablet' | 'mobile' | 'mobile-sm'
ну так его вычислить заранее прийдется
источник

Ti

Time is the side effect in ☄️ effector
Dmitriy Shuleshov
ну так его вычислить заранее прийдется
Как и isMobile и все такое
источник

Ti

Time is the side effect in ☄️ effector
Плюс в том что в компонент ты будешь прокидывать строку mobile, или проп mobile, зависит от того что компонент решает, все равно где-то придется преобразовывать
А тут все само выводится для всех этапов
источник

🦉⁣

🦉 ⁣ in ☄️ effector
Dmitriy Shuleshov
ну так его вычислить заранее прийдется
ну да
источник

yv

yumaa verdin in ☄️ effector
yumaa verdin
мой вариант с функциями клёвый, но в проде я бы не стал такое юзать (если только не вся кодовая база прямо обмазана функциональщиной).
насколько я знаю (могу ошибаться), ещё ни в один JS движок не завезли оптимизацию хвостового вызова, это когнитивно напрягает в таком использовании :)
ух ты, сафари и вебкит меня удивили :)
источник

D

Dmitry in ☄️ effector
как юзая createApi добавить гард ?
источник

D

Dmitry in ☄️ effector
это переписать можно ?
источник

A

Andrey in ☄️ effector
Dmitry
это переписать можно ?
насчет createApi/guard - нет
но:

guard(addTodo, {
 filter: v => v.value.length > 5,
 target: todoApi.addTodo
});

можно ведь так, чтобы без форвардов
источник