[#IDEADEV-31611] Warning about unused parameter in event handler

IDEA: Development

Warning about unused parameter in event handler

Created: 16 Sep 08 16:50 Updated: 10 Nov 08 21:40

Component/s:

Flex Support

Fix Version/s:

Diana 9144

Original Estimate:

Unknown

Remaining Estimate:

Unknown

Time Spent:

Unknown

File Attachments:

None

Image Attachments:

1. 30.10.png
(9 kb)

2. bug.png
(4 kb)

Issue Links:

Duplicate

This issue is duplicated by:
	~~IDEA-20281~~ "unused javascript symbol" on require...
	~~IDEA-20512~~ no way in flex to disable unused para...

Build:	8,792
Fixed in build:	9,126

Description

« Hide

see example image

All

Comments

Work Log

Change History

Sort Order:

[ Permlink | « Hide ]

Maxim Mossienko - 16 Sep 08 16:57

Warning about return seems to be valid

[ Permlink | « Hide ]

Yuriy Yarovoy - 16 Sep 08 17:50

Да, точно. Круто.

[ Permlink | « Hide ]

Maxim Mossienko - 27 Sep 08 15:10

could you please show example usages of such event handler?

[ Permlink | « Hide ]

Yuriy Yarovoy - 27 Sep 08 18:58

Обработчик события - это стандартная функция с одним обязательным параметром в виде экземпляра события (наследника класса flash.events.Event). Параметр обязательный из-за того, что объект рассылающий событие всем подписанным слушателям (наследник класса flash.event.EventDispatcher) передает в качестве этого параметра объект события. Подписка какого либо метода на получение события производится следующим образом:

var mySprite:Sprite = new Sprite();
// подписываем функцию-обработчик mouseDownHandler на получение события нажатия кнопки мыши по спрайту.
mySprite.addEventListener(MouseEvent.MOUSE_DOWN, mouseDownHandler);

Кстати, и квикфикс строки:

mySprite.addEventListener(MouseEvent.MOUSE_DOWN, mouseDownHandler);

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

Готов ответить и на другие вопросы, если они есть.

[ Permlink | « Hide ]

Vladimir Krivosheev - 30 Oct 08 20:12

Также warning не должен быть для callback - их сигнатура четко определена и менять ее мы тоже не можем как и в случае с eventHandler. См. Array.filter и т. п.