Успевает ли передаться правильный запрос в движок в моем случае?

Question

[axeax]

Есть кусок кода на nodejs. Приблизительно выглядит это так:

mysql.query('SELECT * FROM table WHERE id = '+arr[i]+';',  function(err, res){...});
delete arr[i];

Суть вопроса: успевает ли передаться правильный запрос в движок? Почему то через раз я получаю нулевой результат выполнения (нужная строка есть 100%, пробовал один и тот же ID выбирать.) Есть такое чувство что элемент массива удаляется раньше и в запрос идет undefined. Если после запроса убрать delete arr[i]; то всё нормально. При помещении delete arr[i]; в callback происходит тоже самое.

Answer 1

[Vit]

Оно и не должно работать, потому что так нельзя писать.
Если удаление не в каллбэке - то оно сработает раньше чем запрос и arr[i] будет undefined.
Если в каллбэке, то к моменту вызова каллбэка a[i] будет являться вообще не тем, чем вам хотелось бы, и побьются "чужие" данные.

В общем нельзя так делать, как у вас.

Comments

[axeax]

Тогда как правильно? Удалять надо полюбому, а в каллбэке а[] будет являться тем чем надо потому что это глобальный массив (элемент i для примера, там нет цикла)

[axeax]

фу блин имена перепутал, массив в общем один и тот же везде) но это я только тут перепутал так что проблема остается

[Vit]

> а в каллбэке а[] будет являться тем чем надо
Нет. Вы замыкаете переменную i, ссылку на нее, а она к моменту вызова уже другая. Не верите - откройте любую яваскриптовую консоль и напишите такое:

var a = [1, 2, 3, 4, 5];
var i = 0;
var f = function(){delete a[i]};
i = 3;
f.call();

И посмотрите, что у вас будет в массиве a.

Вы лучше скажите, почему вам "полюбому надо удалять" и понимаете ли вы, что именно делает оператор delete в яваскрипте?

[axeax]

> Нет. Вы замыкаете переменную i, ссылку на нее, а она к моменту вызова уже другая. - это я понимаю
Но перед вызовом функции i не меняется.
Удалять полюбому надо - но не сам элемент как раз, чтобы не произошло смещения. Тоесть после такого удаления при обращении к этому элементу я должен видеть undefined.
И как тогда объяснить что работает это через раз?
Приведу такой код:
i = 5;
function a(b){console.log('r'+b)};
a(i);
i = 2;
в функцию ведь попадёт 5 а не 2. Так почему тогда в запрос попадает undefined? Это особенность js или mysql?
То же самое:
i = 5;
function a(b, callback){
callback();
}
a(i, function(){i=2}); //ошибка то почему то возникает при вызове, т.е. в этом примере в ф-ию передается 5, а только потом i становится 2. А в моём случае i при вызове уже становится 2

[Vit]

>Так почему тогда в запрос попадает undefined? Это особенность js или mysql?
Это даже не особенность, это нормальное поведение асинхронного яваскрипт кода с замыканиями.
А ваши примеры из последнего комментария не имеют отношения к рассматриваемого вопросу, потому что, во-первых, там нет асинхронного кода, во-вторых нет замыканий.
Рекомендую поглубже разобраться с такими фундаментальными понятиями яваскрипта, как асинхронное выполнение, EventLoop, замыкания, области видимости.

Возвращаясь к вашей задачи. Я так понимаю, вы в цикле делаете запросы. После выполнения i-го запроса вам надо i-й элемент массива сделать undefined. Делайте запрос в отдельной функции, не создавая замыканий:

function oneRequest(a, i) {
//делаем запрос с a[i], в каллбэке a[i] делаем undefined.
}

for (i=0; i<n; ++i) {
  oneRequest(a, i);
}

Но даже в этом случае например после цикла у вас не факт, что все значения будут undefined и не факт, что к этому моменту выполнятся все запросы, потому что они асинхронные.

Если вы точно хотите поймать момент, когда выполнятся все запросы, то заведите счетчик в глобальной переменной, в каллбэке увеличивайте его на 1 и там же (в каллбэке) проверяйте, достиг ли счетчик размера массива. Если достиг - значит этот тот самый момент когда все запросы выполнены.

[axeax]

В том и дело что не цикл,я написал в первом комментарии об этом

[axeax]

Ладно, в общем спасибо, пока сделаю костыль и покурю мануалы)