Access the Constructor of the Channel in the constructor of ReflectiveChannelFactory. (#8718)

Motivation:

We should access the Constructor of the passed in class in the Constructor of ReflectiveChannelFactory only to reduce the overhead but also fail-fast.

Modifications:

Access the Constructor early.

Result:

Fails fast and less performance overhead.
This commit is contained in:
Norman Maurer 2019-01-15 08:38:13 +01:00 committed by GitHub
parent 4ac5264f0e
commit c424599593
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -16,33 +16,40 @@
package io.netty.channel; package io.netty.channel;
import io.netty.util.internal.ObjectUtil;
import io.netty.util.internal.StringUtil; import io.netty.util.internal.StringUtil;
import java.lang.reflect.Constructor;
/** /**
* A {@link ChannelFactory} that instantiates a new {@link Channel} by invoking its default constructor reflectively. * A {@link ChannelFactory} that instantiates a new {@link Channel} by invoking its default constructor reflectively.
*/ */
public class ReflectiveChannelFactory<T extends Channel> implements ChannelFactory<T> { public class ReflectiveChannelFactory<T extends Channel> implements ChannelFactory<T> {
private final Class<? extends T> clazz; private final Constructor<? extends T> constructor;
public ReflectiveChannelFactory(Class<? extends T> clazz) { public ReflectiveChannelFactory(Class<? extends T> clazz) {
if (clazz == null) { ObjectUtil.checkNotNull(clazz, "clazz");
throw new NullPointerException("clazz"); try {
this.constructor = clazz.getConstructor();
} catch (NoSuchMethodException e) {
throw new IllegalArgumentException("Class " + StringUtil.simpleClassName(clazz) +
" does not have a public non-arg constructor", e);
} }
this.clazz = clazz;
} }
@Override @Override
public T newChannel() { public T newChannel() {
try { try {
return clazz.getConstructor().newInstance(); return constructor.newInstance();
} catch (Throwable t) { } catch (Throwable t) {
throw new ChannelException("Unable to create Channel from class " + clazz, t); throw new ChannelException("Unable to create Channel from class " + constructor.getDeclaringClass(), t);
} }
} }
@Override @Override
public String toString() { public String toString() {
return StringUtil.simpleClassName(clazz) + ".class"; return StringUtil.simpleClassName(ReflectiveChannelFactory.class) +
'(' + StringUtil.simpleClassName(constructor.getDeclaringClass()) + ".class)";
} }
} }