Почему переменная цикла конкатенируется?

Question

[Urukhayy]

Есть цикл for(var i in obj)
Если мы выведем в цикле
alert(i+7);
То получим не сумму чисел, а строку типа - 07, 17, 27 и .т.д
Как суммировать его?

Второй вопрос:
Почему переменная цикла(i), являющаяся аргументом авто-вызываемой функции в этом же цикле, принимает значения не по порядку? То есть в функцию передаются все итерации в различном порядке: 3,2,6,4,0,1....n. Как это решить?

Answer 1

[Иван Гонтаренко]

потому что вероятно i строка. т.е. typeof i == 'string'
сделайте
alert(parseInt(i)+7);

p.s.
10 + 20 = 30
"10" + 20 = "1020"

Comments

[YemSalat]

parseInt(i, 10) - для старых браузеров (ИЕ8 например)

[yttrium]

+i+7 или i*1+7

[Иван Гонтаренко]

yttrium: или ~~i + 7;
=)

[Urukhayy]

Дак а что лучше, Иван Гонтаренко: ? )

[yttrium]

лучше parseInt, оно делает именно то как читается, а читабельность кода это очень важно

[Иван Гонтаренко]

Urukhayy: да, верно говорит yttrium , читабельность - это важно. Хотя по сути это ( ~~i + 7; ) самый быстрый способ, но мало значительна, выбирайте читабельность в данном варианте.

Answer 2

[Alexander Sydorenko]

Первый ответ. Потому что (элемент зачеркнуто) ключ массива obj у вас судя по всему строка а не число. А если к строке прибавить число, получим строку - 07.
Второй ответ. Потому что автовызваемая функция - замыкание. Замыкается на аргументе, который передается. Передается по прорядку, начинает выполнятся по порядку, заканчивает выполнятся в произвольном порядке. Выполняется когда приходит ее очередь, используя замкнутый в себе аргумент. Решение - не использовать автовызываемую функцию в цикле :)

Comments

[Urukhayy]

А как быть, если она мне нужна? ) Для того, чтобы в ajax, который в цикле, передать i цикла.

[Urukhayy]

А если я буду использовать простой вызов функции, которая создана вне цикла? (В глобальной области). Они будут по порядку?

[yttrium]

два неверных ответа

[Alexander Sydorenko]

Urukhayy: Если функция будет принимать аргументом индекс по которому итерирует цикл, и не будет асинхронной, то будут по порядку. При этих же условиях, даже вызов в цикле вместо (function(){write(i)})(i)
(function(a){doSome(a)})(i); приведет к упорядочиванию. Если на прав, то возможно yttrium: поправит.

[Urukhayy]

То есть в ajax запросе по любому будут не по порядку?

[Alexander Sydorenko]

Urukhayy: Да, однозначно.. так как возврат из запроса асинхронный, и порядок окончания выполнение не будет гарантировано совпадать с порядком окончания выполнения даже тех же самых запросов к серверу. Так как среда, положение звезд, и т.д. Для синхронизации асинхронных процессов нужно смотреть в сторону jQuery deffered или просто js deffered. В любом случае для того чтобы гарантировать порядок выполнения асинхронных запросов, Вам перед началом каждого шага необходимо дождаться возврата из предыдущего.

[Urukhayy]

Alexander Sydorenko: Тогда что посоветуете в ajax запросе на выборку из БД MySQL, если нужно строго учесть порядок строк? То есть чтобы каждая строчка из БД вернулась под таким же индексом, как итерация цикла..

[Alexander Sydorenko]

Urukhayy: А зачем возникла необходимость выбирать по строчке, а не массив строк отсортированный order by по в требуемом порядке? А бы строил запрос который пачкой вернет мне массив, пусть даже с фейковым полем индекс если оно надо но его нет..

[Urukhayy]

Alexander Sydorenko: Так оно и есть) Возвращаю массив. Но дело в том, что когда я "обрабатываю" вернувшееся в JS, я использую цикл for(var i in data), где data - массив результата. А так как в функцию поступает беспорядочное значение i, соответственно и материал публикуется без порядка.

