Разумно ли импортировать одни и те же библиотеки в каждый модуль?

Question

[kr_ilya]

Если мне нужны одни и те же библиотеки в разных модулях, я делаю так:
a.js

const os = require('os'),
	path = require('path'),
	fs = require('fs'),
	kb = require('./modules/keyboards.js'),
	config = require('./config.js');

var state = {};

require('./modules/b')(state);

...

b.js

const os = require('os'),
	path = require('path'),
	fs = require('fs'),
	kb = require('../modules/keyboards.js'),
	config = require('../config.js');

module.exports = (state) => {

...
}

А правильно ли вообще так делать? Как это влияет на производительность? Как это сделать лучше?
Я считаю, что можно все модули передавать напрямую в функцию (как это сделано со state), но я не хочу захламлять аргументы.

Comments

[Михаил]

require на один и тот же модуль работает только раз, в другие разы из кэша

[kr_ilya]

Михаил, понял, спасибо

[Михаил]

Energy2, и если уж на то пошло, на каждую переменную своя var let/const, не стоит писать их через запятую

[Lynn «Кофеман»]

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

Я бы оставил всё как есть исходя из принципа наименьшего удивления. Любой человек знакомый с нодой однозначно понимает что значит fs = require('fs') в отличии от какого-нибудь fs = require('../myglobals.js').fs

[kr_ilya]

Михаил, чего это так? Как по мне так выглядит лучше

[Михаил]

Energy2, посмотрите общепринятые "стандарты" стиля кода в js, я тоже раньше так писал, но когда устроился в компанию, пришлось переучиваться, так что лучше сразу привыкать к общему стилю

[kr_ilya]

Михаил, приму к сведению

Answer 1

[Alex]

В каждом модуле импортируйте всё и исключительно то, что нужно для работы непосредственно этого модуля. Так ваша система будет гибче и работать с каждым модулем отдельно будет проще. Модули устроены так, что импортируются только раз, не зависимо от того, сколько раз вы вызываете require.

Ваш подход вполне нормальный, если вы конечно не импортируете в модуль А что-то что используется только в модуле Б и не используется в модуле А непосредственно.

Выносить все импорты в отдельный файл, как предлагает Дмитрий я бы не стал.

А

все модули передавать напрямую в функцию (как это сделано со state)

оправдано только в том случае, если вы реализуете паттерн Стратегия

Comments

[Дмитрий]

Выносить все импорты в отдельный файл, как предлагает Дмитрий я бы не стал.

Почему?

[Alex]

Это приведет к кошмарной путанице при разрастании.

• Что будет когда модуль А начнет зависеть от большего числа библиотек, которые не использубтся в модуле Б?
  
• Или появится третий модуль С, который зависит от совершенно другого набора подмодулей?
  
• Или когда у вас таких модулей будут десятки? Как будет выглядеть ваш imports.js? Как собрание всех зависимостей со всему проекта? Если после этого, кто-то отрефакторит модуль А и Б и они перестанут использовать os как определить используется ли os где-то ещё или его можно безопасно удалить из imports.js?
  
• Или как быть с условным подключением? Когда нам не нужно подключать вообще все модули сразу, а только некоторые в зависимости от разных условий.
  

[Дмитрий]

Alex, Насколько я понимаю тут речь идет о нескольких стандартных библиотеках и автор написал, что они используются во всех модулях. Если не используются во всех, то конечно можно делать как-то более разумно.

Answer 2

[Дмитрий]

Тогда уж просто соберите их все в одном файле в объект и импортируйте объект, зачем так сложно?

const os = require('os'),
  path = require('path'),
  fs = require('fs'),
  kb = require('../modules/keyboards.js'),
  config = require('../config.js');

module.exports = { os, path, fs, kb, config }