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

2020-03-06 16:54:06 +08:00
 Umenezumi

大佬们好

我目前有个一个拥有大概 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());
    }


}
620 次点击
所在节点    问与答
2 条回复
Umenezumi
2020-03-06 18:21:42 +08:00
aguesuka
2020-03-07 09:51:07 +08:00
按我的理解,一个复杂的业务想要精简代码,只能通过合并类似逻辑来做。比如 @注解(参数一,参数二)的表达能力不会比 字段=函数(参数一,参数二)强。能用注解表达的逻辑肯定能用更少行的代码表达。注解的优点是字段和逻辑在一起,缺点是注解的参数是字符串,如果写错了只有在运行时才会发现。如果要给个建议的话,把 methodName 改成 enum,每个 enum 保存一个 BiFuntion。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/650468

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX