Рекомендации по определению переменных JavaScript

Question

При настройке приложения dojo рекомендуемая передовая практика для инициализации переменных в отношении объектов dom? Я обнаружил, что мне обычно нужна ссылка на родительский и дочерний объекты поверх самого виджета и создаются глобальные заполнители со следующим кодом.

var formItems = ["Item_1", "Item_2", "Item_3", ... "Item_15"]; 

formItems.forEach(function(formItem) { 
    var items = ["frm", "dom", "dijit"]; 
    items.forEach(function(item) { 
     var x = item + formItem; 
     window[x] = undefined; 
    }); 
}); 

против

var frmItem_1, frmItem_2, frmItem_3, ... frmItem_15; 
var domItem_1, domItem_2, domItem_3, ... domItem_15; 
var dijitItem_1, dijitItem_2, dijitItem_3, ... dijitItem_15; 

Переменные фактически заселена позже в приложении додзё после того как все было анализироваться, но на «сохранить», JSHint расстраивается из-за отсутствия реальных явных переменных определяется.

Answer 1

Мне не понравился ни один из этих вариантов. Я предпочитаю иметь как можно меньше глобальных переменных. Я бы рекомендовал инициализировать массив для хранения ваших элементов, а затем ссылаться на этот массив в вашем приложении dojo.

var formItems = []; 
for(var i = 0; i < 15; i++) { 
    formItems.push({ 
     frm: null, 
     dom: null, 
     dijit: null 
    }); 
} 

И в вашем приложении додзё заменить:

• frmItem_N с formItems[N].frm
• domItem_N с formItems[N].dom
• dijitItem_N с formItems[N].dijit

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

Answer 2

Все это субъективно и зависит от вашего проекта. Вы можете настроить JSHint, чтобы сделать вас счастливым.

Если вы хотите удовлетворить ошибки без изменения каких-либо конфигураций JSHint, инициализируйте переменные за пределами цикла, а затем определите их внутри.

var formItems = ["Item_1", "Item_2", "Item_3", ... "Item_15"], 
    items, 
    x; 

formItems.forEach(function(formItem) { 
    items = ["frm", "dom", "dijit"]; 
    items.forEach(function(item) { 
     x = item + formItem; 
     window[x] = undefined; 
    }); 
});