Есть сомнения. Когда передавать ф-ции через аргументы, а когда просто

Question

Есть сомнения. Когда передавать ф-ции через аргументы, а когда просто

заимпортить и использовать в любом месте?
Сам предполагаю, что чистую ф-цию можно импортить без проблем. А вот сайд ефект ф-ции лучше передавать через аргументы, чтобы потом подменить в тестах.

#backend #javascript #node.js #programming #russian

0

01.01.2021

20 ответов

4 просмотра

Vlad Petrov Автор вопроса

Типа вот такого имелось ввиду ts pg

0

01.01.2021

Pavel Shakhov (pongo)

да, я понимаю идею

0

01.01.2021

Vlad Petrov Автор вопроса

Тоесть ты согласен, что сайд эфект ф-ции лучше передавать через аргументы. А чистые можно импортить. Правильно?

0

01.01.2021

Pavel Shakhov (pongo)

подход вроде и норм. но вдруг будут какие-то минусы у него? ну, длинный список переменных? можно тогда передавать объект dependencies

0

01.01.2021

Vlad Petrov Автор вопроса

длинный список зависимостей говорит о неправильном разделении.

0

01.01.2021

Pavel Shakhov (pongo)

кстати, при помощи nock эта функция и без моков легко тестируется

0

01.01.2021

Vlad Petrov Автор вопроса

Подход с моком импортом мне кажется каким то неестественным и неправильным. Вот и хотел спросить в более опытных в этом.

0

01.01.2021

Vlad Petrov Автор вопроса

Та, что с аргументом или импортом?

0

01.01.2021

Pavel Shakhov (pongo)

я про bankRates(). nock мокает сетевые запросы

0

01.01.2021

Pavel Shakhov (pongo)

я сам почти никогда не мокаю импорты и стараюсь передавать зависимости в параметры функции или конструктора. но мне тоже хотелось бы услышать разные мнения

0

01.01.2021

Vlad Petrov Автор вопроса

Я бы вообще не смотрел бы в этом направлении. Зачем фейкать сеть. Если можно просто фейкнуть контракт Currency => Promise<number>

0

01.01.2021

Pavel Shakhov (pongo)

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

0

01.01.2021

Vlad Petrov Автор вопроса

Я считаю, что мокать fetch - запашок. Это сильно низкоуровневая деталь. Лучше подменять контракт высокого уровня

0

01.01.2021

Pavel Shakhov (pongo)

да, в общем случае ты прав

0

01.01.2021

Vlad Petrov Автор вопроса

Я вот хз как такое люди пишут без тс?.. Это же боль.

0

01.01.2021

倫太郎岡部

А если люди которые кодят всей командой на одном сервере по ссш в виме пхп

0

01.01.2021

Vlad Petrov Автор вопроса

Пхп.. Какие то предрассудки. Пхп далеко ушел. Есть прекрасные тулы аля symfony(не уверен, что на ноде есть хоть что то подобное) и psalm(stat analysis). А говнокодить везде можно.

0

01.01.2021

倫太郎岡部

я просто реальную историю одного из клиентов из индии рассказал)

0

01.01.2021

Vlad Petrov Автор вопроса

Да, это просто крайность. Но в конторах нашего региона часто не бывает: тестов, типизации, ci, cd, контейнеризации и т.д. А это всё тоже боль.

0

01.01.2021

Pavel Shakhov (pongo) · Accepted Answer

Pavel Shakhov (pongo)

да вроде логично звучит

0

01.01.2021

170 похожих чатов

Есть сомнения. Когда передавать ф-ции через аргументы, а когда просто

20 ответов

Похожие вопросы