Как в nodejs принято объявлять переменные?

Question

[vasIvas]

В чистом js принято объявлять свойства через запятую c одним оператором объявления -

var a = 0,
	b = 1,
	c = 3;

Но на githab я очень часто вижу, когда свойства объявляются как в коде ниже -

var a = 0;
var b = 1;
var c = 2;

Конвенции найти не смог, по этому спрашиваю - как правильно?

Answer 1

[Сергей Протько]

А как хотите. Мне нравится первый вариант но у него есть один неприятный минус - в git log-ах добавление переменной затронет две строки а не одну:

var a,
      b;

var a,
-     b;
+     b,
+     c;

второй вариант с этой точки зрения более приятный. Так что решайте что вам важнее. Красивые логи или приятный виду код.

Comments

[vasIvas]

Спасибо.

[Юрий Пузыня]

Есть у первого варианта и еще одна проблема: забудете запятую в конце поставить, получите глобальную переменную

[Юрий Пузыня]

К слову есть еще и третья спорная конструкция const fs = require('fs')

[vasIvas]

3y3: константа естественно была бы лучше, так как явным образом указывала бы на неизменяемость объекта fs, но у меня нет их :( Я пользуюсь gulp, а у него видно старая версия...

[Тимур Шемсединов]

Главное, НЕ используйте дибильный способ:

var a = 0
  , b = 1
  , c = 3;

И отступы в пробелах делайте, а не табуляцией, именно 4 пробела, даже если везде отступ в 2 используете, то для var делайте 4.

[Сергей Протько]

Тимур Шемсединов: я обычно так делаю:

var
  fs = require('fs'),
  gulp = require('gulp'),
  less = require('gulp-less')
;

По поводу дебильного способа - согласен.

vasIvas а при чем тут gulp? Это node.js может не уметь делать.

[Тимур Шемсединов]

Да, Сергей Протько, способ красивый, жалко, что ни как нельзя после последней скобки еще одну запятую добавить. Кстати, если уже пошел разговор про require, то иметь его в глобальном контексте - это не халяльно, я требую введения в ближайших версиях: var require = require('require'); // 100% halal

[vasIvas]

Сергей Протько: а где мне указать версию ноды, если я в консоли запускаю не node а gulp?

[Сергей Протько]

vasIvas: https://github.com/gulpjs/gulp/issues/66

[Юрий Пузыня]

Тимур Шемсединов , между тем как раз сейчас require не глобальный, а локальный для каждого модуля, но многое изменится с выходом 0.12 =)

[Тимур Шемсединов]

3y3, почти так, но не совсем. require - это искуственная конструкция, накрученная над vm.createateScript. Внутри каждого скрипта, созданного через createateScript или (через его производные методы runInContext, runInNewContext...) все переменные глобальные. А модуль module.js, реализует метод require - подделывая его внутри песочницы (как и другие псевдо-нативные конструкции):

... 
var sandbox = {};
for (var k in global) {
  sandbox[k] = global[k];
}
sandbox.require = require;
sandbox.exports = self.exports;
sandbox.__filename = filename;
sandbox.__dirname = dirname;
sandbox.module = self;
sandbox.global = sandbox;
sandbox.root = root;
...

Но это бы тоже не помогло, нужно же еще не дать скрипту нагадить в глобал, обернув его в функцию, это делает Module.wrap(content) который оборачивает исходник в функцию, доклеивая строки "(function(global) {" + source + "} (sandbox));" это псевдокод конечно, для пояснения того, как это работает.
То есть, уже сейчас можно не использовать нодовский require, а сделать свой, который бедет как-то инача подделывать глобальный контекст для модулей. Я вот использую свою реализацию, потому, что мне нужно забирать из скрипта локальные переменные без прямого экспорта, т.е. тырить их из модуля и сохранять, а потом, когда этот модуль запускается опять, то восстанавливать их в его локальный контекст в том виде, как было на момент окончания его работы. Как обстоят дела в 0.11 (0.12) честно - не знаю, не смотрел еще, но все мои юниттесты и интеграционные тесты на 0.11 не падают, значит реализация во многом приемственная.

[Юрий Пузыня]

Тимур Шемсединов, думаю у нас разные понятия об объекте global. В силу специфики моих задач (проект node-inspector), я на своей шкуре прочувствовал, что require никогда не находится в реальном объекте global. Для сравнения возьмем объект console, вот он действительно глобален. Сейчас посмотрел исходники 0.12 - заявленной фичи там нет, а она сильно бы упростила жизнь node-inspector. На правах рекламы - вышел Node Inspector 0.8.0 с консолью и профилировщиками.

[Тимур Шемсединов]

Юрий Пузыня, счастлив встретить человека, который понимает, как работают require, global, sandboxes, а то когда начал использовать sandboxes, то было ощущение, что я единственный, кому это нужно и кто вообще открывал этот код и эти доки, кому не расскажу, все говорят - нет такого в ноде вообще, в ноде есть только app.get :)