DropShadowFilter и GlowFilter в Starling
Запись от Котяра размещена 16.01.2013 в 17:21
Вышла новая стабильная версия фреймворка Starling.
Из нового:
— GPU-фильтры (Blur, Drop Shadow, Glow, Color Matrix)
— улучшены твины
— управление ресурсами(Asset Managements)
— режимы масштабирования
Более подробно тут
Всё хорошо, но DropShadowFilter и GlowFilter сделаны не как нормальные классы, а через фабричные методы класса BlurFilter.
Код AS3:var dropShadow:BlurFilter = BlurFilter.createDropShadow(); var glow:BlurFilter = BlurFilter.createGlow();
Сделал форк и добавил 2 класса.
https://github.com/k0t0vich/Starling...hadowFilter.as
https://github.com/k0t0vich/Starling.../GlowFilter.as
Послал пуллреквест автору старлинга, но он отказался его принять.
Считает , что фабричные методы удобнее и проще для пользователей:
Цитата:
Thanks a lot for the pull request! For now, I'll leave it like it is, though: I think most people will be able to simply use the methods and properties of the BlurFilter instead. I think this would be a nice extension, though! (For the Starling Wiki.) So it would be great if you could add it there instead!
Если вы согласны со мной, то прокомментируйте здесь.
Всего комментариев 20
Комментарии
16.01.2013 17:47
|
|
|
Как-то он непоследовательно тогда сделал. Зачем тогда вообще BlurFilter? Делал бы уже все фабричными методами в FragmentFilter, а то как-то ни туда ни сюда.
|
|
16.01.2013 18:34
|
|
|
Не вижу причин почему вы оба не можете быть правы.
По сути если блур фильтр покрывает сразу 3 варианта привычных флеш фильтров, то оставить его в виде одного класса не так уж и страшно, особенно что статичные фабричные методы в старлинге не новость. Если смотреть со стороны привычного флеш апи, то да, фильт - отдельный класс, отдельный инстанс. Я бы вообще пошел дальше и позволил на лету превращать блур в дропшадоу и т.д., не пересоздавая инстанс(это и сейчас можно, главное знать что менять), но я смотрю не со стороны апи, а просто меньше инстансов плодить и меньше GC скармливать. Да и вообще это проблема не старлинга, а восприятия. Я бы в жизни не заметил, что нету new DropShadow() |
|
16.01.2013 18:52
|
|
|
Цитата:
По сути если блур фильтр покрывает сразу 3 варианта привычных флеш фильтров, то оставить его в виде одного класса не так уж и страшно
GlowFilter ну никак не похож на BlurFilter и тем более им не является. Кроме того, никак нельзя поменять цвет и альфу у инстанса этих фильтров. Или задать их не в конструкторе. И то, что многое делается в старлинге криво, не отменяет того, что это криво. Надо переделывать "прямо". |
|
16.01.2013 18:59
|
|
|
|
Просто проект Старлинга вышел из стадии "домашнего" проекта. Его пиарят Адобовцы, включают в какие-то GameSDK, но даже не озаботились порекомендовать основному автору кодить по принятым конвенциям.
Это печально. Но думаю сообщество может и должно повлиять. |
|
16.01.2013 22:09
|
|
|
Цитата:
Пользователя АПИ не должны волновать детали реализации.
|
|
16.01.2013 22:17
|
|
|
Автор Старлинга — редиска. Адоб — редиска. Котяра, присоединяюсь к справедливому негодованию.
|
|
16.01.2013 23:20
|
|
|
В итоге автор-то задумался.
|
|
17.01.2013 01:39
|
|
|
|
Цитата:
дальше вы уж сами делайте что хотите, оборачивайте в классы билд методы, делайти билд методы для классов и т.д.
|
|
17.01.2013 02:33
|
|
|
Те, кто со мной знакомы, уже не впервые слышат эту истрию, так что сорри если повторю её еще раз. Делал я как то раз проект на старлинге, и решил часть въюшек обернуть в MXML. Обернул, сверстал пару въюх, посмотрел - вроде не плохо. Когда сверстал несколько въюх, решил что пора прикручивать логику. Написав следующий код:
Код AS3:[Bindable] public var model:GameModel; Цитата:
1) Modify the compiler (it's open source, afterall) to require Starling's dispatcher instead of the native one for binding code.
|
|
17.01.2013 12:01
|
|
|
|
Взглянул на Genom2D..
такая же упячка: Код AS3:public class GFlashObject extends GTexturedQuad private var __mNativeMatrix:Matrix; private var __sTextureId:String; protected var _bInvalidate:Boolean = false; p_x -= p_camera.rViewRectangle.x + p_camera.rViewRectangle.width/2; var mapHalfWidth:Number = __iTileWidth * __iWidth * .5; |
|
Обновил(-а) Котяра 17.01.2013 в 12:15
|
|
17.01.2013 13:31
|
|
|
Цитата:
Буду придираться к каждой букве и в меру сил требовать\просить исправлений.
|
|
17.01.2013 14:58
|
|
|
Цитата:
PrimaryFeather commented 3 hours ago
Thanks for the comments on this issue! I see your point -- I'll probably really change this in the next major version. Anybody who agrees OR disagrees, please comment here, too! Thanks in advance =) |
|
17.01.2013 16:23
|
|
|
Чем вам уже Genom2D не нравится? что двойные подчеркивания у приватов? или что не вы его написали?
|
|
17.01.2013 16:39
|
|
|
|
Указанием типа в именах переменных.
|
|
18.01.2013 01:49
|
|
|
|
двойным указанием)
Код AS3:private var __mNativeMatrix:Matrix; |
|
18.01.2013 01:53
|
|
|
|
Цитата:
Указанием типа в именах переменных.
аля Код AS3:_nativeMatrix:Matrix; Тем-более дубль. |
|
18.01.2013 02:31
|
|
|
|
Цитата:
Чем вам уже Genom2D не нравится?
атф не конвертит, в мувик нельзя задать "mov" а надо ["mov1", "mov2"] а вообще сырой и одним чуваком пишется |
|
Обновил(-а) Котяра 23.02.2013 в 14:18
|
|
24.01.2013 23:25
|
|
|
|
PS: Не зря написал этот пост. Наспамили Даниэлю хорошо. Всем спасибо)
Обещал в к следующей версии подумать и включить. |
|
Обновил(-а) Котяра 24.01.2013 в 23:42
|
|
19.11.2014 20:09
|
|
|
И как прогресс?
|
|
20.11.2014 20:03
|
|
|
Нету же прогресса
|
Последние записи от Котяра
- Страх и ненависть в Нью-Дели или сборка мультипака для arm7 и x86 c Adobe AIR 14 в FB (16.06.2014)
- Нативный EventDispatcher в старлинге (27.11.2013)
- Нужны ошибки компиляции при создании экземпляра синглетона извне? Запросто! (13.09.2013)
- ARP - новый формат упаковки ресурсов (07.02.2013)
- DropShadowFilter и GlowFilter в Starling (16.01.2013)
