代码规范:禁用e.printStackTrace()打印异常

平时不注意(关键也没什么大佬指点,哈哈,其实应该多读读阿里巴巴开发手册),今天在跟项目主管探讨云缴费项目的重构过程中,给项目主管看了一下自己写的代码,被吐槽了几句(实在丢脸),因为平时没这个意识,槽点就是下面这句代码:

        try {
			jiaMiData = DESTool.encrypt(desKey, String.valueOf(fa));
		} catch (Exception e1) {
			e1.printStackTrace();
			log.error("data报文加密失败!");
		}

主管说:e1.printStackTrace()这句话相当鸡肋,怎么个鸡肋法呢?

代码执行过程中,当try语句中出现异常时,会执行catch中的语句,java运行时系统会自动将catch括号中的Exception e 初始化(实例化Exception类型对象)。e是此对象引用名称,会自动调用Exception类中指定的方法,比如nullPointerException等等一大串信息,也就是说,e.printStackTrace()这句话会在命令行打印异常信息以及在程序中出错的位置及原因(代码行号)。

特意度娘了一下,e1.printStackTrace()这句话还引发过血案,大概是说很多web的请求被卡住,因为字符串常量池所在的内存块没有空间,根本原因是因为e.printStackTrace() 语句产生的字符串记录的是堆栈信息,太长太多,内存被填满了!

看到这里,十分汗颜,知道为什么主管要吐槽了。e.printStackTrace() 在实际生产中,除了不断的刷控制台,并没用什么卵用,而且会导致日志系统无法采集到异常错误的具体信息比如日期,时间,等级等等,所以正确的(规范)写法是:

try {
			jiaMiData = DESTool.encrypt(desKey, String.valueOf(fa));
		} catch (Exception e1) {
			log.error("data报文加密失败!", e1.getMessage());
		}

 

  • 10
    点赞
  • 39
    收藏
    觉得还不错? 一键收藏
  • 14
    评论
评论 14
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值