OnRelease в цикле

[diclofoss]

У меня есть цикл:


for (i=0; i<total; i++) {
image[i] = xmlNode.childNodes[i].childNodes[0].firstChild.nodeValue;
link[i] = xmlNode.childNodes[i].childNodes[1].firstChild.nodeValue;
...
ls = link[i];
_root['clip_d']['mc'+i].onRelease = function() {
trace(ls);
};
}


В чем моя ошибка, почему при нажатии по кнопке функция берет последний ls а не те которые были установлены в цикле.

[Molecula]

А так:
for (i=0; i<6; i++) {
_root['clip_d']['mc'+i].onRelease = function() {
trace(xmlNode.childNodes[this._name.substr(2,2)].childNodes[0].firstChild.nodeValue);
trace(xmlNode.childNodes[this._name.substr(2,2)].childNodes[1].firstChild.nodeValue);
}
}

[diclofoss]

спасибо. а что это за name.substr(2,2)]. просто выдернуто айди из имени?

[Molecula]

Не, оно через два сисвола в имени мува берет два других.
Т.е. у mc2 - это будет 2, а у mc23 - 23, но у mc254 - будет 25, что бы было 254 нужно так написать name.substr(2,3).

[Kikasso]

Т.е. у mc2 - это будет 2, а у mc23 - 23, но у mc254 - будет 25, что бы было 254 нужно так написать name.substr(2,3).
Учите мануал. Нафига тут второй параметр?
var name1 = 'mc1234';
var name2 = 'mc5';
trace( '>' +name1.substr(2) + '<');
trace( '>' +name2.substr(2) + '<');
trace( '>' +name1.substring( 2 ) + '<' );
trace( '>' +name2.substring( 2 ) + '<' );

[Molecula]

If length is not specified, the substring includes all the characters from the start to the end of the string.

Кстати, да.
что там за пропеллер у тебя?

[Kikasso]

что там за пропеллер у тебя?Это когда-то костюмы на Кракатук делали.

[Molecula]

:drinks:

[etc]

А так:
for (i=0; i<6; i++) {
_root['clip_d']['mc'+i].onRelease = function() {
trace(xmlNode.childNodes[this._name.substr(2,2)].childNodes[0].firstChild.nodeValue);
trace(xmlNode.childNodes[this._name.substr(2,2)].childNodes[1].firstChild.nodeValue);
}
}
И вам лень копипастить «xmlNode.childNodes[this._name.substr(2,2)].childNodes[1].firstChild.nodeValue» сотню раз? Известно ли вам, что строки подобного плана не только ухудшают читабельность, но ещё и зверско тормозят плеер?

[Molecula]

А как же быть?

[etc]

Делать ссылки на ноды и работать с ними, а не копировать по сто раз один и тот же путь.

[Molecula]

Понял.

[diclofoss]

спасибо за замечания... пока конечно подтормаживает у меня больше из за загрузки превъюшек:)
Выглядит вот так вот:
http://www.madeinparis.ru/inap2.php

В дальнейшем хочу ещё прелоадинг написать для каждой картинке:)
Пока демка и не всё ещё реализованно. Но нужно в скорейшем образом избавиться от кривого джава скрипта написанного ранее.

[udaaff]

У меня есть цикл:


for (i=0; i<total; i++) {
image[i] = xmlNode.childNodes[i].childNodes[0].firstChild.nodeValue;
link[i] = xmlNode.childNodes[i].childNodes[1].firstChild.nodeValue;
...
ls = link[i];
_root['clip_d']['mc'+i].onRelease = function() {
trace(ls);
};
}


В чем моя ошибка, почему при нажатии по кнопке функция берет последний ls а не те которые были установлены в цикле.

Еще парочка вариантов:
for (i=0; i<total; i++) {
image[i] = xmlNode.childNodes[i].childNodes[0].firstChild.nodeValue;
link[i] = xmlNode.childNodes[i].childNodes[1].firstChild.nodeValue;
//
ls = link[i];
_root['clip_d']['mc'+i][ls] = ls;
_root['clip_d']['mc'+i].onRelease = function() {
trace(this.ls);
};
}
for (i=0; i<total; i++) {
image[i] = xmlNode.childNodes[i].childNodes[0].firstChild.nodeValue;
link[i] = xmlNode.childNodes[i].childNodes[1].firstChild.nodeValue;
//
ls = link[i];
_root['clip_d']['mc'+i].onRelease = genOnRelease(ls);
}
function genOnRelease(ls) {
return function () {
trace(ls);
};
}

[diclofoss]

спасибо.