V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
Umenezumi
V2EX  ›  问与答

关于注解设计以及反射的问题

  •  
  •   Umenezumi · 2020-03-06 16:54:06 +08:00 · 618 次点击
    这是一个创建于 1563 天前的主题,其中的信息可能已经有所发展或是发生改变。

    大佬们好

    我目前有个一个拥有大概 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());
        }
    
    
    }
    
    2 条回复    2020-03-07 09:51:07 +08:00
    Umenezumi
        1
    Umenezumi  
    OP
       2020-03-06 18:21:42 +08:00
    aguesuka
        2
    aguesuka  
       2020-03-07 09:51:07 +08:00 via Android
    按我的理解,一个复杂的业务想要精简代码,只能通过合并类似逻辑来做。比如 @注解(参数一,参数二)的表达能力不会比 字段=函数(参数一,参数二)强。能用注解表达的逻辑肯定能用更少行的代码表达。注解的优点是字段和逻辑在一起,缺点是注解的参数是字符串,如果写错了只有在运行时才会发现。如果要给个建议的话,把 methodName 改成 enum,每个 enum 保存一个 BiFuntion。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   928 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 25ms · UTC 19:13 · PVG 03:13 · LAX 12:13 · JFK 15:13
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.