Как изменить данные с помощью цикла в jquery?

Question

[Максим Савичев]

Суть такова, у меня есть список данных в таблице и нужно в режиме онлайн с помощью стороннего сервиса эти данные высчитывать. А именно нужно высчитывать онлайн серверов с помощью ip.

//Считываю количество строк таблицы
var sum = $('.serverlist').length;
console.log(sum);
//Вытаскиваю с помощью jQuery данные ip адресов
//Элементы в html имеют id = "ip1".."ip8"
for(var i=1; i<=sum; i++ ) {
    var ip = $("#ip"+i).html();
    var url = "http://mysite=" + ip;
    $.getJSON(url, function (json) {
        var onl = console.log(json.player_online);
        var max =console.log(json.player_max);
        console.log(i);
        $("#online"+i).text(onl + "/" +max);
    });
}

В консоли все отлично отображается и выводится, но я не могу изменить элемент "#online+i" потому что переменная "i" выдает либо success, либо 4.
Пробовал вписывать "i" во входные параметры функции, получалось так же.

Мне нужно, чтобы "i" в строке "$("#online"+i).text(onl + "/" +max);" принимала значение "i" из цикла. Но я не могу понять как это можно сделать.

Answer 1

[Алексей Уколов]

Вам нужно сделать замыкание, чтобы работать с копией переменной, а не с ней самой, поскольку код внутри цикла выполняется после его завершения, когда переменная i имеет максимальное значение.
https://learn.javascript.ru/closures-usage#армия-ф...

Answer 2

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

Перемнные в цикле объявлять нельзя!

for(var i=1, ip, url; i<=sum; i++ ) {
    ip = $("#ip"+i).html();
    url = "http://mysite=" + ip;
    (function(b){
        $.getJSON(url, function (json) {
           console.log(json.player_online, json.player_max, b);
           $("#online"+b).text(onl + "/" +max);
        });
    })(i);
}

Comments

[Максим Савичев]

Про переменные учту, спасибо за ответ

[Алексей Уколов]

Не нельзя, а не рекомендуется. Ваш вариант ничуть не лучше, имхо. Переменные стоит объявлять отдельно в самом начале функции, здесь вы их уж очень сильно закамуфлировали.

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

Алексей Уколов: область видимости одна, нельзя 100 раз в ней объявлять одни и те же переменные. Почему не рекоммендуется?

[Алексей Уколов]

Потому что даже если в цикле написать var ip, и выполнить цикл 100 раз, объявление переменной "всплывет" в начало функции, никакой ошибки это не вызовет и код будет работать, как ожидалось. Никаких ограничений в этом плане не накладывается, но это считается плохой практикой, именно поэтому "не рекомендуется", а не "нельзя".

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

Алексей Уколов: а, мы про разные "нельзя", я про то, что в стиле правильного кода, вы про физическое =)