那些年,我們踩過(guò)的 Java 坑

Photo?@?Drew Farwell?

文? |??常意

1.前言

中國(guó)有句老話叫"事不過(guò)三"，指一個(gè)人犯了同樣的錯(cuò)誤，一次兩次三次還可以原諒，超過(guò)三次就不可原諒了。有人指出這個(gè)“三”是虛數(shù)，用來(lái)泛指多次，所以"事不過(guò)三"不包括“三”。至于"事不過(guò)三"包不包括“三”，可能跟每個(gè)人的底線有關(guān)系，屬于哲學(xué)范疇，不在本文的討論范圍之內(nèi)。

1.對(duì)象比較方法

1.1.問(wèn)題現(xiàn)象

Short shortValue = (short)12345;
System.out.println(shortValue == 12345); // true
System.out.println(12345 == shortValue); // true
Integer intValue = 12345;
System.out.println(intValue == 12345); // true
System.out.println(12345 == intValue); // true
Long longValue = 12345L;
System.out.println(longValue == 12345); // true
System.out.println(12345 == longValue); // true

Short shortValue = (short)12345;
System.out.println(Objects.equals(shortValue, 12345)); // false
System.out.println(Objects.equals(12345, shortValue)); // false
Integer intValue = 12345;
System.out.println(Objects.equals(intValue, 12345)); // true
System.out.println(Objects.equals(12345, intValue)); // true
Long longValue = 12345L;
System.out.println(Objects.equals(longValue, 12345)); // false
System.out.println(Objects.equals(12345, longValue)); // false

1.2.問(wèn)題分析

7 ? getstatic java.lang.System.out : java.io.PrintStream [22]
10 ?aload_1 [shortValue]
11 ?invokevirtual java.lang.Short.shortValue() : short [28]
14 ?sipush 12345
17 ?if_icmpne 24
20 ?iconst_1
21 ?goto 25
24 ?iconst_0
25 ?invokevirtual java.io.PrintStream.println(boolean) : void [32]

7 ? getstatic java.lang.System.out : java.io.PrintStream [22]
10 ?aload_1 [shortValue]
11 ?sipush 12345
14 ?invokestatic java.lang.Integer.valueOf(int) : java.lang.Integer [28]
17 ?invokestatic java.util.Objects.equals(java.lang.Object, java.lang.Object) : boolean [33]
20 ?invokevirtual java.io.PrintStream.println(boolean) : void [39]

在 Java 語(yǔ)言中，整數(shù)的默認(rèn)數(shù)據(jù)類型是 int ，小數(shù)的默認(rèn)數(shù)據(jù)類型是 double 。

public static boolean equals(Object a, Object b) {
 ? ?return (a == b) || (a != null && a.equals(b));
}

public boolean equals(Object obj) {
 ? ?if (obj instanceof Short) {
 ? ? ? ?return value == ((Short)obj).shortValue();
 ?  }
 ? ?return false;
}

1.3.避坑方法

Short shortValue = (short)12345;
System.out.println(shortValue == (short)12345); // true
System.out.println((short)12345 == shortValue); // true
Integer intValue = 12345;
System.out.println(intValue == 12345); // true
System.out.println(12345 == intValue); // true
Long longValue = 12345L;
System.out.println(longValue == 12345L); // true
System.out.println(12345L == longValue); // true

第二段代碼可以這樣寫(xiě)：

Short shortValue = (short)12345;
System.out.println(Objects.equals(shortValue, (short)12345)); // true
System.out.println(Objects.equals((short)12345, shortValue)); // true
Integer intValue = 12345;
System.out.println(Objects.equals(intValue, 12345)); // true
System.out.println(Objects.equals(12345, intValue)); // true
Long longValue = 12345L;
System.out.println(Objects.equals(longValue, 12345L)); // true
System.out.println(Objects.equals(12345L, longValue)); // true

Unlikely argument type for equals(): int seems to be unrelated to Short
Unlikely argument type for equals(): Short seems to be unrelated to int
Unlikely argument type for equals(): int seems to be unrelated to Long
Unlikely argument type for equals(): Long seems to be unrelated to int

