无聊系列

在工作中，常遇见乱处理Exception的情况：

1. 要么吞掉异常，不打印任何日志；
2. 要么记录日志时，日志级别不对、或者把重要的出错堆栈信息干掉，在做生产问题排查时，简直让人抓狂。

我这篇博文，也是对记录的一个开源组件，对异常自行K掉，造成我排查耗费了好久的时间--。https://www.cnblogs.com/chongsha/p/11931109.html

下面我们用一段代码对1进行举例，该代码是网上随便搜的，原作者请勿见怪。

 /*
   * 加密
   * 1.构造密钥生成器
   * 2.根据ecnodeRules规则初始化密钥生成器
   * 3.产生密钥
   * 4.创建和初始化密码器
   * 5.内容加密
   * 6.返回字符串
   */
    public static String AESEncode(String encodeRules,String content){
        try {
            //1.构造密钥生成器，指定为AES算法,不区分大小写
            KeyGenerator keygen=KeyGenerator.getInstance("AES");
            //2.根据ecnodeRules规则初始化密钥生成器
            //生成一个128位的随机源,根据传入的字节数组
            keygen.init(128, new SecureRandom(encodeRules.getBytes()));
              //3.产生原始对称密钥
            SecretKey original_key=keygen.generateKey();
              //4.获得原始对称密钥的字节数组
            byte [] raw=original_key.getEncoded();
            //5.根据字节数组生成AES密钥
            SecretKey key=new SecretKeySpec(raw, "AES");
              //6.根据指定算法AES自成密码器
            Cipher cipher=Cipher.getInstance("AES");
              //7.初始化密码器，第一个参数为加密(Encrypt_mode)或者解密解密(Decrypt_mode)操作，第二个参数为使用的KEY
            cipher.init(Cipher.ENCRYPT_MODE, key);
            //8.获取加密内容的字节数组(这里要设置为utf-8)不然内容中如果有中文和英文混合中文就会解密为乱码
            byte [] byte_encode=content.getBytes("utf-8");
            //9.根据密码器的初始化方式--加密：将数据加密
            byte [] byte_AES=cipher.doFinal(byte_encode);
          //10.将加密后的数据转换为字符串
            //这里用Base64Encoder中会找不到包
            //解决办法：
            //在项目的Build path中先移除JRE System Library，再添加库JRE System Library，重新编译后就一切正常了。
            String AES_encode=new String(new BASE64Encoder().encode(byte_AES));
          //11.将字符串返回
            return AES_encode;
        } catch (NoSuchAlgorithmException e) {
            e.printStackTrace();
        } catch (NoSuchPaddingException e) {
            e.printStackTrace();
        } catch (InvalidKeyException e) {
            e.printStackTrace();
        } catch (IllegalBlockSizeException e) {
            e.printStackTrace();
        } catch (BadPaddingException e) {
            e.printStackTrace();
        } catch (UnsupportedEncodingException e) {
            e.printStackTrace();
        }
        
        //如果有错就返加nulll
        return null;         
    }

该段代码主要的问题是：

1. 吃掉了异常，因为是公共类，连日志记录都没有
2. 出现异常后，仍然返回了一个null值。

这个方法在我们平时使用时，如果不读源码，直接使用，第一直觉是，返回正确结果，如果不正确，那就会抛出异常。但是这段代码却返回了null，使用者遇到时，会抓狂，这是什么情况啊，为啥不对，明明没有报错，万般无奈，进代码一看。。。原来是把异常给干掉了。

对此代码做出的改进建议是：

1. 在方法上声明throws是
2. 如果你觉得1方案不爽，可以直接一个大的catch Exception，然后throw new RuntimeException(e.getMessage(), e)；
3. 出错了就是出错了，不能把错误自己干掉，然后返回一个null。

要么记录日志时，日志级别不对、或者把重要的出错堆栈信息干掉，在做生产问题排查时，简直让人抓狂。

在用log4j记录日志时，请正确使用logger.error()来记录日志，请注意该方法的重载，不要使用 Exception的getMessage()方法只记录异常的消息，而把异常的错误堆栈给抛弃，异常的错误堆栈是很有用的信息，会告诉你在哪行代码出错了，这样你可以快速的定位错误。

package com.demo;

public class Test {

    public static void main(String[] args) {
        try {
            int a = 0;
            int b = 1;

            System.out.println(b / a);
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

}

这段代码的错误堆栈信息：

java.lang.ArithmeticException: / by zero
at com.demo.Test.main(Test.java:10)

这行错误信息at com.demo.Test.main(Test.java:10)标明了出错位置，可以快速定位是在什么地方。所以在记录日志的时候，请不要把错误堆栈信息干掉了。