[BEEHIVE-893] Change dispatchEvent method signature to use varargs

• Type: Improvement
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Component/s: Controls
• Environment: N/A
• Assignee: Unassigned
• Reporter: Xibin Zeng
• Created: 23/Aug/05 01:18
• Updated: 26/Aug/05 02:23

Description

I suggest we use varargs in the org.apache.beehive.controls.api.events.EventDispatcher interface. This would make client code that use the method easier to read and easier to contrstuct. The fowllowing is the detail of this suggestion:

Before:

public Object dispatchEvent(ControlHandle target, EventRef event, Object [] args)
throws IllegalAccessException, IllegalArgumentException,
InvocationTargetException;

After

public Object dispatchEvent(ControlHandle target, EventRef event, Object... args)
throws IllegalAccessException, IllegalArgumentException,
InvocationTargetException;

For example, to dispatch a no-arg event, the client code that calls this method would be

dispatcher.dispatchEvent(target, event);

Rather than

dispatcher.dispatchEvent(target, event, new Object[] {});

Similarily, it looks nicer when you have arguments.

Thanks