Call to Short.equals(Integer) in xxx.Xxx.main(String[]) [Scariest(1), High confidence]
Call to Integer.equals(Short) in xxx.Xxx.main(String[]) [Scariest(1), High confidence]
Call to Long.equals(Integer) in xxx.Xxx.main(String[]) [Scariest(1), High confidence]
Call to Integer.equals(Long) in xxx.Xxx.main(String[]) [Scariest(1), High confidence]

2.三元表達(dá)式拆包

三元表達(dá)式是 Java 編碼中的一個(gè)固定語(yǔ)法格式：“條件表達(dá)式？表達(dá)式 1 ：表達(dá)式 2 ”。三元表達(dá)式的邏輯為：“如果條件表達(dá)式成立，則執(zhí)行表達(dá)式 1 ，否則執(zhí)行表達(dá)式 2 ”。

2.1.問(wèn)題現(xiàn)象

boolean condition = false;
Double value1 = 1.0D;
Double value2 = 2.0D;
Double value3 = null;
Double result = condition ? value1 * value2 : value3; // 拋出空指針異常

2.2.問(wèn)題分析

17 ?iload_1 [condition]
18 ?ifeq 33
21 ?aload_2 [value1]
22 ?invokevirtual java.lang.Double.doubleValue() : double [24]
25 ?aload_3 [value2]
26 ?invokevirtual java.lang.Double.doubleValue() : double [24]
29 ?dmul
30 ?goto 38
33 ?aload 4 [value3]
35 ?invokevirtual java.lang.Double.doubleValue() : double [24]
38 ?invokestatic java.lang.Double.valueOf(double) : java.lang.Double [16]
41 ?astore 5 [result]
43 ?getstatic java.lang.System.out : java.io.PrintStream [28]
46 ?aload 5 [result]

1. 若兩個(gè)表達(dá)式類型相同，返回值類型為該類型；
2. 若兩個(gè)表達(dá)式類型不同，但類型不可轉(zhuǎn)換，返回值類型為Object類型；
3. 若兩個(gè)表達(dá)式類型不同，但類型可以轉(zhuǎn)化，先把包裝數(shù)據(jù)類型轉(zhuǎn)化為基本數(shù)據(jù)類型，然后按照基本數(shù)據(jù)類型的轉(zhuǎn)換規(guī)則（byte

boolean condition = false;
Double value1 = 1.0D;
Double value2 = 2.0D;
Double value3 = null;
Integer value4 = null;
// 返回類型為Double,不拋出空指針異常
Double result1 = condition ? value1 : value3;
// 返回類型為double,會(huì)拋出空指針異常
Double result2 = condition ? value1 : value4;
// 返回類型為double,不拋出空指針異常
Double result3 = !condition ? value1 * value2 : value3;
// 返回類型為double,會(huì)拋出空指針異常
Double result4 = condition ? value1 * value2 : value3;

2.3.避坑方法

boolean condition = false;
Double value1 = 1.0D;
Double value2 = 2.0D;
Double value3 = null;
Double result;
if (condition) {
 ? ?result = value1 * value2;
} else {
 ? ?result = value3;
}

boolean condition = false;
double value1 = 1.0D;
double value2 = 2.0D;
double value3 = 3.0D;
double result = condition ? value1 * value2 : value3;

3.泛型對(duì)象賦值

3.1.問(wèn)題現(xiàn)象

/** 分頁(yè)數(shù)據(jù)VO類 */
@Getter
@Setter
@ToString
@NoArgsConstructor
@AllArgsConstructor
public class PageDataVO {
 ? ?/** 總共數(shù)量 */
 ? ?private Long totalCount;
 ? ?/** 數(shù)據(jù)列表 */
 ? ?private List dataList;
}

/** 用戶DAO接口 */
@Mapper
public interface UserDAO {
 ? ?/** 統(tǒng)計(jì)用戶數(shù)量 */
 ? ?public Long countUser(@Param("query") UserQueryVO query);
 ? ?/** 查詢用戶信息 */
 ? ?public List queryUser(@Param("query") UserQueryVO query);
}

