java代码审查

1.工具类或者常量类里面的方法都是静态的，建议直接用类名调用，不用创建对象，所以将构造方法私有化，禁止创建对象

1 2 3 4 5 6 7 8 9 10 11 12 13

//error example: public class StringUtils{ /** * 判断一个字符串是否为英文中文或数字 * * @param str * @return */ public static boolean isLetterDigitOrChinese( String str ) { String regex = "^[a-z0-9A-Zu4e00-u9fa5]+$"; return str.matches( regex ); } }

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

//right example: public class StringUtils{ //添加构造私有构造方法 private StringUtils{ } /** * 判断一个字符串是否为英文中文或数字 * * @param str * @return */ public static boolean isLetterDigitOrChinese( String str ) { String regex = "^[a-z0-9A-Zu4e00-u9fa5]+$"; return str.matches( regex ); } }

2.NPE(空指针异常)-equals

1 2 3 4 5 6 7 8 9 10 11 12 13 14

//error String testString="abc"; if(testString.equals("abc")){ 大专栏  java代码审查v class="line"> }else{ } //right String testString="abc"; if("abc".eqauls(testString)){ //code }else{ //code }

3.map list 新建 (java 1.7 )

1 2 3 4 5 6 7

// Noncompliant List<String> strings = new ArrayList<String>(); Map<String,List<Integer>> map = new HashMap<String,List<Integer>>(); //Compliant Solution List<String> strings = new ArrayList<>(); Map<String,List<Integer>> map = new HashMap<>();

4.使用isEmpty 来判断集合是否为空

1 2 3 4 5 6 7 8 9

//Noncompliant if(data.size==0){ } //compliant if(data.isEmpty){ }

5.数组定义 (“[]” 在变量名字前面)

1 2 3 4 5

//Noncompliant int arr[]={}; //compliant int[] arr={};

6.字符串转换为数字

1 2 3 4 5 6

//Noncompliant String numString="12222222"; long numLong=Long.valueOf(numString); //compliant String numString="12222222"; long posNum=Long.parseLong( posMap.get( "num" ).toString());

7.公共静态成员应该加上final，也就是public static final 一般不分家