еще раз addEventListener в Array

[misha_yu]

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

Задача добавить лисенер к массиву мувиклипов. Массив вложен в клип на стэйдже который в свою очередь маскирован.
Должна получится прокрутка превьюшек фотографий, при клике на одну из которых должна подрузится большая фотка.

Вопрос. Если объект в массиве один - лисенер добавляется, если больше - нет.

addEventListener(Event.ENTER_FRAME,loopFunction);
function loopFunction(event:Event) {
simplePreviewAdd()}
 
public function simplePreviewAdd():void {
var preview:PreviewClass=new PreviewClass  ;
addChildAt(preview,0);
previewArray.push(preview);
previewArray[addedPreviewCount].addEventListener(MouseEvent.MOUSE_OVER,previewArrayMouseDown,false, 0, true);
trace (" MOUSE_OVER ")
addedPreviewCount++;
}

пробовал вытащить имя клипа при наведении на него чтобы назначить лисенер при наведении на клип event.target

trace(event.target.name);

он возвращает : root1
- ну это ваще че то ни то.

Спасибо

Добавлено через 4 минуты
сори, в код иф не дописал.
на самом деле так:

addEventListener(Event.ENTER_FRAME,loopFunction);
function loopFunction(event:Event) {
if (addedPreviewCount<previewArray.length) {
simplePreviewAdd()}}
 
public function simplePreviewAdd():void {
var preview:PreviewClass=new PreviewClass  ;
addChildAt(preview,0);
previewArray.push(preview);
previewArray[addedPreviewCount].addEventListener(MouseEvent.MOUSE_OVER,previewArrayMouseDown,false, 0, true);
trace (" MOUSE_OVER ")
addedPreviewCount++;
}

суть такая

[s8000_1]

Что-то как-то кривовато...
Во-первых нет "()" при создании экземпляра класса, во-вторых совершенно незачем таким образом назначать слушателей. Вот так все должно работать:

public function simplePreviewAdd():void {
 var preview:PreviewClass = new PreviewClass();
 preview.addEventListener(MouseEvent.MOUSE_OVER, previewMouseDownHandler);
 addChildAt(preview,0);
 previewArray.push(preview);
}

В-третьих не понимаю, почему бы не добавлять превью через for(...){...}, а не событие ENTER_FRAME

[DarkLight]

s8000_1, в AS3 () при создании экземпляра писать необязательно

[etc]

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

s8000_1, в AS3 () при создании экземпляра писать необязательно

Обязательно. Без вариантов. Не смотря на то, что позволяет так делать.

[ZlPredator]

Не обязательно.
AS3 это, по сути, усовершенствованный ECMA262. Привожу вырезку из спецификации по ECMA262:

MemberExpression :
...
new MemberExpression Arguments

NewExpression :
MemberExpression
new NewExpression

Arguments :
( )
( ArgumentList )

LeftHandSideExpression :
NewExpression
CallExpression

В случае использования стиля NewExpression (без скобок), не обсчитывается список параметров и вызывается конструктор по умолчанию (11.2.2 спецификации).

Говорить что использовать не правильно идеологически тоже нельзя. В C++, например, много ситуаций когда НЕОБХОДИМО использовать синтаксис без скобок.

[misha_yu]

большое спасибо, все заработало

[terbooter]

Скобочки-то лучше писать...

[ZlPredator]

Почему? Не написание скобок не может привести ни к одной ошибке (в отличие от ';'), абсолютно разрешенная запись.

[BlooDHounD]

ZlPredator, это такая же идеологическая борьба, как писать 5 == i, а не i == 5. ошибка может подкрасться незаметно %)

[iNils]

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

Почему? Не написание скобок не может привести ни к одной ошибке (в отличие от ';'), абсолютно разрешенная запись.

Дело не в ошибке, а в культуре написания кода.
Также как не стоит писать

if (условие) действие

надо всегда писать

if (условие) {
	действие
}

Код более читабельный