/** 用戶服務(wù)類 */
@Service
public class UserService {
 ? ?/** 用戶DAO */
 ? ?@Autowired
 ? ?private UserDAO userDAO;

 ? ?/** 查詢用戶信息 */
 ? ?public PageDataVO queryUser(UserQueryVO query) {
 ? ? ? ?List dataList = null;
 ? ? ? ?Long totalCount = userDAO.countUser(query);
 ? ? ? ?if (Objects.nonNull(totalCount) && totalCount.compareTo(0L) > 0) {
 ? ? ? ? ? ?dataList = userDAO.queryUser(query);
 ? ? ?  }
 ? ? ? ?return new PageDataVO(totalCount, dataList);
 ?  }
}

/** 用戶控制器類 */
@Controller
@RequestMapping("/user")
public class UserController {
 ? ?/** 用戶服務(wù) */
 ? ?@Autowired
 ? ?private UserService userService;

 ? ?/** 查詢用戶 */
 ? ?@ResponseBody
 ? ?@RequestMapping(value = "/query", method = RequestMethod.POST)
 ? ?public Result> queryUser(@RequestBody UserQueryVO query) {
 ? ? ? ?PageDataVO pageData = userService.queryUser(query);
 ? ? ? ?return ResultBuilder.success(pageData);
 ?  }
}

3.2.問(wèn)題分析

ArrayList list = new ArrayList();

ArrayList list = new ArrayList();

// 第一種情況
ArrayList list1 = new ArrayList();
// 第二種情況
ArrayList list2 = new ArrayList();

3.3.避坑方法

【推薦】集合泛型定義時(shí)，在 JDK7 及以上，使用 diamond 語(yǔ)法或全省略。說(shuō)明：菱形泛型，即 diamond，直接使用<>來(lái)指代前邊已經(jīng)指定的類型。正例：

// <> diamond 方式
HashMap userCache = new HashMap<>(16);
// 全省略方式
ArrayList users = new ArrayList(10);

return new PageDataVO<>(totalCount, dataList);

現(xiàn)在，在 Eclipse 的問(wèn)題窗口中，我們會(huì)看到這樣的錯(cuò)誤：

Cannot infer type arguments for PageDataVO<>

4.泛型屬性拷貝

Spring 的 BeanUtils.copyProperties 方法，是一個(gè)很好用的屬性拷貝工具方法。

4.1.問(wèn)題現(xiàn)象

/** 基礎(chǔ)DO類 */
@Getter
@Setter
@ToString
public class BaseDO {
 ? ?private T id;
 ? ?private Date gmtCreate;
 ? ?private Date gmtModified;
}

/** 用戶DO類 */
@Getter
@Setter
@ToString
public class UserDO extends BaseDO{
 ? ?private String name;
 ? ?private String description;
}

/** 用戶VO類 */
@Getter
@Setter
@ToString
public static class UserVO {
 ? ?private Long id;
 ? ?private String name;
 ? ?private String description;
}

/** 用戶服務(wù)類 */
@Service
public class UserService {
 ? ?/** 用戶DAO */
 ? ?@Autowired
 ? ?private UserDAO userDAO;

 ? ?/** 查詢用戶 */
 ? ?public List queryUser(UserQueryVO query) {
 ? ? ? ?// 查詢用戶信息
 ? ? ? ?List userDOList = userDAO.queryUser(query);
 ? ? ? ?if (CollectionUtils.isEmpty()) {
 ? ? ? ? ? ?return Collections.emptyList();
 ? ? ?  }

 ? ? ? ?// 轉(zhuǎn)化用戶列表
 ? ? ? ?List userVOList = new ArrayList<>(userDOList.size());
 ? ? ? ?for (UserDO userDO : userDOList) {
 ? ? ? ? ? ?UserVO userVO = new UserVO();
 ? ? ? ? ? ?BeanUtils.copyProperties(userDO, userVO);
 ? ? ? ? ? ?userVOList.add(userVO);
 ? ? ?  }

 ? ? ? ?// 返回用戶列表
 ? ? ? ?return userVOList;
 ?  }
}

[{"description":"This is a tester.","name":"tester"},...]

