catch中为什么不建议使用e.printStackTrace

为什么尽量不用e.printStackTrace

直接用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