[yttrium]

for(var i in data) - возвращает ключи, которые не упорядочены! У него есть несколько тонкостей
используйте for(var i = 0; i

[Urukhayy]

yttrium: Только что попробовал, все точно также. Использую такое же замыкание как и вы. В общем всё проходит так:

Есть ajax запрос на выборку массива новостей. В свойстве success{} - я объявляю цикл(пробовал и тот, и дугой). В этом цикле у меня идет всё, как вы описали в своем ответе. И в итоге все ведет в глобальную функцию. В ней я получаю i - alert(i), и оно беспорядочно.

[Urukhayy]

yttrium: Хотя нет, замыкание в самой функции(в глобальной области) я не использовал, т.к. значение i не постоянное, тобишь как и должно быть, но беспорядочное

[yttrium]

Urukhayy: ок. тогда нужно синхронизировать всё это дело. Добавлять в очередь и после этого уже обрабатывать

[Alexander Sydorenko]

Urukhayy: Покажите на jsfiddle код, пусть даже на рабочий там. У вас на ajax.success приходит массив, верно? и? вы в цикле берете for(var row in data) data[row] и что с ним делаете? Если массив пришел упорядоченный, все должно быть ок, какой то локальный косяк..

[Urukhayy]

Alexander Sydorenko: yttrium: Вот код, там первый варинт, с автовызываемой функцией. Но даже если пробовать выносить функцию в глобальную область, тоже самое. jsfiddle.net/vtLz5897

[Alexander Sydorenko]

Urukhayy: Там они и будут в беспорядке.. вы же повторно для каждой строки в массиве ответов делаете запрос.. А нельзя упорядочить ответ сервера? что бы в одном аякс-ответе Вы получили все что Вы довытягиваете в процессе?

[Urukhayy]

Alexander Sydorenko: А там такая относительность: к одной новости n-ное кол-во картинок....

[Alexander Sydorenko]

Urukhayy: верните это все из сервера, по перовому ajax запросу и не мучайте его лишний раз. На генерации ответа к каждой новости, приложите список необходимых картинок. И Веб сервер скажет вам спасибо, и с синхронизацией разбираться для такой задачи не придется.

Answer 3

[yttrium]

for (var i in obj) цикл предназначеный для перечисления ключей объекта, иногда можно использовать для массивов, но очень осторожно.
for(var i = 0; i<a.length; i++) цикл для массивов, здесь индексы будут по порядку

UPD:
А чтобы переменную i не замыкало, её нужно передать в функцию в качестве аргумента

function func(i){
    return function(){
        console.log('Переменная i = ' + i + ', тип ' + typeof i);
    }
}
for(var i in [9]) {setTimeout(func(i),0);}
for(var i in {1:2}) {setTimeout(func(i),0);}

в хроме выдает
Переменная i = 0, тип string
Переменная i = 1, тип string
Как видим оба значения типа string. Это происходит потому, что цикл "for in" приводит ключи и индексы к типу string

Comments

[Urukhayy]

Кстати, оптимальней же будет такой вариант: for(var i = a.length-1; i != -1; i--)
Ведь в Вашем варианте в каждой итерации будет вызываться метод length, а в моем только 1 раз. Не так ли?

[yttrium]

Urukhayy: a.length это не метод, это свойство

[Urukhayy]

Комментарий к Вашему UPD:
А зачем в функции вы возвращаете еще функцию? Да, это замыкание, ног зачем он тут?

[yttrium]

Urukhayy: Классический пример:

function func(i){
        console.log('Переменная i = ' + i + ', тип ' + typeof i);
}
for(var i = 0; i<[1,2,3,4,5].length; i++){setTimeout(function(){func(i)},0);}

Выдаст пять одинаковых строк

[Urukhayy]

yttrium: Ааа... из-за цикла?
И да, почему Вы используете SetTimeout?

[yttrium]

"эмитация" асинхронности