lotus

贵有恒何必三更眠五更起 最无益只怕一日曝十日寒

  博客园  :: 首页  :: 新随笔  :: 联系 :: 订阅 订阅  :: 管理


直接用e.printStackTrace来打印日志有错吗?当然不错,但是不好。主要原因有以下几点:

1、占用太多内存,造成锁死
要打印字符串输出到控制台上,需要字符串常量池所在的内存块有足够的空间。然而,因为e.printStackTrace() 语句要产生的字符串记录的是堆栈信息,太长太多,内存被填满了!大量线程产出字符串产出到一半,等待有内存被释放,锁死了,导致整个应用挂掉了。

2、日志交错混合,不易读
从开始学习java,老师就教我们写代码要具有可读性,易读性。同样,日志的输出如果不具有易读性,也是让程序员非常头疼的一件事。那么为什么e.printStackTrace输出的日志具有不易读性了呢?

printStackTrace()默认使用了System.err输出流进行输出,与System.out是两个不同的输出流,那么在打印时自然就形成了交叉。再就是输出流是有缓冲区的,所以对于什么时候具体输出也形成了随机。

/**
     * 根据权限不同路由到不同的页面
     *
     * @param response response
     */
    @RequestMapping("/route/index")
    public void routing(HttpServletResponse response) throws IOException {
        log.info("进入:/route/index 页面");
        List<String> roles = null;
        try {
            //获取当前用户角色列表
            List<RoleModel> curUserRoleList = centerService.getCurUserRoleList();
            log.info("当前用户角色列表,{}", JSON.toJSONString(curUserRoleList));
            //角色列表
            roles = curUserRoleList.stream().filter(it -> it.getDescription() != null).map(RoleModel::getDescription).distinct().collect(Collectors.toList());
            log.info("角色列表,{}", JSON.toJSONString(roles));
        } catch (Exception e) {
            log.error("LoginController.routing", e);
            response.sendRedirect("/");
            return;
        }
        response.sendRedirect(roleUtil.rolesRank(roles));
    }


总结:
短时间内大量请求访问此接口 -> 代码本身有问题,很多情况下抛异常 -> e.printStackTrace() 来打印异常到控制台 -> 产生错误堆栈字符串到字符串池内存空间 -> 此内存空间一下子被占满了 -> 开始在此内存空间产出字符串的线程还没完全生产完整,就没空间了 -> 大量线程产出字符串产出到一半,等在这儿(等有内存了继续搞啊)-> 相互等待,等内存,锁死了,整个应用挂掉了。

建议:使用log日志来替代e.printStackTrace

posted on 2021-07-15 15:08  白露~  阅读(2015)  评论(0编辑  收藏  举报