4.2.問(wèn)題分析

for (UserDO userDO : userDOList) {
 ? ?UserVO userVO = new UserVO();
 ? ?userVO.setId(userDO.getId());
 ? ?userVO.setName(userDO.getName());
 ? ?userVO.setDescription(userDO.getDescription());
 ? ?userVOList.add(userVO);
}

Object value = readMethod.invoke(source);
if (Objects.nonNull(value) && ClassUtils.isAssignable(writeMethod.getParameterTypes()[0], value.getClass())) {
 ? ... // 賦值相關(guān)代碼
}

4.3.避坑方法

5.Set對(duì)象排重

5.1.問(wèn)題現(xiàn)象

/** 城市輔助類 */
@Slf4j
public class CityHelper {
 ? ?/** 測(cè)試主方法 */
 ? ?public static void main(String[] args) {
 ? ? ? ?Collection cityCollection = readCities2("cities.csv");
 ? ? ? ?log.info(JSON.toJSONString(cityCollection));
 ?  }

/** 讀取城市 */
 ? ?public static Collection readCities(String fileName) {
 ? ? ? ?try (FileInputStream stream = new FileInputStream(fileName);
 ? ? ? ? ? ?InputStreamReader reader = new InputStreamReader(stream, "GBK");
 ? ? ? ? ? ?CSVParser parser = new CSVParser(reader, CSVFormat.DEFAULT.withHeader())) {
 ? ? ? ? ? ?Set citySet = new HashSet<>(1024);
 ? ? ? ? ? ?Iterator iterator = parser.iterator();
 ? ? ? ? ? ?while (iterator.hasNext()) {
 ? ? ? ? ? ? ? ?citySet.add(parseCity(iterator.next()));
 ? ? ? ? ?  }
 ? ? ? ? ? ?return citySet;
 ? ? ?  } catch (IOException e) {
 ? ? ? ? ? ?log.warn("讀取所有城市異常", e);
 ? ? ?  }
 ? ? ? ?return Collections.emptyList();
 ?  }

/** 解析城市 */
 ? ?private static City parseCity(CSVRecord record) {
 ? ? ? ?City city = new City();
 ? ? ? ?city.setCode(record.get(0));
 ? ? ? ?city.setName(record.get(1));
 ? ? ? ?return city;
 ?  }

 ? ?/** 城市類 */
 ? ?@Getter
 ? ?@Setter
 ? ?@ToString
 ? ?private static class City {
 ? ? ? ?/** 城市編碼 */
 ? ? ? ?private String code;
 ? ? ? ?/** 城市名稱 */
 ? ? ? ?private String name;
 ?  }
}

編碼,名稱
010,北京
020,廣州
010,北京

[{"code":"010","name":"北京"},{"code":"020","name":"廣州"},{"code":"010","name":"北京"}]

但是，并沒(méi)有對(duì)城市“北京”進(jìn)行排重。

5.2.問(wèn)題分析

當(dāng)向集合 Set 中增加對(duì)象時(shí)，首先集合計(jì)算要增加對(duì)象的 hashCode ，根據(jù)該值來(lái)得到一個(gè)位置用來(lái)存放當(dāng)前對(duì)象。如在該位置沒(méi)有一個(gè)對(duì)象存在的話，那么集合 Set 認(rèn)為該對(duì)象在集合中不存在，直接增加進(jìn)去。如果在該位置有一個(gè)對(duì)象存在的話，接著將準(zhǔn)備增加到集合中的對(duì)象與該位置上的對(duì)象進(jìn)行 equals 方法比較：如果該 equals 方法返回 false ，那么集合認(rèn)為集合中不存在該對(duì)象，就把該對(duì)象放在這個(gè)對(duì)象之后；如果 equals 方法返回 true ，那么就認(rèn)為集合中已經(jīng)存在該對(duì)象了，就不會(huì)再將該對(duì)象增加到集合中了。所以，在哈希表中判斷兩個(gè)元素是否重復(fù)要使用到 hashCode 方法和 equals 方法。hashCode 方法決定數(shù)據(jù)在表中的存儲(chǔ)位置，而 equals 方法判斷表中是否存在相同的數(shù)據(jù)。

