Цикл: неправильный счет

[TanaTiX]

Сабж.

for (n = 2; n <= 6; n++){
     _root.top_menu["top_b" + n].onPress = function(){
          trace(n);
     }
}

При нажатии на соответствующую кнопку выдает 7. Т.е. с самого начала ролика происходит счет до 6, а потом при нажатии плюсуется еще 1 и выносится в трэйс.
Что не правильно(желательно с объяснениями) в этом коде и как это исправить?

[iNils]

1. Персонально вам говорю, перед вставкой код, форматируйте его. Собюлюдайте табуляцию, ставте между операндами пробелы и тп. Это упрощает чтения кода глазами, а у вас каша.
2. Избегайте писать конструкции n <= 6, пишите лучше n < 7
3.

for (n = 2; n < 7; n++) {
	_root.top_menu["top_b" + n].n = n;
	_root.top_menu["top_b" + n].onPress = function () {
		trace (this.n);
	};
}

[udaaff]

for (n = 2; n < 7; n++) {
	var top_b = _root.top_menu["top_b" + n];
	top_b.n = n;
	top_b.onPress = function() {
		trace(this.n);
	}
}

[TanaTiX]

Спасибо, работает.

Сообщение от iNils

1. Персонально вам говорю, перед вставкой код, форматируйте его. Собюлюдайте табуляцию, ставте между операндами пробелы и тп.

ок, учту

Сообщение от iNils

2. Избегайте писать конструкции n <= 6, пишите лучше n < 7

Почему?
И если не сложно, можно объяснить 2ю строчку кода?

[Mr. Fixit]

MovieClip - динамический клас. И ты просто создаешь ему свойство, в даном случае, n и присваиваешь текущее числовое значение.

[iNils]

Почему?

Простая оптимизация, <= требует чуть больше операций (на уровне байт кода), чем <.

И если не сложно, можно объяснить 2ю строчку кода?

В строки trace(n); n это имя переменной изменяемое циклом, а не ее значение.
Поэтому втроая строчка создает локальную переменную внутри кнопки и присваивает ее текущие значение n цикла.
То есть у вас на самом деле два n: локальное в кнопке и у предка.

[TanaTiX]

У меня внутри мувика при нажатии должна проигрываться анимация. Я по аналогии внутри цикла ввожу:

top_b.menu_title_mc.gotoAndPlay(21);

Но тем не менее ничего не проигрывается, т.е. в данному случае top_b опять воспринимается как глобальная переменная, а не как свойство мувика. Что нужно исправить?

[udaaff]

this.menu_title_mc.gotoAndPlay(21);

[TanaTiX]

udaaff, я это пробовал, но не работает. Если написать тот код, который я написал вначале, то срабатывает не на нажимаемой кнопке, а только на последней из списка однотипных

[udaaff]

Должен срабатывать...

for (n = 2; n < 7; n++) {
	var top_b = _root.top_menu["top_b" + n];
	top_b.n = n;
	top_b.onPress = function() {
		trace(this.n);
		this.menu_title_mc.gotoAndPlay(21);
	}
}