老哥们，我感觉我写的代码像 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> 

• 資深大佬 : taogen

  1. 代码有个错误，personInfoDTOList.add(personInfoDTO); 把这行放到第一层 for 循环最后一行而不是第二层 for 循环。修改后再试试时间吧。
  2. 数据库 join 也和两层 for 循环类似，数据库可以有索引（索引效率 n^2 => nlogn ），Java 也可以构建类似索引，如 hashmap 。
  3. 建议 Java 中构建索引（把第二层 for 循环用 addressMap.get(person.getAddress()) 替代，再试试时间效率。

• 資深大佬 : taogen

  代码错误 2：
  if (addressDO.getName().equals(personInfoDTO.getName()))
  to
  if (addressDO.getName().equals(personInfoDTO.getAddress()))

• 資深大佬 : taogen

  另外，测试 SQL 执行时间。SQL 应该直接在数据库中执行，不要用 JDBC 。

• 資深大佬 : xuanbg

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

  有时候觉得自己写出了完美的业务代码，多一个字符都破坏美感，在产品刀架脖子的情况下不得不 if else

• 資深大佬 : hbolive

  @jzphx 这种情况很常见

• 資深大佬 : HolmLoh

  @jzphx #5
  经常碰到这种情况…我的做法是如果后续会拓展很多分支我就会开始翻设计模式

• 資深大佬 : noparking188

  我每天工作都糊

• 資深大佬 : AEDaydreamer

  @noparking188 太精辟了,第二天 review 一下代码发现能优化的地方很多.可是新的需求又来了,真的没时间改.

• 資深大佬 : fangcan

  你们重构完不需要测试部再测下么？还是都是自己测试吗

• 主 資深大佬 : beichenhpy

  @taogen 感谢

• 資深大佬 : darklowly

  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

  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

  不知道怎么支持 markdown,将就看吧。

  还有一点，你的代码也是密密麻麻的，不好看。函数内部，不同的逻辑区，需要分段，段之间用一个空行隔开。