Help me make this method more solid:
/**
* Check if the method is declared in the interface.
* Assumes the method was obtained from a concrete class that
* implements the interface, and return true if the method overrides
* a method from the interface.
*/
public static boolean isDeclaredInInterface(Method method, Class<?> interfaceClass) {
for (Method methodInInterface : interfaceClass.getMethods())
{
if (methodInInterface.getName().equals(method.getName()))
return true;
}
return false;
}
This is a good start:
Replace:
for (Method methodInInterface : interfaceClass.getMethods())
{
if (methodInInterface.getName().equals(method.getName()))
return true;
}
with:
for (Method methodInInterface : interfaceClass.getMethods()) {
if (methodInInterface.getName().equals(method.getName())) {
return true;
}
}
:)
How about this:
try {
interfaceClass.getMethod(method.getName(), method.getParameterTypes());
return true;
} catch (NoSuchMethodException e) {
return false;
}
If you want to avoid catching NoSuchMethodException
from Yashai's answer:
for (Method ifaceMethod : iface.getMethods()) {
if (ifaceMethod.getName().equals(candidate.getName()) &&
Arrays.equals(ifaceMethod.getParameterTypes(), candidate.getParameterTypes())) {
return true;
}
}
return false;
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With