Author: oheger
Date: Wed Mar 16 21:19:12 2011
New Revision: 1082303
URL: http://svn.apache.org/viewvc?rev=1082303&view=rev
Log:
Checkstyle
Modified:
commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/event/EventUtils.java
Modified:
commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/event/EventUtils.java
URL:
http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/event/EventUtils.java?rev=1082303&r1=1082302&r2=1082303&view=diff
==============================================================================
---
commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/event/EventUtils.java
(original)
+++
commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/event/EventUtils.java
Wed Mar 16 21:19:12 2011
@@ -49,9 +49,13 @@ public class EventUtils {
try {
MethodUtils.invokeMethod(eventSource, "add" +
listenerType.getSimpleName(), listener);
} catch (NoSuchMethodException e) {
- throw new IllegalArgumentException("Class " +
eventSource.getClass().getName() + " does not have a public add" +
listenerType.getSimpleName() + " method which takes a parameter of type " +
listenerType.getName() + ".");
+ throw new IllegalArgumentException("Class " +
eventSource.getClass().getName()
+ + " does not have a public add" +
listenerType.getSimpleName()
+ + " method which takes a parameter of type " +
listenerType.getName() + ".");
} catch (IllegalAccessException e) {
- throw new IllegalArgumentException("Class " +
eventSource.getClass().getName() + " does not have an accessible add" +
listenerType.getSimpleName () + " method which takes a parameter of type " +
listenerType.getName() + ".");
+ throw new IllegalArgumentException("Class " +
eventSource.getClass().getName()
+ + " does not have an accessible add" +
listenerType.getSimpleName ()
+ + " method which takes a parameter of type " +
listenerType.getName() + ".");
} catch (InvocationTargetException e) {
throw new RuntimeException("Unable to add listener.",
e.getCause());
}
@@ -68,8 +72,10 @@ public class EventUtils {
* @param eventTypes the event types (method names) from the listener
interface (if none specified, all will be
* supported)
*/
- public static <L> void bindEventsToMethod(Object target, String
methodName, Object eventSource, Class<L> listenerType, String... eventTypes) {
- final L listener =
listenerType.cast(Proxy.newProxyInstance(target.getClass().getClassLoader(),
new Class[] { listenerType }, new EventBindingInvocationHandler(target,
methodName, eventTypes)));
+ public static <L> void bindEventsToMethod(Object target, String
methodName, Object eventSource,
+ Class<L> listenerType, String... eventTypes) {
+ final L listener =
listenerType.cast(Proxy.newProxyInstance(target.getClass().getClassLoader(),
+ new Class[] { listenerType }, new
EventBindingInvocationHandler(target, methodName, eventTypes)));
addEventListener(eventSource, listenerType, listener);
}
@@ -78,12 +84,28 @@ public class EventUtils {
private final String methodName;
private final Set<String> eventTypes;
+ /**
+ * Creates a new instance of {@code EventBindingInvocationHandler}.
+ *
+ * @param target the target object for method invocations
+ * @param methodName the name of the method to be invoked
+ * @param eventTypes the names of the supported event types
+ */
public EventBindingInvocationHandler(final Object target, final String
methodName, String[] eventTypes) {
this.target = target;
this.methodName = methodName;
this.eventTypes = new HashSet<String>(Arrays.asList(eventTypes));
}
+ /**
+ * Handles a method invocation on the proxy object.
+ *
+ * @param proxy the proxy instance
+ * @param method the method to be invoked
+ * @param parameters the parameters for the method invocation
+ * @return the result of the method call
+ * @throws Throwable if an error occurs
+ */
public Object invoke(final Object proxy, final Method method, final
Object[] parameters) throws Throwable {
if ( eventTypes.isEmpty() ||
eventTypes.contains(method.getName())) {
if (hasMatchingParametersMethod(method)) {
@@ -95,6 +117,12 @@ public class EventUtils {
return null;
}
+ /**
+ * Checks whether a method for the passed in parameters can be found.
+ *
+ * @param method the method to be invoked
+ * @return a flag whether the parameters could be matched
+ */
private boolean hasMatchingParametersMethod(final Method method) {
return MethodUtils.getAccessibleMethod(target.getClass(),
methodName, method.getParameterTypes()) != null;
}