Телеграмм чат группы frontendua страница 1464

2020 July 28

EO

Eugene Obrezkov in Frontend UA

Andrey Listochkin

Ответ. Один метод, кторый берет объект с параметрами.

Билдеры отстойно мокать, они мешают тестированию кода.

ничего не мокайте без крайней необходимости, пожалуйста 🙂

источник

14:03пожаловаться #1

IT

Ivan Tytarenko in Frontend UA

Andrey Listochkin

Ответ. Один метод, кторый берет объект с параметрами.

Билдеры отстойно мокать, они мешают тестированию кода.

мы как раз этого избежать хотим, чтобы не надо было передавать объекты - они ппц неочевидные и потому лучше мы сами будем хендлить некоторые вещи в методах

источник

14:04пожаловаться #2

AL

Andrey Listochkin in Frontend UA

Хз, как API консьюмер я никогда не видел апи, где мне нравилось и было комфортнее юзать билдеры. Почт всегда это cute, но не делает ничего полезного.

источник

14:06пожаловаться #3

AV

Anton Verinov in Frontend UA

Eugene Obrezkov

ничего не мокайте без крайней необходимости, пожалуйста 🙂

И ничего не билдите, когда язык не джава нормально в опциональные аргументы умеет

источник

14:06пожаловаться #4

AL

Andrey Listochkin in Frontend UA

А вот бороться с тем, что я из-за билдеров не могу нормально код генерить, моккать или делать еще какие-то метапрограмные штуки легко - это случается стабильно из года в год.

источник

14:07пожаловаться #5

EO

Eugene Obrezkov in Frontend UA

Anton Verinov

И ничего не билдите, когда язык не джава нормально в опциональные аргументы умеет

сочувствую тем, кому приходится вместо чейна билда лепить 20 (опциональных?) аргументов
хорошо что у нас такого нету, и на том спасибо

источник

14:09пожаловаться #6

AL

Andrey Listochkin in Frontend UA

Поэтому объект с опциональными полями - это отлично

источник

14:09пожаловаться #7

AV

Anton Verinov in Frontend UA

Andrey Listochkin

Поэтому объект с опциональными полями - это отлично

(это и имел в виду, да)

источник

14:10пожаловаться #8

IT

Ivan Tytarenko in Frontend UA

Andrey Listochkin

Поэтому объект с опциональными полями - это отлично

одно но - много контринтуитивных моментов и связанных полей в разных файлах.
Потому объекты тут не особо подойдут

источник

14:15пожаловаться #9

IT

Ivan Tytarenko in Frontend UA

Eugene Obrezkov

сочувствую тем, кому приходится вместо чейна билда лепить 20 (опциональных?) аргументов
хорошо что у нас такого нету, и на том спасибо

йеп

источник

14:15пожаловаться #10

IT

Ivan Tytarenko in Frontend UA

Ivan Tytarenko

одно но - много контринтуитивных моментов и связанных полей в разных файлах.
Потому объекты тут не особо подойдут

Конечно всегда можно и свои объекты создавать, а потом перегонять в нужный формат. Но это уже вкусовщина

источник

14:16пожаловаться #11

AL

Andrey Listochkin in Frontend UA

Ivan Tytarenko

одно но - много контринтуитивных моментов и связанных полей в разных файлах.
Потому объекты тут не особо подойдут

пахнет ошибкой дизайна

источник

14:16пожаловаться #12

AL

Andrey Listochkin in Frontend UA

Ivan Tytarenko

Конечно всегда можно и свои объекты создавать, а потом перегонять в нужный формат. Но это уже вкусовщина

лучше, чем билдер

источник

14:16пожаловаться #13

IT

Ivan Tytarenko in Frontend UA

Andrey Listochkin

пахнет ошибкой дизайна

жиза 🙂 Это все еще и разные команды делают

источник

14:17пожаловаться #14

IT

Ivan Tytarenko in Frontend UA

Andrey Listochkin

лучше, чем билдер

да-да. Это я уже понял 🙂

источник

14:18пожаловаться #15

AL

Andrey Listochkin in Frontend UA

вообще Law of Demeter не зря появился

источник

14:18пожаловаться #16

AL

Andrey Listochkin in Frontend UA

имхо вам никто не мешает сделать и с билдером и с объектами, а потом поискать по виксовому коду, кто как вашим апи пользуется. Узнаете, что 90% вызовов будет одинаковым - это 3-4 параметра. Сделаете для этого случая метод отдельный, и ваша апишка станет лучше чем любое другое АПИ в компании.

источник

14:20пожаловаться #17

IT

Ivan Tytarenko in Frontend UA

Andrey Listochkin

имхо вам никто не мешает сделать и с билдером и с объектами, а потом поискать по виксовому коду, кто как вашим апи пользуется. Узнаете, что 90% вызовов будет одинаковым - это 3-4 параметра. Сделаете для этого случая метод отдельный, и ваша апишка станет лучше чем любое другое АПИ в компании.

ты предлагаешь потом мигрировать кучу апишек в других проектах?

И там не 3-4 параметра, в том-то и дело. В паре объектов по 10+, во втором 2-3, в третьем зоопарк в целом - там еще и компонент может быть

источник

14:23пожаловаться #18

DB

Dima Bildin in Frontend UA

А ещё объект типизировать звучит проще, чем чейн в билдере

источник

14:35пожаловаться #19

DB

Dima Bildin in Frontend UA

Типа если не просто optional параметры, а «если этот не передан, то должны передать вот тот»

источник

14:35пожаловаться #20