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