А нужна ли переменная?

Question

[Evgeniy Kvasyuk]

Имею привычку вначале любой функции объявлять переменные для любых данных, к которыми будет работать функция.

var func = function (str) {
var workStr = str,
    workStrLength  = str.length;
}

Делаю так даже если в переменную вносится свойство какой-то другой переменной (как описано выше) или выполняется какое-то одноразовое приметивное действие. Встает вопрос а стоит ли так делать? Или не стоит лишний раз плодить переменные?

Answer 1

[Viktor Vsk]

Макконнел говорит, что нужно различать input переменные и output переменные.
Т.е. если с одной стороны вы str используете на входе и иначе используете - на выходе - то разделение гуд. Если нет - то смысл непонятент.

Тогда ответ такой - если вам удобно (или вашей команде так удобно) - то ок. Иначе - неясно.
Все равно, что написать:
var trueValue = true но если это каким-то образом повышает читаемость, то это проблема дизайна

Answer 2

[Виталий Инчин ☢]

workStrLength (12 символов) VS str.length (10 символов) выносить не стоит
А вот когда так, грубо говоря:
dDE VS document.documentElement.clientHeight, то, конечно, лучше записать отдельно.

А вообще, такие мелочи делают код приятнее и удобнее для просмотра. Писать так, как вы - правильно, в каком-то смысле, так что не волнуйтесь :)

Comments

[Viktor Vsk]

пример плохой
document.documentElement.clientHeight - это не переменная, а цепочка вызовов, так что в любой случае это совершенно разные вещи

[Виталий Инчин ☢]

Виктор Ablebeam: dDE - переменная

Answer 3

[Сергей Сергей]

Ничего страшного, ящитаю.
Ведь постепенно код растет, а вместе с ним и потребность в переменных. А так хоть какие-то будут описаны заранее.

И вообще, интерпретатор все стерпит.

Answer 4

[Дмитрий Ковальский]

Ну если у вас нет ограничений на размер файлов скриптов, да и минимизатор используете - то не страшно. Хотя привычка странная.

Answer 5

[Денис Инешин]

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

function foo (superVeryLongStingName, config) {
    var str = superVeryLongStingName; // шорткат 1
    var option = config.group.value; // шорткат 2
}

Answer 6

[Вячеслав]

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

var $node = $(element);
var cntNode = $node.length;
if(cntNode < 5){
    $node.remove();
}

Также, для удобочитаемости при обходе массивов циклом for, или в подобных случаях можно записывать во внешнюю переменную элемент на данной операции
Например:

for(var i = 0; i < obj.valArray.length; ++i){
    var currentItem = obj.valArray[i];
}