分析上面的問(wèn)題，由于沒(méi)有重寫(xiě) City 類的 hashCode 方法和 equals 方法，就會(huì)采用 Object 類的 hashCode 方法和 equals 方法。其實(shí)現(xiàn)如下：

public native int hashCode();
public boolean equals(Object obj) {
 ? ?return (this == obj);
}

/** 城市類 */
@Getter
@Setter
@ToString
private static class City {
 ? ?/** 城市編碼 */
 ? ?private String code;
 ? ?/** 城市名稱 */
 ? ?private String name;

 ? ?/** 判斷相等 */
 ? ?@Override
 ? ?public boolean equals(Object obj) {
 ? ? ? ?if (obj == this) {
 ? ? ? ? ? ?return true;
 ? ? ?  }
 ? ? ? ?if (Objects.isNull(obj)) {
 ? ? ? ? ? ?return false;
 ? ? ?  }
 ? ? ? ?if (obj.getClass() != this.getClass()) {
 ? ? ? ? ? ?return false;
 ? ? ?  }
 ? ? ? ?return Objects.equals(this.code, ((City)obj).code);
 ?  }

 ? ?/** 哈希編碼 */
 ? ?@Override
 ? ?public int hashCode() {
 ? ? ? ?return Objects.hashCode(this.code);
 ?  }
}

[{"code":"010","name":"北京"},{"code":"020","name":"廣州"}]

5.3.避坑方法

List citySet = new ArrayList<>(1024);
Iterator iterator = parser.iterator();
while (iterator.hasNext()) {
 ? ?citySet.add(parseCity(iterator.next()));
}
return citySet;

Map cityMap = new HashMap<>(1024);
Iterator iterator = parser.iterator();
while (iterator.hasNext()) {
 ? ?City city = parseCity(iterator.next());
 ? ?cityMap.put(city.getCode(), city);
}
return cityMap.values();

6.公有方法代理

6.1.問(wèn)題現(xiàn)象

/** 用戶服務(wù)類 */
@Service
public class UserService {
 ? ?/** 超級(jí)用戶 */
 ? ?private User superUser;

/** 設(shè)置超級(jí)用戶 */
 ? ?public void setSuperUser(User superUser) {
 ? ? ? ?this.superUser = superUser;
 ?  }

 ? ?/** 獲取超級(jí)用戶 */
 ? ?public final User getSuperUser() {
 ? ? ? ?return this.superUser;
 ?  }
}

/** 公司服務(wù)類 */
@Service
public class CompanyService {
 ? ?/** 公司DAO */
 ? ?@Autowired
 ? ?private CompanyDAO companyDAO;
 ? ?/** 用戶服務(wù) */
 ? ?@Autowired
 ? ?private UserService userService;

 ? ?/** 刪除公司 */
 ? ?public void deleteCompany(Long companyId, Long operatorId) {
 ? ? ? ?// 設(shè)置超級(jí)用戶
 ? ? ? ?userService.setSuperUser(new User(0L, "admin", "超級(jí)用戶"));

 ? ? ? ?// 驗(yàn)證超級(jí)用戶
 ? ? ? ?if (!Objects.equals(operatorId, userService.getSuperUser().getId())) {
 ? ? ? ? ? ?throw new ExampleException("只有超級(jí)用戶才能刪除公司");
 ? ? ?  }

 ? ? ? ?// 刪除公司信息
 ? ? ? ?companyDAO.delete(companyId, operatorId);
 ?  }
}

/** AOP配置類 */
@Slf4j
@Aspect
@Configuration
public class AopConfiguration {
 ? ?/** 環(huán)繞方法 */
 ? ?@Around("execution(* org.changyi.springboot.service..*.*(..))")
 ? ?public Object around(ProceedingJoinPoint joinPoint) {
 ? ? ? ?try {
 ? ? ? ? ? ?log.info("開(kāi)始調(diào)用服務(wù)方法...");
 ? ? ? ? ? ?return joinPoint.proceed();
 ? ? ?  } catch (Throwable e) {
 ? ? ? ? ? ?log.error(e.getMessage(), e);
 ? ? ? ? ? ?throw new ExampleException(e.getMessage(), e);
 ? ? ?  }
 ?  }
}

