规范代码

1、MyBatis 不要为了多个查询条件而写 1 = 1

当遇到多个查询条件,使用 where 1 = 1 可以很方便的解决我们的问题,但是这样很可能会造成非常大的性能损失,因为添加了 "where 1 = 1" 的过滤条件之后,数据库系统就无法使用索引等查询优化策略,数据库系统将会被迫对每行数据进行扫描(即全表扫描)以比较此行是否满足过滤条件,当表中的数据量较大时查询速度会非常慢;此外,还会存在 SQL 注入的风险。

反例:

<select id="queryBookInfo" parameterType="com.caroly.test.BookInfo" resultType="java.lang.Integer">
    select count(*) from t_rule_BookInfo t where 1=1
    <if test="title !=null and title !='' ">
        AND title = #{title}
    </if>
    <if test="author !=null and author !='' ">
        AND author = #{author}
    </if>
</select>

反例:

<select id="queryBookInfo" parameterType="com.caroly.test.BookInfo" resultType="java.lang.Integer">
    select count(*) from t_rule_BookInfo t
    <where>
        <if test="title !=null and title !='' ">
            title = #{title}
        </if>
        <if test="author !=null and author !='' ">
            AND author = #{author}
        </if>
    </where>
</select>

UPDATE 操作也一样,可以用标记代替 1=1。


2、迭代 entrySet() 获取 Map 的 key 和 value

当循环中只需要获取 Map 的主键 key 时,迭代 keySet() 是正确的;但是,当需要主键 key 和取值 value 时,迭代 entrySet() 才是更高效的做法,其比先迭代 keySet() 后再去通过 get 取值性能更佳。

反例:

Map<String, String> map= new HashMap<>();
for (String key: map.keySet()){
    System.out.println(key+ " "+ map.get(key));
}

正例:

Map<String, String> map= new HashMap<>();
for (Map.Entry<String, String> entry: map.entrySet()){
    System.out.println(entry.getKey()+ " "+ entry.getValue());
}

3、使用 Collection.isEmpty() 检测空

使用 Collection.size() 来检测是否为空在逻辑上没有问题,但是使用 Collection.isEmpty() 使得代码更易读,并且可以获得更好的性能;除此之外,任何 Collection.isEmpty() 实现的时间复杂度都是 O(1) ,不需要多次循环遍历,但是某些通过 Collection.size() 方法实现的时间复杂度可能是O(n)。

反例:

LinkedList<Object> collection= new LinkedList<>();
if(collection.size()== 0){
    System.out.println("collection is empty.");
}

正例:

LinkedList<Object> collection = new LinkedList<>();
if (collection.isEmpty()){
    System.out.println("collection is empty.");
}
// 检测是否为 null,可以使用 CollectionUtils.isEmpty
if(CollectionUtils.isEmpty(collection)){
    System.out.println("collection is null.");
}

一般情况下 null 和 isEmpty 会联合使用:

// 这里先判断有没有这个集合再判断这个集合里有没有对象。顺序不能反;
if(collection != null && !collection.isEmpty()){
    ...
}

4、初始化集合时尽量指定其大小

尽量在初始化时指定集合的大小,能有效减少集合的扩容次数,因为集合每次扩容的时间复杂度很可能是 O(n),耗费时间和性能。

反例:

// 初始化 list,往 list 中添加元素:
int[] arr = new int[]{1,2,3,4};
List<Integer> list = new ArrayList<>();
for (int i : arr){
    list.add(i);
}

正例:

// 初始化 list,往 list 中添加元素:
int[] arr = new int[]{1,2,3,4};
// 指定集合 list 的容量大小
List<Integer> list = new ArrayList<>(arr.length);
for (int i : arr){
    list.add(i);
}

5、使用 StringBuilder 拼接字符串

一般的字符串拼接在编译期 Java 会对其进行优化,但是在循环中字符串的拼接 Java 编译期无法执行优化,所以需要使用 StringBuilder 进行替换。

反例:

String str = "";
for (int i = 0; i < 10; i++){
    // 在循环中字符串拼接 Java 不会对其进行优化
    str += i;
}

正例:

String str1 = "Love";
String str2 = "Courage";
String strConcat = str1 + str2;  // Java 编译器会对该普通模式的字符串拼接进行优化
StringBuilder sb = new StringBuilder();
for (int i = 0; i < 10; i++){
    // 在循环中,Java 编译器无法进行优化,所以要手动使用 StringBuilder
    sb.append(i);
}

6、若需频繁调用 Collection.contains 方法则使用 Set

在 Java 集合类库中,List的 contains 方法普遍时间复杂度为 O(n),若代码中需要频繁调用 contains 方法查找数据则先将集合 list 转换成 HashSet 实现,将 O(n) 的时间复杂度变为 O(1)。

反例:

List<Object> list = new ArrayList<>();
for (int i = 0; i <= Integer.MAX_VALUE; i++){
    // 时间复杂度为 O(n)
    if (list.contains(i))
        System.out.println("list contains "+ i);
}

正例:

List<Object> list = new ArrayList<>();
Set<Object> set = new HashSet<>(list);
for (int i = 0; i <= Integer.MAX_VALUE; i++){
    // 时间复杂度为 O(1)
    if (set.contains(i)){
        System.out.println("list contains "+ i);
    }
}

7、使用静态代码块实现赋值静态成员变量

对于集合类型的静态成员变量,应该使用静态代码块赋值,而不是使用集合实现来赋值。

反例:

private static Map<String, Integer> map = new HashMap<String, Integer>(){
    {
        map.put("Leo",1);
        map.put("Family-loving",2);
        map.put("Cold on the out side passionate on the inside",3);
    }
};
private static List<String> list = new ArrayList<String>(){
    {
        list.add("Sagittarius");
        list.add("Charming");
        list.add("Perfectionist");
    }
};

正例:

private static Map<String, Integer> map = new HashMap<String, Integer>();
static {
    map.put("First",1);
    map.put("Second",2);
    map.put("Third",3);
}

private static List<String> list = new ArrayList<>();
static {
    list.add("First");
    list.add("Second");
    list.add("Third");
}

8、工具类中屏蔽构造函数

工具类是一堆静态字段和函数的集合,其不应该被实例化;但是,Java 为每个没有明确定义构造函数的类添加了一个隐式公有构造函数,为了避免不必要的实例化,应该显式定义私有构造函数来屏蔽这个隐式公有构造函数。

反例: