公司做HW外包,每行代码都要经过checkstyle,重复,圈复杂度的各种考验,而检测工具包含篡改过的第三方,或HW的DIY。公司的同事在各种规则下,也找到了各种应对方法。
昨天发现项目一个Bug,Debug后发现如下代码段(非原码,仅示例)
1 public static Object errExample(byte[] bytes){ 2 Object obj = null; 3 ObjectInputStream ois; 4 try { 5 ByteArrayInputStream bis = new ByteArrayInputStream(bytes); 6 ois = new ObjectInputStream(bis); 7 obj = ois.readObject(); 8 } catch (IOException e) { 9 // Wrong, because may be subclass of IOExcption 10 Logger.getRootLogger().error("IOException is : " + "diy message."); 11 } catch (ClassNotFoundException e) { 12 Logger.getRootLogger().error("ClassNotFoundException is " + "diy message."); 13 } 14 return obj; 15 }
在第10行,捕捉了非运行时异常IOException,直接手工写了IOException,但实际项目中抛出的却是子类EOFException。原因呢,是在Socket实现RMI时byte数组长度写死,而实际的对象转码后超过门限,多余的被截去了。
所以在记录日志时,只使用代码应该就差不多了。
Logger.getRootLogger().error(e);
因为类Throwable已经写的够用了。
1 public String toString() { 2 String s = getClass().getName(); 3 String message = getLocalizedMessage(); 4 return (message != null) ? (s + ": " + message) : s; 5 }