Чи є хорошою практикою використання рефлексії, якщо сильно зменшується кількість кодового коду?
В основному існує компроміс між продуктивністю та, можливо, читабельністю з одного боку та абстракцією / автоматизацією / зменшенням кодового коду з іншого боку.
Редагувати: Ось приклад рекомендованого використання рефлексії .
Для прикладу, припустимо, є абстрактний клас, Baseякий має 10 полів і має 3 підкласи SubclassA, SubclassBі SubclassCкожне з 10 різними полями; всі вони прості боби. Проблема полягає в тому, що ви отримуєте два Baseпосилання типу і хочете дізнатися, чи відповідні їм об'єкти одного (під) типу і рівні.
В якості рішень є неочищене рішення, в якому спочатку ви перевіряєте, чи типи рівні, а потім перевіряєте всі поля, або ви можете використовувати відображення і динамічно бачити, чи є вони одного типу, і повторювати всі методи, які починаються з "отримати" (конвенція над конфігурацією), викликайте їх на обох об'єктах і виклик дорівнює за результатами.
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");
}
}