Форум Flasher.ru - импорт мувиклипа из библиотеки по имени массива

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

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

Цитата:

Сообщение от BlooDHounD (Сообщение 904007)

а у Вас целый букет быдлокода:
1. static до нэймспэйса.
2. название переменной MovieClipsClassesObject с большой буквы.
3. не понятно почему оно является переменной, а не константой.
4. если приватная, то почему без подчёркивания?
5. объект создаёте через литерал { }, а не через new Object(). советую второй вариант.
6. метод называется newMovieClip а параметром передаётся imageName.
7. с названиями в принципе беда: MovieClipsClassesObject, movieClipsClass. непонятно.
8. в качестве результат возвращается экземпляр MovieClip, что само по себе плохо. а что, если вам нужно будет сделать наследника спрайта? или битмапа? или ещё чего угодно? напишите ещё один такой же метод?
9. возвращаете Вы готовый экземпляр, но опираясь на пункт 7, возникает вопрос: а что если надо будет передать параметр в конструктор класса?
10. непонятно откуда берётся resourceLoader, и почему он один. а если подгружаемых библиотек будет много?

на счёт пункта 1 и 4, многие со мной не согласятся. я не очень настаиваю.

5. так было рекомендовано в соглашении Flex SDK coding conventions and best practices читал перевод тут http://racer242.blogspot.com/2008/05/flex_23.html

Цитата:

Object
Используйте вместо new Object() литералы объекта:
можно: {}
нельзя: new Object()

8. Ну во первых тема топика - мувиклипы. Во вторых у меня только три типа - мувиклип, битмап, sound. Мне удобнее использовать отдельные три метода, т.к. в них есть свои особенности, и для строгой типизации. Удобно когда можно писать addChild(newMovieClip("unit")) вместо addChild(MovieClip(newSomething("unit"))).
9.В конструктор класса MovieClip ничего передавать не надо. Там где надо(в битмап например) -там передается, там другая функция...
10. Много библиотек не планируется, откуда берется объясняется выше по коду. Приводить не стал, т.к. это выходит за рамки темы топика (обсуждаем как извлечь из библиотеки, а не как загрузить библиотеку)
1,2,3,4,6,7 - Спасибо, исправлю =)

Добавлено через 9 минут

Цитата:

Сообщение от BlooDHounD (Сообщение 904007)

написать, что типа этого, было бы логичнее:

Код AS3:

public static function getClass(name:String):Class

так конечно универсальнее, но у меня например:

Код AS3:

addChild(getBitmap("chest"));

addChild(newMovieClip("unit"));

а у Вас будет что-то типа

Код AS3:

addChild(new Bitmap( BitmapData(new getClass("chest"))));

addChild(MovieClip(new getClass("unit")));

5. если Вы будите опираться на эту конвенцию то вскоре встретитесь с проблемами. например у литерала {} не так как у всех работает перебор через for .. in
8. тема топика мувиклипы, только потому, что автор, видимо, перешёл к нас с АС2, в котором ничего другого и нету. к тому же даже гуру по привычки называю всю визуалку "мувиками". тупо привычка из старых времён.
9. ваше право. пишете 10 раз один и тот же рас так нравится =)
10. я уже понял, что Вы любите писать одноразовый код.

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

если у Вас в библиотеке не окажется указанного класса, то у вас произойдёт целый каскад ошибок:
1. new movieClipsClass() - Вы не проверяете существование класса. то есть будет ошибка.
2. даже если Вы поставите проверку в пункте 1 и вернёте null, то у Вас всё упадёт на addChild.

и ещё новый быдлокод:
не используйте для приведение типов конструкции вида MovieClip( object ). это конвертация а не приведение типов. используйте оператор as.

в общем как хотите.

Цитата:

Вы не проверяете существование класса. то есть будет ошибка.

Вы тоже не проверяете вроде? =)

Код AS3:

