Size: a a a

2020 July 23

AO

Aleksandr Osipov in ☄️ effector
Andrey
ну, мне к сожалению, даже не удалось толком с проблемой ознакомиться)
точнее, я понял начальный посыл, но конкретно в чем она проявлялась - нет
Ну речь о том что канонично выделять отдельно API слой, я так понял @popuguy о том что не гоже хардкодить энпоинты в моделях и наверное в теории он прав, но на практике как я уже сказал у меня не было такого что /persons поменяли на /animals и мне пришлось бы лихорадочно заменять по всему коду
источник

DS

Dmitriy Shuleshov in ☄️ effector
🚀🔬 🚀🔬🚀🔬
то есть идея в том что если ужать прям всё до байта, то внесение правок в отдельные части станет проблематичным из-за массового реюза этого кода
время репоста поговорок)
источник

AO

Aleksandr Osipov in ☄️ effector
Aleksandr Osipov
Ну речь о том что канонично выделять отдельно API слой, я так понял @popuguy о том что не гоже хардкодить энпоинты в моделях и наверное в теории он прав, но на практике как я уже сказал у меня не было такого что /persons поменяли на /animals и мне пришлось бы лихорадочно заменять по всему коду
и самое главное что просто так эндпоинты не меняют, если сменили то наверное и семантика изменилась и надо менять еще и название метода api
источник

н#

не ты зло, а moment.... in ☄️ effector
источник

н#

не ты зло, а moment.... in ☄️ effector
источник

AO

Aleksandr Osipov in ☄️ effector
Aleksandr Osipov
и самое главное что просто так эндпоинты не меняют, если сменили то наверное и семантика изменилась и надо менять еще и название метода api
ну то есть скажем есть у меня в api метод fetchPersons  который дергает /persons и потом бек такой взял и поменял на /animals, но ведь это скорее всего не спроста, наверное изменились и данные  и смысл эндпоинта и надо будет и fetchPersons заменить на fetchAnimals
источник

🦜

🦜 in ☄️ effector
Aleksandr Osipov
и самое главное что просто так эндпоинты не меняют, если сменили то наверное и семантика изменилась и надо менять еще и название метода api
я описал с чем мне приходилось столкнуться, когда беки решили сменить фреймворк для bff
источник

A

Andrey in ☄️ effector
Aleksandr Osipov
Ну речь о том что канонично выделять отдельно API слой, я так понял @popuguy о том что не гоже хардкодить энпоинты в моделях и наверное в теории он прав, но на практике как я уже сказал у меня не было такого что /persons поменяли на /animals и мне пришлось бы лихорадочно заменять по всему коду
ну, вроде в этом методе никто не мешает экспортировать методы для создания реквестов в данном домене, в слое апи)

типа export const createUserRequest если уж настолько хочется абстрагирования
источник

🚀🚀

🚀🔬 🚀🔬🚀🔬... in ☄️ effector
Aleksandr Osipov
Ну речь о том что канонично выделять отдельно API слой, я так понял @popuguy о том что не гоже хардкодить энпоинты в моделях и наверное в теории он прав, но на практике как я уже сказал у меня не было такого что /persons поменяли на /animals и мне пришлось бы лихорадочно заменять по всему коду
«лихорадочно»: ну вот мы приходим к тому что это вопрос отношения к самому процессу

лично я предпочту написать код так, чтобы глобальный поиск с заменой вскода работал на нём без проблем. к примеру, строка '/users' (с кавычками) меняется одинаково вне зависимости от объёма использования
источник

н#

не ты зло, а moment.... in ☄️ effector
@ZeroBias могу делать пулл реквест?)
источник

🚀🚀

🚀🔬 🚀🔬🚀🔬... in ☄️ effector
не ты зло, а moment.js 👨‍💻 #stayhome
@ZeroBias могу делать пулл реквест?)
да, конечно! 👍
источник

AO

Aleksandr Osipov in ☄️ effector
🦜
я описал с чем мне приходилось столкнуться, когда беки решили сменить фреймворк для bff
я понял твои аргументы, но в конкретном случае не вижу для себя смысла еще одну абстракцию вводить в виде api-прослойки
источник

🦜

🦜 in ☄️ effector
https://t.me/effector_ru/145231

тогда дальше этого мы не продвинемся
источник

🦜

🦜 in ☄️ effector
/thread
источник

A

Andrey in ☄️ effector
а обвешиваться абстракциями, чтобы потом, когда заказчик красную кнопку тыкнул "поменять все!!", раз в 20 лет, и сидеть их поддерживать - сомнительное удовольствие)
источник

🦜

🦜 in ☄️ effector
Aleksandr Osipov
я понял твои аргументы, но в конкретном случае не вижу для себя смысла еще одну абстракцию вводить в виде api-прослойки
но ты ввел dataProvider как абстракцию
источник

𝐁𝐏

𝐁𝐨𝐡𝐝𝐚𝐧 𝐏𝐞𝐭𝐫𝐨𝐯... in ☄️ effector
Молодец)
источник

AO

Aleksandr Osipov in ☄️ effector
🦜
но ты ввел dataProvider как абстракцию
ладно забей, не думал что такую дискуссию мой говнокод вызовет :)
источник

AO

Aleksandr Osipov in ☄️ effector
🦜
но ты ввел dataProvider как абстракцию
на это были причины
источник

A

Andrey in ☄️ effector
Aleksandr Osipov
ладно забей, не думал что такую дискуссию мой говнокод вызовет :)
та норм код, плюсую за такой подход
источник