Почему такой код считается нормальным?

Question

[Михаил]

Всем привет. Открыл документацию к hapi.js и увидел такой пример кода:

const Hapi = require('hapi');
const server = Hapi.server({});

async function start() {
    try {
        await server.start();
    }
    catch (err) {
        console.log(err);
    }
};

start();

В универе я писал на С, и меня учили, что:
1) В функциях не должно быть глобальных переменных
2) Каждая функция должна быть написана так, чтобы её можно было скопировать в другой проект без изменений

В данном же примере кода всё это нарушается, ибо:
1) В функции start используется глобальная (для этой функции) переменная server
2) Если я перенесу функцию start в другой файл, то мне придётся ещё дописывать часть кода
3) Если вдруг я в другом проекте сделаю const app = Hapi.server({}), то функция start опять не сработает

Чего я не понимаю? Как мне понять, какие подходы более правильные?

Comments

[Сергей Горностаев]

Кем считается?

[Михаил]

Сергей Горностаев, авторами фреймворка и сообществом node.js

[Сергей Горностаев]

Михаил, в огромном количестве популярных проектов встречается говнокод. Например, битрикс состоит из него чуть более, чем полностью. Одни просто не понимают, что это говнокод. Другие предпочитают закрывать на это глаза.

[Иосиф Копейкин]

Забудь все, чему тебя учили в универе)

[Илья Шатохин]

Михаил, вызывать require внутри функции - плохая практика, т.к. поиск пути к модулю отнюдь не бесплатен и высчитывается заново при вызове require. Всегда инициализируйте модули в загловке файла. На хабре была целая статья на эту тему.

И да, в вашем примере нет глобальных переменных. Все файлы (модули) в node.js вызываются в контексте анонимной функции, в которую передаются такие аргументы, как "require", "process" и т.д.

Answer 1

[index0h]

В функциях не должно быть глобальных переменных

Да, это плохая практика, но nodejs ее пропагандирует, увы.

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

В реальной жизни это утверждение справедливо только для библиотек общего назначения. Когда ваши функции/методы реализуют бизнес логику приложения - лучше их написать менее универсальными, но более поддерживаемыми. Все равно переносить в другой проект вы их не будете

Comments

[Антон Швец]

В каком месте в Ноде пропагандируют глобальные переменные?

[index0h]

В require например, учитывая, что константы в js - не совсем константы

[Антон Швец]

index0h, сам require это переменная которая передается в модуль. И он тоже не глобальный.
Объявления переменных в начале модуля действуют только в пределах него. Считай запускаемый файл это main метод.
А глобальную константу можно сделать так

Object.defineProperty(global, 'MYCONST', {
  value : 37,
  writable : false,
  configurable: false
});

console.log(MYCONST)  //-> 37
MYCONST = 17;
console.log(MYCONST) //-> 37
delete MYCONST
console.log(MYCONST)   //-> 37

Можно еще сеттер добавить, который выбрасывает исключение.

[index0h]

Антон Значение, получаемое через require общее для всех файлов, в которых оно подключается. Да, отличия от глобальных переменных есть, но не такие существенные.

Answer 2

[RidgeA]

Такой подход общепринят в NodeJS.

Кстати, модуль в NodeJS по сути является функцией. Так что копирование функции `start` - это копирование части
функции по сути :-)

Ничто не мешает написать так:

async function start() {
    const Hapi = require('hapi');
    const server = Hapi.server({});

    try {
        await server.start();
    }
    catch (err) {
        console.log(err);
    }
};

start();

Comments

[Антон Швец]

а че не сразу require('hapi') .server({}).start()?

[Михаил]

Понятно, что данный код можно исправить. Вопрос в том, почему данный код считается нормальным и висит на главной странице фреймворка

[Антон Швец]

Михаил, да я к тому что переменная server скорее всего будет использована где-то еще.
А в примере выше она замурована в функции.
Корректно было бы принимать Hapi как параметр и возвращать server, но тогда исчезают отличия этой функции от Hapi.server({})

Answer 3

[Антон Швец]

в данном случае это просто обертка для асинхронного выполнения.
Можно было написать

const Hapi = require('hapi');
const server = Hapi.server({});

server.start().catch(console.error);

И всё. Без всяких функций которые вы куда-то хотите копировать

Плюс в данном случае const server = Hapi.server({});
это не глобальная переменная. Это переменная с областью видимости в пределах текущего модуля (js файла)
Глобальные переменные в Ноде это свойста объекта global

Comments

[Михаил]

Почему не глобальная? Относительно функции start это глобальная переменная

[Антон Швец]

Михаил, чего?
Это переменная из окружения (или правильнее -скоупа) функции start. В js это часто применяют, замыкания те же.
Именно глобальных переменных как таковых в Ноде вообще нет, а объект global используют крайне редко, и это да, дурной тон.
https://habrahabr.ru/company/ruvds/blog/341646/ вопрос 7

С таким же успехом можно сказать что свойства объекта глобальные для его методов. Они же не передаются!

[Михаил]

Антон, хм, хорошо. Давайте я немного изменю вопрос. Функция start а том виде, в котором она сейчас написана, является чистой?

З. Ы. Да, переменная start не является глобальной в общепринятом смысле. Был не прав

[Антон Швец]

Михаил, функция старт фактически является приватным методом для текущего модуля.
А

const Hapi = require('hapi');
const server = Hapi.server({});

это его свойства. Смотрите на это с такой стороны.

Answer 4

[Константин Китманов]

Да это же просто пример. В реальной жизни никто не мешает вынести эту функцию в отдельный файл и передавать в нее server параметром. Просто если бы в доках это все разбили на три файла, читать их было бы сложнее:)

Answer 5

[Дмитрий]

Здесь никаких противоречий нет. В NodeJS модуль это функция. Весь описанный вами код под капотом выглядит вот так

(function(exports, require, module, __filename, __dirname) {
const Hapi = require('hapi');
const server = Hapi.server({});

async function start() {
    try {
        await server.start();
    }
    catch (err) {
        console.log(err);
    }
};

start();
});