在工作上,我近对一个现有的Java项目代码进行了清理。完成之后,我发现了一些反复出现的不规范代码。所以,我把它们整理成了一个列表出来分享给我的同行希望能引起注意并改善代码的质量和可维护性。

  这个列表不区分顺序,全部来自一些代码质量检查工具,如 CheckStyle, FindBugs 和 PMD。

  在Eclipse中格式化源代码并管理import语句

  Eclipse提供了自动格式化源代码和管理import语句的功能(并移除未使用的语句)。你可以使用下面的快捷键来使用这些功能。

  Ctrl + Shift + F – 格式化源代码。

  Ctrl + Shift + O – 管理import语句并移除未使用的语句。

  除了手动执行这两个功能外,你还可以让Eclipse在保存文件的时候自动格式化源代码并自动管理import语句。要做到这个,在Eclipse中,到 Window -> Preferences -> Java -> Editor -> Save Actions并启用 Perform the selected actions on save,选中 Format source code和 organize imports。

  避免在方法中出现多个return语句(退出点):

  在你的方法中,确保只有一个 退出点。不要在一个方法中使用多于一个return语句。

  如,下面的代码是 不推荐的,因为它有多个退出点(return语句)。

    private boolean isEligible(int age){ 
      if(age > 18){ 
        return true; 
      }else{ 
        return false; 
      } 
    }

  上面的代码可以这么写(当然,下面的代码还可以改进,后面再说)。

    private boolean isEligible(int age){ 
      boolean result; 
      if(age > 18){ 
        result = true; 
      }else{ 
        result = false; 
      } 
      return result; 
    }

  简化if-else方法:

  我们写了一些只要一个参数的工具方法,检查一些条件并根据条件返回一个值。如,在上面见到的 isEligible方法。

    private boolean isEligible(int age){ 
      boolean result; 
      if(age > 18){ 
        result = true; 
      }else{ 
        result = false; 
      } 
      return result; 
    }

  可以只使用一个return语句来重写此方法。

    private boolean isEligible(int age){ 
      return age > 18; 
    }