Форум Flasher.ru - обращение сразу ко всем экземплярам определенного класса

Форум Flasher.ru (http://www.flasher.ru/forum/index.php)

- ActionScript 3.0 (http://www.flasher.ru/forum/forumdisplay.php?f=83)

Цитата:

можно "забыть" удалить экземпляр из статик массива, когда объект станет ненужным

Так ведь можно и со сцены забыть его удалить =) Выше предложена реализация (ADDED_TO_STAGE, REMOVED_FROM_STAGE) при которой забыть не получится. В данном случае солидарен со всеми, кто за статичый массив

Не вижу смысла отягощать дополнительными функциями класс, заморачиваться на слушание событий типа ADDED_TO_STAGE / REMOVED_FROM_STAGE. Хранить это все еще где-то. Когда задача поставлена предельно просто и ясно: перебрать все объекты на сцене определенного класса. Зачем вот надо придумывать лишнее, когда конкретно в данном случае можно обойтись одной-единственной функцией. А для гибкости, можно класс динамически подсовывать:

Код AS3:

function getChildrenOfClass( container:DisplayObjectContainer, classToMatch:Class ):Array

{

        if( !container || !classToMatch )

                return null;

 

        var outInstances:Array = [];

        var d:DisplayObject;

        var i:int = container.numChildren;

        while( i-- )

        {

                d = container.getChildAt( i );

 

                if( d is classToMatch )

                        outInstances.push( d );

        }

 

        return outInstances;

}

Пример юзания:

Код AS3:

var shapes:Array/**Shape*/ = getChildrenOfClass( root as DisplayObjectContainer, Shape );

(Код не компилил, мог пропустить какую-нибудь мелкую ошибку).

Автор не ставил задачу "максимально быстро", где возможно было бы оправдано пихать лишний функционал в кастомные классы.
Приведенный пример максимально гибок / абстрактен и может быть использован независимо от реализации кастомных классов.

@i.o.:

а если объекты искомого класса находятся внутри другого объекта находящегося на сцене, предлагаете рекурсивно бегать по всем DisplayObjectContainer'ам?

mayakwd, а что у вас на сцене тысячи объектов валяются, что это может сказаться на переборе, даже рекурсивном? Если да, то постом выше сказано:

Цитата:

Автор не ставил задачу "максимально быстро", где возможно было бы оправдано пихать лишний функционал в кастомные классы.

В таком варианте - я ничего не имею против, если от этого есть реальная выгода.

@i.o.:

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

i.o., не было задачи "перебрать все элементы", была задача "выбрать нужные". А ваш вариант хоть и имеет правл на жизнь, но вданном контексте ИМХО менее "ООПшный", мы же не gc пишем.

Вот статик массив, как раз, есть частное решение. То что я предложил - общее.

TanaTiX
Если вам платят за количествно набранных строк и за "только ООПшный код", а не за здравый взгляд на мир вещей, то я понимаю ваше несогласие.

Цитата:

Сообщение от mayakwd (Сообщение 943180)

@i.o.:

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

ИМХО для продуктвиности как раз и делаются частные решения ;)))
Так как условия и цели в разных случаях могут различаться, не бывает универсального кода.

Цитата:

Сообщение от andrew911 (Сообщение 943183)

пожалуй я не буду спорить.
вырвать из контекста можно что угодно )

Цитата:

я считаю что код должен быть продуктивным

Ну вот вы меня прям-таки вынудили сделать тесты для своего непродуктивного кода.. Честно говоря ожидал хуже.
В упрощенном виде тест такой:

Код AS3:

//{ filling

 

var myContainer:Sprite = new Sprite();

 

i = -1;

while( ++i < 100*1000 )

        myContainer.addChild( new Shape() );

 

i = -1;

while( ++i < 100*1000 )

        myContainer.addChild( new Sprite() );

 

//} filling

 

 

var tm01:int = getTimer();

 

result = getChildrenOfClass( myContainer, Shape );

 

tm01 = getTimer() - tm01;

 

// tm01: 35ms

Как мы видим, провал в 35мс появляется только при переборе и проверке 200'000 (двухсот тысяч) детей в контейнере.
Как же непродуктивно!... :p:p