大佬们好
我目前有个一个拥有大概 200 个属性的结果类(暂时不改动类结构)
类中大概五十余个“基础指标”(不需要计算)
大概一百三十个“二次计算指标”(均为某两个基础指标计算而成)
大概十来个“三次计算指标”(为某两个二次计算指标计算而成)
上一任类中写了一百多个 if 用于计算指标 如:
if(Objects.nonNull(指标 A) && Objects.nonNull(指标 B)){
二次计算指标 = (计算工具类.计算(指标 A,指标 B)
}
后来指标越积累越多,导致这个类现在 2000 多行(其中 1000 多行 if 判断),既丑陋又恶心
我尝试使用注解以及反射完成这些指标的计算(我基础很差,比较菜),搞出了下面这个 Demo,
虽然解决了我的问题,但是还是感觉不是很“优雅”,希望大家提点意见或者设计思路,十分感谢 注解类:
@Retention(RetentionPolicy.RUNTIME)
@Target(ElementType.FIELD)
public @interface Calculate {
String a();
String b();
String methodName();
}
测试类
@Data
public class TestAnnotiation {
private int a = 2;
private int b = 1;
//二次计算指标
@Calculate(a = "a", b = "b", methodName = "calc")
private BigDecimal c;
//三次计算指标
@Calculate(a = "a", b = "c", methodName = "calc")
private BigDecimal d;
public static void main(String[] args) {
TestAnnotiation test = new TestAnnotiation();
Class<? extends TestAnnotiation> clazz = test.getClass();
List<Field> fields = Arrays.asList(clazz.getDeclaredFields());
fields.stream().forEach(e -> {
if (e.isAnnotationPresent(Calculate.class)) {
Calculate annotation = e.getAnnotation(Calculate.class);
e.setAccessible(true);
try {
Method getMethodA = clazz.getMethod("get" + annotation.a().toUpperCase());
Method getMethodB = clazz.getMethod("get" + annotation.b().toUpperCase());
Method caculateMethod = CalculationUtils.class.getMethod(annotation.methodName(), Number.class, Number.class);
e.set(test, caculateMethod.invoke(null, getMethodA.invoke(test), getMethodB.invoke(test)));
} catch (NoSuchMethodException | IllegalAccessException | InvocationTargetException ex) {
ex.printStackTrace();
}
}
});
System.out.println("test.getC() = " + test.getC());
System.out.println("test.getD() = " + test.getD());
}
}
1
Umenezumi OP |
2
aguesuka 2020-03-07 09:51:07 +08:00 via Android
按我的理解,一个复杂的业务想要精简代码,只能通过合并类似逻辑来做。比如 @注解(参数一,参数二)的表达能力不会比 字段=函数(参数一,参数二)强。能用注解表达的逻辑肯定能用更少行的代码表达。注解的优点是字段和逻辑在一起,缺点是注解的参数是字符串,如果写错了只有在运行时才会发现。如果要给个建议的话,把 methodName 改成 enum,每个 enum 保存一个 BiFuntion。
|