程序员大忌！千万不要在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，如果你能确保你这张表的数据在几十条或者百条数据之间，那么可以这么写，希望大家在开发过程中少遇到坑。