lotus

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

  博客园  :: 首页  :: 新随笔  :: 联系 :: 订阅 订阅  :: 管理
  1846 随笔 :: 0 文章 :: 109 评论 :: 288万 阅读


直接用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   白露~  阅读(2388)  评论(0编辑  收藏  举报
编辑推荐:
· 探究高空视频全景AR技术的实现原理
· 理解Rust引用及其生命周期标识(上)
· 浏览器原生「磁吸」效果!Anchor Positioning 锚点定位神器解析
· 没有源码,如何修改代码逻辑?
· 一个奇形怪状的面试题:Bean中的CHM要不要加volatile?
阅读排行:
· 分享4款.NET开源、免费、实用的商城系统
· 全程不用写代码,我用AI程序员写了一个飞机大战
· MongoDB 8.0这个新功能碉堡了,比商业数据库还牛
· 白话解读 Dapr 1.15:你的「微服务管家」又秀新绝活了
· 上周热点回顾(2.24-3.2)
点击右上角即可分享
微信分享提示