大佬们好
我目前有个一个拥有大概 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());
}
}
这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。
V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。
V2EX is a community of developers, designers and creative people.