Don't make functions within a loop

@barkar · Регистрация: 11.02.2014

Студворк — интернет-сервис помощи студентам

Есть код:

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
var ul,.... и т.д.;
        for(x = 0; x < len; ++x){
            ul.append(
                $('<li></li>', { data: { 'price': jsonres[x].price } }).append(
                    $('<img>', { src: jsonres[x].url }),
                    $('<p></p>', { text: jsonres[x].name }),
                    $('<p></p>', { text: 'Цвет: '+jsonres[x].color }),
                    $('<p></p>', { text: 'Размер: '+jsonres[x].size }),
                    $('<p></p>', { text: this.digitSpace(jsonres[x].price), data:{'price': jsonres[x].price} }),
                    $('<span></span>', {text:'Удалить', click : function(){
                            var saleShow = $('#saleShow'),
                                prnt = $(this).parent('li'), 
                                a = prnt.index(),
                                itogo = saleShow.find('li:last>span'),
                                result =  itogo.data('itogo') ;
                            $.ajax({
                                type: 'POST',
                                url: '/ajax/ajax.php',
                                dataType: 'json',
                                data: { deleq: a },
                                success: function(delres) {
                                    prnt.stop(false, true).fadeOut(400, function() {
                                        sizeSale.text(delres.length);
                                        var saleShowH = saleShow.outerHeight(true),
                                            saleShowW = saleShow.outerWidth(true);
                                        $('#Lcont').css({'width': saleShowW+'px', 'height': saleShowH+'px'});
                                        result -= prnt.data('price');                                       
                                        itogo.data('itogo', result).text( lightbox.digitSpace(result.toFixed(2)) );
                                        $(this).remove();
                                        _window_.trigger('resize'); 
                                        if(result === 0){ $('#ext').trigger('click');}
                                    });                                 
                                }
                            });
                        }
                    })));
            sum += parseFloat(jsonres[x].price);
        }

Выдает ошибку:в JShint выдает ( Don't make functions within a loop. (W083) );
Как поступить?

@Padimanskas · 06.03.2014, 16:01

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

JavaScript
1
2
3
4
5
6
7
8
var interval = setInterval(function(){
  if(x < len){ 
    x++;
    //  ... ваш мега код...
  } else {
    clearInterval(interval); 
  };
}, 100);

@barkar · 06.03.2014, 16:24 **[ТС]**

Решил так

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
        for(x = 0; x < len; ++x){
            ul.append(
                $('<li></li>', { data: { 'price': jsonres[x].price } }).append(
                    $('<img>', { src: jsonres[x].url }),
                    $('<p></p>', { text: jsonres[x].name }),
                    $('<p></p>', { text: 'Цвет: '+jsonres[x].color }),
                    $('<p></p>', { text: 'Размер: '+jsonres[x].size }),
                    $('<p></p>', { text: this.digitSpace(jsonres[x].price), data:{'price': jsonres[x].price} }),
                    $('<span></span>', {text:'Удалить', click : this.clickDel
                    })));
            sum += parseFloat(jsonres[x].price);
        }

Новые блоги и статьи Все статьи Все блоги /
Midnight Chicago Blues kumehtar 24.03.2026 Такой Midnight Chicago Blues, знаешь?. . Когда вечерние улицы становятся ночными, а ты не можешь уснуть. Ты идёшь в любимый старый бар, и бармен наливает тебе виски. Ты смотришь на пролетающие. . .	Контроль уникальности заводского номера - вариант №2 Maks 24.03.2026 В отличие от предыдущего варианта добавлено прерывание циклов, также добавлены новые переменные для сохранения контекста ошибки перед прерыванием цикла: Процедура ПередЗаписью(Отказ, РежимЗаписи,. . .	SDL3 для Desktop (MinGW): Вывод текста со шрифтом TTF с помощью библиотеки SDL3_ttf на Си и C++ 8Observer8 24.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-text-sdl3-c. zip finish-text-sdl3-cpp. zip	Жизнь в неопределённости kumehtar 23.03.2026 Жизнь — это постоянное существование в неопределённости. Например, даже если у тебя есть список дел, невозможно дойти до точки, где всё окончательно завершено и больше ничего не осталось. В принципе,. . .
Модель здравоСохранения: работники работают быстрее после её введения. anaschu 23.03.2026 geJalZw1fLo Корпорация до введения программа здравоохранения имела много невыполненных работниками заданий, после введения программы количество заданий выросло. Но на выплатах по больничным это. . .	Контроль уникальности заводского номера - вариант №1 Maks 23.03.2026 Алгоритм контроля уникальности заводского (или серийного) номера на примере документа выдачи шин для спецтехники с табличной частью. Данные берутся из регистра сведений, по которому настроено. . .	Хочу заставить корпорации вкладываться в здоровье сотрудников: делаю мат модель здравосохранения anaschu 22.03.2026 e7EYtONaj8Y Z4Tv2zpXVVo https:/ / github. com/ shumilovas/ med2. git	Программный отбор элементов справочника по группе Maks 22.03.2026 Установка программного отбора элементов справочника "Номенклатура" из модуля формы документа. В качестве фильтра для отбора справочника служит группа номенклатуры. Отбор по наименованию группы. . .