/**
* Функция возвращает массив из элементов с уникальными значениями
*/
function unique(array){
return $.grep(
array,
function(el, index) {
return index == $.inArray(el, array);
}
);
}
var tema = [1,1,3,2,2,3,6,7,8,2,1,9],
name_test_box = '',
arr = [];
tema = unique(tema).sort();
$('label[for^="drink"]', '#form_1').each(
function() {
var str_for = $(this).attr('for');
if (str_for) {
arr[str_for] = this.innerHTML;
}
}
);
for (var i = 0, count = tema.length; i < count; i++) {
if (!arr['drink' + tema[i]]) {
continue;
}
name_test_box +=
'<option value="' + tema[i] + '">' +
arr['drink' + tema[i]] +
'</option>';
}
Комметарий:
Используйте по возможности context при поиске элементов DOM, а именно:
$('label[for^="drink"]', '#form_1')
, где form_1 - id формы.
т.е. изначально осуществляется поиск контекста, в данном случае формы, в котором будет происходить поиск всех элементов label с атрибутом for, значение которого начинается со строки drink.
Внутри же цикла, мы будем оперировать уже найденным массивом arr с необходимыми значениями, а не искать элементы в цикле каждый раз (в каждой итерации) по всему DOM-дереву элементов.
P.S. Валидно и для IE6.
UPD:
function unique(array){
var used = {},
isIE = $.browser.msie && $.browser.version <= 8 && !window.XMLHttpRequest;
return (isIE)
? $.grep(
array,
function(el, index) {
return index == $.inArray(el, array);
}
)
: array.filter(function(obj) {
return obj in used ? 0 : (used[obj] = 1);
});
}
Тестовые данные:
var data=[];
/* заполняем массив повторяющимися значениями */
for(var i = 0, a = 0; i < 100000; i++, a++) {
if (a > 2) {
a = 0
}
data[i] = 1 + a;
}
console.time('time');
console.log(unique(data));
console.timeEnd('time');