remove unused and not very elegant code

master
Josha von Gizycki 8 years ago
parent 71a4e70ae2
commit 2dbc4a5806

@ -30,8 +30,4 @@ class Instantiator<T> {
return constructor.newInstance(parameters); return constructor.newInstance(parameters);
} }
T createSingleton() {
return null;
}
} }

@ -92,7 +92,7 @@ class IntegrityCheck {
} }
private void collectConstructors(ArrayList<Constructor> l, Constructor<?>[] arr) { private void collectConstructors(ArrayList<Constructor> l, Constructor<?>[] arr) {
if(arr.length == 0) { if (arr.length == 0) {
return; return;
} }
@ -101,7 +101,7 @@ class IntegrityCheck {
throw new MoreThanOneConstructor(clz); throw new MoreThanOneConstructor(clz);
} }
if(!Modifier.isPublic(arr[0].getModifiers())) { if (!Modifier.isPublic(arr[0].getModifiers())) {
throw new NoVisibleConstructor(clz); throw new NoVisibleConstructor(clz);
} }
@ -121,9 +121,7 @@ class IntegrityCheck {
} }
private void checkCycles() { private void checkCycles() {
definitions.forEach(def -> { definitions.forEach(def -> checkConstructorParameter(def.getClz(), null));
checkConstructorParameter(null, def.getClz());
});
} }
private void checkConstructorParameter(Class<?> rootClz, Class<?> paramClz) { private void checkConstructorParameter(Class<?> rootClz, Class<?> paramClz) {
@ -131,13 +129,16 @@ class IntegrityCheck {
throw new DependencyCycleDetected(rootClz); throw new DependencyCycleDetected(rootClz);
} }
Class<?>[] parameterTypes = findDefinition(paramClz).getConstructor().getParameterTypes(); ServiceDefinition def;
for (Class<?> innerParamClz : parameterTypes) { if (paramClz == null) {
if (rootClz == null) { def = findDefinition(rootClz);
checkConstructorParameter(paramClz, innerParamClz);
} else { } else {
checkConstructorParameter(rootClz, innerParamClz); def = findDefinition(paramClz);
} }
Class<?>[] parameterTypes = def.getConstructor().getParameterTypes();
for (Class<?> innerParamClz : parameterTypes) {
checkConstructorParameter(rootClz, innerParamClz);
} }
} }

Loading…
Cancel
Save