6.2.問(wèn)題分析

public class UserService$$EnhancerBySpringCGLIB$$a2c3b345 extends UserService implements SpringProxy, Advised, Factory {
 ?  ......
 ? ?public final void setSuperUser(User var1) {
 ? ? ? ?MethodInterceptor var10000 = this.CGLIB$CALLBACK_0;
 ? ? ? ?if (var10000 == null) {
 ? ? ? ? ? ?CGLIB$BIND_CALLBACKS(this);
 ? ? ? ? ? ?var10000 = this.CGLIB$CALLBACK_0;
 ? ? ?  }

 ? ? ? ?if (var10000 != null) {
 ? ? ? ? ? ?var10000.intercept(this, CGLIB$setSuperUser$0$Method, new Object[]{var1}, CGLIB$setSuperUser$0$Proxy);
 ? ? ?  } else {
 ? ? ? ? ? ?super.setSuperUser(var1);
 ? ? ?  }
 ?  }
 ?  ......
}

6.3.避坑方法

7.公有字段代理

public class ParseConfig {
 ? ?public final SymbolTable symbolTable = new SymbolTable(4096);
 ?  ......
}

7.1.問(wèn)題現(xiàn)象

/** 用戶服務(wù)類 */
@Service
public class UserService {
 ? ?/** 超級(jí)用戶 */
 ? ?public final User superUser = new User(0L, "admin", "超級(jí)用戶");
 ?  ......
}

/** 公司服務(wù)類 */
@Service
public class CompanyService {
 ? ?/** 公司DAO */
 ? ?@Autowired
 ? ?private CompanyDAO companyDAO;
 ? ?/** 用戶服務(wù) */
 ? ?@Autowired
 ? ?private UserService userService;

 ? ?/** 刪除公司 */
 ? ?public void deleteCompany(Long companyId, Long operatorId) {
 ? ? ? ?// 驗(yàn)證超級(jí)用戶
 ? ? ? ?if (!Objects.equals(operatorId, userService.superUser.getId())) {
 ? ? ? ? ? ?throw new ExampleException("只有超級(jí)用戶才能刪除公司");
 ? ? ?  }

 ? ? ? ?// 刪除公司信息
 ? ? ? ?companyDAO.delete(companyId, operatorId);
 ?  }
}

7.2.問(wèn)題分析

7.3.避坑方法

/** 用戶服務(wù)類 */
@Service
public class UserService {
 ? ?/** 超級(jí)用戶 */
 ? ?public static final User SUPER_USER = new User(0L, "admin", "超級(jí)用戶");
 ?  ......
}

/** 使用代碼 */
if (!Objects.equals(operatorId, UserService.SUPER_USER.getId())) {
 ? ?throw new ExampleException("只有超級(jí)用戶才能刪除公司");
}

/** 用戶服務(wù)類 */
@Service
public class UserService {
 ? ?/** 超級(jí)用戶 */
 ? ?private User superUser = new User(0L, "admin", "超級(jí)用戶");
 ? ?/** 獲取超級(jí)用戶 */
 ? ?public User getSuperUser() {
 ? ? ? ?return this.superUser;
 ?  }
 ?  ......
}

/** 使用代碼 */
if (!Objects.equals(operatorId, userService.getSuperUser().getId())) {
 ? ?throw new ExampleException("只有超級(jí)用戶才能刪除公司");
}

(1)JavaBean類必須是一個(gè)公共類，并將其訪問(wèn)屬性設(shè)置為public，如：public class User{......}(2)JavaBean類必須有一個(gè)空的構(gòu)造函數(shù)：類中必須有一個(gè)不帶參數(shù)的公用構(gòu)造器(3)一個(gè)JavaBean類不應(yīng)有公共實(shí)例變量，類變量都為private，如：private Integer id;(4)屬性應(yīng)該通過(guò)一組getter/setter方法來(lái)訪問(wèn)。

后記