程序员大忌!千万不要在for循环里写sql语句

今天一早,客户反馈说xx功能特别慢,然后我就查看了下那个功能的表,发现只有几千条数据,并不会使这个功能特别慢啊,然后我就开始查找问题,终于功夫不负有心人,在代码中发现了问题

        List<Hospital> hospitalList = hospitalService.listByWhere(hospital);
        if (hospitalList.size() > 0) {
            hospitalList.forEach(v -> {
                User user = userService.getById(v.getWyId());
                if (user != null) {
                    v.setWyName(user.getName());
                }
            });
        }

不知道是谁一时图懒,在for循环里写了查询,几千条数据,就会访问几千次数据库,导致这个功能特别缓慢,然后我大概优化了下代码:

        // 加载用户信息
        List<User> userList = userService.listAll();
        Map<String, String> userMap = new HashMap();
        for (User user : userList) {
            userMap.put(user.getUserId(), user.getName());
        }
        List<Hospital> hospitalList = hospitalService.listByWhere(hospital);
        if (hospitalList.size() > 0) {
            hospitalList.forEach(v -> {
                String name = userMap.get(v.getWyId());
                if (name != null) {
                    v.setWyName(name);
                }
            });
        }

别看只是加了这简单的几行代码,效率可是提升了好几倍,并不是不可以在for循环中写sql,如果你能确保你这张表的数据在几十条或者百条数据之间,那么可以这么写,希望大家在开发过程中少遇到坑。

  • 23
    点赞
  • 65
    收藏
    觉得还不错? 一键收藏
  • 15
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 15
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

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

抵扣说明:

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

余额充值