Почему не работает обновление сущности в базе данных при переносе строк в таблице?

Question

[Vovan1995]

Есть таблица с данными, после переноса строки при помощи скрипта sortable , прохожусь по всем строкам и беру айдишник с первого столбца, потом передаю в функцию айдишник и номер столбца для обновления базы данных через рест и аякс.
Этот код не работает, почему и как можно улучшить?

$(function () {
    $("#vk-request-body").sortable({
        connectWith: ".connectedSortable",
        stop: function () {
            var table = document.getElementById("vk-request-table");
            for (var i = 1, row; row = table.rows[i]; i++) {
                var id = row.cells[0].textContent;
                updateVkRequestAfterSort(id, i);
            }
        }
    }).disableSelection();

});

function updateVkRequestAfterSort(id, rowNumber) {
    var url = "/vk/request/update/" + id;
    var mass = {
        numberVkField: rowNumber
    };
    var emp = JSON.stringify(mass);
    $.ajax({
        url: url,
        data: emp,
        contentType: "application/json",
        type: 'POST',
        dataType: 'JSON',
        success: function (returnObj) {
        },
        error: function (error) {
            console.log(error);
        }
    });
}

Answer 1

[Dies Irae]

У вас ошибка в цикле for:

for (var i = 1, row; row = table.rows[i]; i++) {
                var id = row.cells[0].textContent;
                updateVkRequestAfterSort(id, i);
            }

for (начало; условие; шаг) {
  // ... тело цикла ...
}

Условие задано неверно. Оператор сравнения (==) или (===). Да и в логике ошибка. Наверное должно быть так: i < table.rows.lenght
Вообще, советую проходиться по всем элементам списка и вытаскивать их позицию из ui.item.index(), а не наоборот. Исключение - если позиция более важна чем элемент списка.
Вот тут небольшой пример по определению позиции - codepen.

Comments

[Vovan1995]

проблемка была немного в другом, но все равно спасибо. Про получение индексов таким способом было полезно узнать)) ну и цикл нормальный сделал