Странный глюк в цикле

[alex90aa]

Привожу код:

for(i=0;i<move.length;i++)

{

left2=left0+Number(move[1]);

top2=top0+Number(move[2]);

width2=Number(move[3]);

height2=Number(move[4]);

link=move[0];

title=move[5];

//alert(title);

document.write("<table title='"+title+"' id='t"+i+"' style='cursor:pointer; position:absolute; left:"+String(left2)+"; top:"+String(top2)+"; width:"+String(width2)+"px; height:"+String(height2)+"px;'><tr><td title='"+title+"'> </td></tr></table>")

if (document.addEventListener) { // Gecko

document.getElementById("t"+i).addEventListener('click', function() {location.href=link; }, false);

} else if (document.attachEvent) { // IE, Opera

document.getElementById("t"+i).attachEvent('onclick', function() { location.href=link; });

} else {

document.getElementById("t"+i)['onclick'] = function() { location.href=link; }

}

}

переменная link берется из массива в зависимости от цикла. затем для таблицы с соотв id добавляется событие.

Проблема: получается что у всех table одинаковое событие, т.е. все переходят на одну и ту же ссылку, которая явялется последней ссылкой в массиве (массив выводил, все ссылки разные)

Edited August 24, 2009 by alex90aa

[dyadya]

Я бы сделал объявление link да и всех переменных через var

- - - - - 
var link=move[i][0];
var title=move[i][5];

Edited August 24, 2009 by dyadya

[s0rr0w]

Замыкания нужны. Или не нужны, смотря как переделывать код

[alex90aa]

Замыкания нужны. Или не нужны, смотря как переделывать код

а можно попдробнее?

[Great Rash]

Пускай у нас есть массив ссылок, и наша задача — сделать так, чтобы при клике на каждую выводился алертом ее порядковый номер. Первое решение, что приходит в голову, выглядит так:

for (var i = 0; i < links.length; i++) {
   links[i].onclick = function() {
	  alert(i);
   }
}

На деле же оказывается, что при клике на любую ссылку выводится одно и то же число — значение links.length. Почему так происходит? В связи с замыканием объявленная вспомогательная переменная i продолжает существовать, при чём и в тот момент, когда мы кликаем по ссылке. Поскольку к тому времени цикл уже прошёл, i остаётся равным кол-ву ссылок — это значение мы и видим при кликах.

Решается эта проблема следующим образом:

for (var i = 0; i < links.length; i++) {
   (function(i) {
	  links[i].onclick = function() {
		 alert(i);
	  }
   })(i);
}

© Хабрахабр http://habrahabr.ru/blogs/webdev/38642/ (сервера у них лежат, поэтому запостил так)

А вообще гуглите батенька.

[alex90aa]

Простите за мою тупость, но я не смог понять что надо, поэтому решение своей проблемы сделал в PHP

[Great Rash]

Дык это... я ж все написал?! Чего не понятного то?