帮助在Java

Question

我在与比我自己缺乏更有效地做到这一点的能力以外，没有理由对我的代码加倍麻烦的反射和注解......

for (Method curr: all){ 
     if (curr.isAnnotationPresent(anno)){ 
      if (anno == Pre.class){ 
       for (String str : curr.getAnnotation(Pre.class).value()){ 
        if (str.equals(method.getName()) && curr.getReturnType() == boolean.class && curr.getParameterTypes().length == 0){ 
         toRun.add(curr); 
        } 
       } 
      } if (anno == Post.class) { 
       for (String str : curr.getAnnotation(Post.class).value()){ 
        if (str.equals(method.getName()) && curr.getReturnType() == boolean.class && curr.getParameterTypes().length == 0){ 
         toRun.add(curr); 
        } 
       } 
      } 
     } 
    } 

阿鲁是一个参数 - Class<? extends Annotation>和Pre和Post是我的注释，都有一个value()这是一个字符串数组。

当然，这都是因为我让Eclipse自动填充代码，我还不明白。

Answer 1

如果'更有效'，你的意思是'更少的代码行'，那么为什么不合并两个if语句？我可以在它们之间看到的唯一区别是一个有Pre.class在其他有Post.class，你已经有一个类引用方便在阿鲁VAR：

for (Method curr : all) { 
    if (curr.isAnnotationPresent(anno)) { 
     if (anno == Pre.class || anno == Post.class){ 
      for (String str : curr.getAnnotation(anno).value()){ 
       if (str.equals(method.getName()) && curr.getReturnType() == boolean.class && curr.getParameterTypes().length == 0){ 
        toRun.add(curr); 
       } 
      } 
     } 
    } 
}

进一步说，有一个在做外的任何点如果anno既不是Pre.class也不是Post.class，所以你应该在外面做这个检查（这样可以节省迭代方法，如果它用一个你不想要的注解调用的话）。您也可以将检查的returnType和参数长度移到inner for循环之外（这样可以节省每个值的重复检查次数，并且如果它们不是真的，则可以保存迭代）。

if (anno == Pre.class || anno == Post.class){ 
    for (Method curr : all) { 
     if (curr.isAnnotationPresent(anno) && curr.getReturnType() == boolean.class && curr.getParameterTypes().length == 0) { 
      for (String str : curr.getAnnotation(anno).value()){ 
       if (str.equals(method.getName())) { 
        toRun.add(curr); 
       } 
      } 
     } 
    } 
}