c = resourcesLoader.content.loaderInfo.applicationDomain.getDefinition( name ) as Class;

На данный момент меня устраивает что программа упадёт если нет нужного класса. В дальнейшем естественно воспользуюсь Вашим советом =)

Цитата:

не используйте для приведение типов конструкции вида MovieClip( object ). это конвертация а не приведение типов. используйте оператор as.

В той конвенции написано ровно наоборот.

Цитата:

Оператор "as" используйте только в том случае, если при ошибке приведения к типу, вы хотите получить в результате значение "null", а не генерацию исключения:
можно: IUIComponent(child).document
нельзя: (child as IUIComponent).document

Добавлено через 52 секунды

Цитата:

если Вы будите опираться на эту конвенцию то вскоре встретитесь с проблемами.

Посоветуйте, как какую конвенцию мне опираться? На какую например опираетесь Вы?

mickfallout, а зачем мне его там проверять? там у меня ошибки не будет. я не создаю экземпляра и уж тем более никуда его не добавляю. просто возвращаю результат.

конвертация и привидение - разные операции. у них разный смысл и разное назначение.

желаю удачи. продолжайте пользоваться конвенцией приводящей к ошибкам =) я пользуюсь здравым смыслом, и знанием специфики всего того, что использую.

BlooDHounD
Подскажи пожалуйста, а чем конвертация отличается от приведения?
А еще есть моменты когда надо отходить от конвенции?

Цитата:

Сообщение от Aloran (Сообщение 904136)

BlooDHounD
Подскажи пожалуйста, а чем конвертация отличается от приведения?

Приведение не делает преобразования из A в B.

Цитата:

Сообщение от Aloran (Сообщение 904136)

А еще есть моменты когда надо отходить от конвенции?

Есть.

Код AS3:

trace( '5' as Number ); // null

trace( Number( '5' ) ); // 5

 

trace( '<xml>' as XML ); // null

trace( XML( '<xml />' ) ); // <xml />

 

trace( new Sprite() as MovieClip ); // null

trace( MovieClip( new Sprite() ) ); // TypeError #1034

приведение типа всегда возвращает ссылку на тот же объект, что у вас и был, либо null.
конвертация при возвращает ту же ссылку, только если типы совпадают, иначе она пытается создать объект нового типа на базе нового. если и этого не выходит она выбрасывает ошибку.

лично я и не помню, где я в последний раз использовал конвертацию.
для преобразования текста в число я использую parseInt и parseFloat.
для приведения к строке использую метод toString.
для создания XML я всегда пишу new XML().
ну и т.д. то есть убираю посредников.

Конвертация вроде как шустрее работает.

Цитата:

Сообщение от dimarik (Сообщение 904323)

Конвертация вроде как шустрее работает.

Код AS3:

public function test() {

        const l:uint = 1e6;

        var i:uint;

        var t:Number;

 

        t = getTimer();

        for ( i=0; i<l; i++ ) {

                this as Sprite;

        }

        trace( getTimer() - t ); // 192

 

        t = getTimer();

        for ( i=0; i<l; i++ ) {

                Sprite( this );

        }

        trace( getTimer() - t ); // 236

}

а если местами поменять, то приведение почти в 2 раза рвёт. лично я уверен, что внутри конвертации сперва происходит приведение, поэтому оно быстрее быть не может. это как конвертацией в String, внутри, которой вызывается toString. так что сомнительно.

p.s.: сегодня прямо день перлов про быстродействие =) записать что ли все ...

По идее приведение должно быть быстрее - при нем не создается новый объект. Возвращается-то либо сам объект, который конвертируем, либо null (который вообще синглтон).

Код AS3:

var a:MovieClip = new MovieClip();

var b:DisplayObject = a as DisplayObject;

trace(a === b); // true

UPD
А это-то как?

Код AS3:

var a:MovieClip = new MovieClip();

var b:DisplayObject = DisplayObject(a);

trace(a === b); // true