
大佬们好
我目前有个一个拥有大概 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 年 3 月 7 日 via Android 按我的理解,一个复杂的业务想要精简代码,只能通过合并类似逻辑来做。比如 @注解(参数一,参数二)的表达能力不会比 字段=函数(参数一,参数二)强。能用注解表达的逻辑肯定能用更少行的代码表达。注解的优点是字段和逻辑在一起,缺点是注解的参数是字符串,如果写错了只有在运行时才会发现。如果要给个建议的话,把 methodName 改成 enum,每个 enum 保存一个 BiFuntion。 |