故事是这样子的。。。

领导让我制定个代码规范（小部门刚成立），然后看了阿里的规约，看到他们不推荐使用三个 left join 以上，心血来潮想测一下 left join 和在 java 中拼哪个快。

由于实力不行，模拟了 200000 条数据，当一次左连接并且加上索引时，很快啊，所有数据大概 200ms，然后我用 java 拼接，算了好久也没出结果。。现在我怀疑是我写的两个 for 复杂度太高了，请大佬们帮忙看一下我的 shi 一样的代码。。

//service 层拼
public Page<PersonInfoDTO> getPersonByJava() {
        long startTime = System.currentTimeMillis();
        IPage<PersonDO> personPage = personMapper.selectPage(new Page<>(),null);
        IPage<AddressDO> addressPage = addressMapper.selectPage(new Page<>(),null);
        //todo 获取 person 信息
        List<PersonDO> personDOList = personPage.getRecords();
        List<PersonInfoDTO> personInfoDTOList = new ArrayList<>();
       //todo 获取所有 address 信息
        List<AddressDO> addressDOList = addressPage.getRecords();
        int personSize = personDOList.size();
        int addressSize = addressDOList.size();
        for (int i = 0; i < personSize ; i++) {
            PersonDO personDO = personDOList.get(i);
            PersonInfoDTO personInfoDTO = new PersonInfoDTO();
            personInfoDTO.setName(personDO.getName());
            personInfoDTO.setSex(personDO.getSex());
            personInfoDTO.setId(personDO.getId());
            for (int j = 0; j < addressSize; j++) {
                AddressDO addressDO = addressDOList.get(j);
                if (addressDO.getName().equals(personInfoDTO.getName())){
                    personInfoDTO.setAddress(addressDO.getAddress());
                    personInfoDTOList.add(personInfoDTO);
                }
            }
        }
        Page<PersonInfoDTO> personInfoDTOPage = new Page<>();
        personInfoDTOPage.setRecords(personInfoDTOList);
        long endTime = System.currentTimeMillis();
        long usedTime  = endTime - startTime;
        System.out.println(usedTime);
        return personInfoDTOPage;
    }

 <!--name 加了索引-->
 <select id="getPersonByJoin" resultType="com.hpy.convertfoodemo.convert.entity.DTO.PersonInfoDTO">
        select t1.id,
               t1.name,
               t1.sex,
               t2.address
        from test.person t1
        left join test.address t2 on t1.name = t2.name
 </select>

xuanbg

2020-11-14 08:04:53 +08:00

所谓屎味代码，归纳下来就是这几种情况：
1 、多余，绕来绕去不知所云的无效代码。解决办法：理清业务逻辑，干掉重写。
2 、逻辑不完备，一旦出现特殊情况就会出 bug 。解决办法：多分析，然后进行归纳，使归纳后的逻辑能覆盖所有的可能性。
3 、低效，使用了不合适的方法处理数据。解决办法：不要因循守旧，也不要人云亦云。扩展视野，多尝试新技术新思路新方法。

darklowly

2020-11-14 14:41:27 +08:00

1 好好写注释，注释的格式是 // 后需要空格

2 好好排版，把代码分成逻辑上的段

3 好好取名字 getPersonByJava，这个名字看起来是单数，正常情况下应该是 listUsers

4 不熟悉 java 不知道 personMapper.selectPage 是干嘛的，选择某一页？

5 分解为两个函数，可能更快，伪代码

UserDTO getUserByID(string id) {
user = getUserRecord(id)
addr = getAddrRecord(id)
return buildUserDTO(user, addr)
}

Page<UserDTO> listUsers() {
// 分页查询用户 ID，具体的看你框架的语法
ids = queryUserIDs()

usersDTO = new ArrayListXXXXXXXXXXXX()
for (id := range ids) {
user = getUserByID(id)
usersDTO.add(user)
}

usersDTO
}

darklowly

2020-11-14 14:42:43 +08:00

1 好好写注释，注释的格式是 // 后需要空格

2 好好排版，把代码分成逻辑上的段

3 好好取名字 getPersonByJava，这个名字看起来是单数，正常情况下应该是 listUsers

4 不熟悉 java 不知道 personMapper.selectPage 是干嘛的，选择某一页？

5 分解为两个函数，可能更快，伪代码

```
UserDTO getUserByID(string id) {
user = getUserRecord(id)
addr = getAddrRecord(id)
return buildUserDTO(user, addr)
}

Page<UserDTO> listUsers() {
// 分页查询用户 ID，具体的看你框架的语法
ids = queryUserIDs()

usersDTO = new ArrayListXXXXXXXXXXXX()
for (id := range ids) {
user = getUserByID(id)
usersDTO.add(user)
}

usersDTO
}
```

老哥们，我感觉我写的代码像 shi 一样。。求指点

故事是这样子的。。。

领导让我制定个代码规范（小部门刚成立），然后看了阿里的规约，看到他们不推荐使用三个 left join 以上，心血来潮想测一下 left join 和在 java 中拼哪个快。