Czy dobrą praktyką jest stosowanie odbicia, jeśli znacznie zmniejsza ilość kodu płyty grzewczej?
Zasadniczo istnieje kompromis między wydajnością i być może czytelnością z jednej strony a abstrakcją / automatyzacją / redukcją kodu płyty kotłowej z drugiej strony.
Edycja: Oto przykład zalecanego zastosowania odbicia .
Dla przykładu załóżmy, że istnieje klasa abstrakcyjna, Base
która ma 10 pól i 3 podklasy SubclassA
, SubclassB
a SubclassC
każda z 10 różnymi polami; wszystkie są prostymi fasolami. Problem polega na tym, że otrzymujesz dwa Base
odwołania do typów i chcesz sprawdzić, czy odpowiadające im obiekty są tego samego (pod) typu i są równe.
Jako rozwiązania istnieje surowe rozwiązanie, w którym najpierw sprawdza się, czy typy są równe, a następnie sprawdza się wszystkie pola lub można użyć odbicia i dynamicznie sprawdzać, czy są one tego samego typu i iterować wszystkie metody zaczynające się od „get” (konwencja ponad konfigurację), wywołaj je na obu obiektach i wywołaj równe na wynikach.
boolean compare(Base base1, Base, base2) {
if (base1 instanceof SubclassA && base2 instanceof SubclassA) {
SubclassA subclassA1 = (SubclassA) base1;
SubclassA subclassA2 = (SubclassA) base2;
compare(subclassA1, subclassA2);
} else if (base1 instanceof SubclassB && base2 instanceof SubclassB) {
//the same
}
//boilerplate
}
boolean compare(SubclassA subA1, SubclassA subA2) {
if (!subA1.getField1().equals(subA2.getField1)) {
return false;
}
if (!subA1.getField2().equals(subA2.getField2)) {
return false;
}
//boilerplate
}
boolean compare(SubclassB subB1, SubclassB subB2) {
//boilerplate
}
//boilerplate
//alternative with reflection
boolean compare(Base base1, Base base2) {
if (!base1.getClass().isAssignableFrom(base2.getClass())) {
System.out.println("not same");
System.exit(1);
}
Method[] methods = base1.getClass().getMethods();
boolean isOk = true;
for (Method method : methods) {
final String methodName = method.getName();
if (methodName.startsWith("get")) {
Object object1 = method.invoke(base1);
Object object2 = method.invoke(base2);
if(object1 == null || object2 == null) {
continue;
}
if (!object1.equals(object2)) {
System.out.println("not equals because " + object1 + " not equal with " + object2);
isOk = false;
}
}
}
if (isOk) {
System.out.println("is OK");
}
}