程序员"/>
优雅编程之做1个“正常”的程序员
开心一笑
他喜欢上一个在自习室一直坐在他前面的女孩。经过无数次挣扎，他递给女孩一张字条：你好！我注意你很久了，能和你做个朋友吗？就又回去做题。女孩看完，收拾了书本到他座位前问：我要走了，你要不要和我一起走？接下来，他说了一句可能是他一生中说过的最经典的话：你先走吧，我还有道题没做……
视频教程
大家好，我录制的视频《Java之优雅编程之道》已经在CSDN学院发布了，有兴趣的同学可以购买观看，相信大家一定会收获到很多知识的。谢谢大家的支持……
视频地址：
提出问题
项目中碰到的几个代码优化问题？？？
简书地址：
解决问题
先看看下面的代码，不知道给你的第一感觉是什么？
/*** 获得设计件当前编辑的章节名称* @param deView* @return*/
@Override
public List<FormResult> getEditChapter(DeView deView,List<FormResult> formResultList) throws Exception{/** 从后端查询出来的数据，不用管是什么数据 **/ResultData resultData = webFormService.getDesignSuitHis(deView.getVid(), deView.getProjectId(), deView.getRevision());for(int i=0;i<10;i++){........//连接数据库，查询数据代码(只是伪代码，不用太较真)webForm = webFormService.findId(id);........}/** 判空 待修改（2） **/if(null != formResultList && formResultList.size() >0){for(int i=0;i<formResultList.size();i++){StringBuffer editorChapper = null;List<TreeModel<TitleDTO>> treeModels = (List<TreeModel<TitleDTO>>)resultData.getEntities().get("titleTreeModel");/** 判空 **/if(null != treeModels && treeModels.size() >0){AllBreak:for(TreeModel<TitleDTO> treeModel:treeModels){List<TitleDTO> titleDTOList = treeModel.getData().getChildren();/** 判空 **/if(null != titleDTOList && titleDTOList.size() > 0){for(TitleDTO titleDTO:titleDTOList){editorChapper = new StringBuffer();/** 判空 **/if((null != titleDTO.getChildForm()) && titleDTO.getChildForm().equals(formResultList.get(i).getFormId())){if(null != titleDTO.getBindAttribute()){String bindEntityVariable = titleDTO.getBindEntityVariable();Map<String, IDataObject> map = (Map<String, IDataObject>)resultData.getEntities().get("dataObjects");IDataObject iDataObject = map.get(bindEntityVariable);String text = iDataObject.getAttributes().get("name").toString();/** 拼凑完整章节 **/editorChapper.append(titleDTO.getTitleNumber()).append(" ").append(text);/** 设置章节名称 **/formResultList.get(i).setChapterName(editorChapper.toString());/** 设置章节Id**/formResultList.get(i).setTitleId(titleDTO.getId());}else{/** 拼凑完整章节 **/editorChapper.append(titleDTO.getTitleNumber());editorChapper.append(" ");editorChapper.append(titleDTO.getText());/** 设置章节名称 **/formResultList.get(i).setChapterName(editorChapper.toString());/** 设置章节Id**/formResultList.get(i).setTitleId(titleDTO.getId());}break AllBreak;}}}}}}}return formResultList;
}
上面代码有很多问题，我们一个一个解决：
1.for循环中查询数据库
既下面这段代码，在for循环中，连接数据库，查询数据。我们都知道，连接数据库，本身就很耗性能
for(int i=0;i<10;i++){
……..
//连接数据库，查询数据代码(只是伪代码，不用太较真)
webForm = webFormService.findId(id);
……..
}
解决：
- 根据业务场景，把for循环中的多次连接数据库查询，写到sql中去查询，既一次性查询出来
- 根据业务场景，看是否可以利用缓存，提高查询效率
2.代码层级太深的问题
**
解决：
针对待修改（2）**，我们要学会用逆向思维的方法，修改如下：
/*** 获得设计件当前编辑的章节名称* @param deView* @return*/
@Override
public List<FormResult> getEditChapter(DeView deView,List<FormResult> formResultList) throws Exception{/** 从后端查询出来的数据，不用管是什么数据 **/ResultData resultData = webFormService.getDesignSuitHis(deView.getVid(), deView.getProjectId(), deView.getRevision());/** 判空 待修改（1） ，现在用逆向思维的方法，修改如下**/if(null == formResultList || formResultList.size() == 0) return；for(int i=0;i<formResultList.size();i++){......省略代码...... }return formResultList;
}
3.方法过长
代码整洁之道，对于函数有这么一句话：函数只应该做一件事情，把一件事情做好，而且只由它来做这一件事情
所有上面的代码其实可以抽出很多方法出来：
既：
/*** 获得设计件当前编辑的章节名称* @param deView* @return*/
@Override
public List<FormResult> getEditChapter(DeView deView,List<FormResult> formResultList) throws Exception{/** 从后端查询出来的数据，不用管是什么数据 **/ResultData resultData = webFormService.getDesignSuitHis(deView.getVid(), deView.getProjectId(), deView.getRevision());for(int i=0;i<10;i++){........//连接数据库，查询数据代码(只是伪代码，不用太较真)webForm = webFormService.findId(id);........}/** 判空 待修改（2） **/if(null == formResultList || formResultList.size() == 0) return；for(int i=0;i<formResultList.size();i++){StringBuffer editorChapper = null;List<TreeModel<TitleDTO>> treeModels = (List<TreeModel<TitleDTO>>)resultData.getEntities().get("titleTreeModel");/** 判空 **/if(null != treeModels && treeModels.size() >0){AllBreak:for(TreeModel<TitleDTO> treeModel:treeModels){List<TitleDTO> titleDTOList = treeModel.getData().getChildren();/** 判空 **/if(null != titleDTOList && titleDTOList.size() > 0){for(TitleDTO titleDTO:titleDTOList){editorChapper = new StringBuffer();/** 判空 **/if((null != titleDTO.getChildForm()) && titleDTO.getChildForm().equals(formResultList.get(i).getFormId())){if(null != titleDTO.getBindAttribute()){//伪代码而已，不用太计较 method3（）；}else{method4（）；}break AllBreak;}}}}}}return formResultList;
}/***方法3*/
public void method3（）{String bindEntityVariable = titleDTO.getBindEntityVariable();Map<String, IDataObject> map = (Map<String, IDataObject>)resultData.getEntities().get("dataObjects");IDataObject iDataObject = map.get(bindEntityVariable);String text = iDataObject.getAttributes().get("name").toString();/** 拼凑完整章节 **/editorChapper.append(titleDTO.getTitleNumber()).append(" ").append(text);/** 设置章节名称 **/formResultList.get(i).setChapterName(editorChapper.toString());/** 设置章节Id**/formResultList.get(i).setTitleId(titleDTO.getId());
}/***方法4*/
public void method4（）{/** 拼凑完整章节 **/editorChapper.append(titleDTO.getTitleNumber());editorChapper.append(" ");editorChapper.append(titleDTO.getText());/** 设置章节名称 **/formResultList.get(i).setChapterName(editorChapper.toString());/** 设置章节Id**/formResultList.get(i).setTitleId(titleDTO.getId());
}
不知道大家有没有注意一点
if（）{..业务代码1
}else{..业务代码2
}for(int i=0;i<100;i++){..业务代码3
}
对于if，和for中的代码，如果是处理业务代码，一般可以抽出单独的一个代码,改成
if（）{method1();
}else{method2();
}for(int i=0;i<100;i++){method3();
} method1（）{//业务代码1
}method2（）{//业务代码2
}method3（）{//业务代码3
}
一切目的都是验证代码简洁之道的那句话!
再看下面的代码：
代码开始new一个PpProject对象，但是在下面的if逻辑中才使用上，假如if逻辑没进入的话，那new出来的对象就使用不上，针对这种情况，就该把上面的代码移到if逻辑判断中，这样才是合理的
读书感悟
来自《把时间当作朋友》
- 心理健康的人不仅应该有能力看清真实的世界，还应该有足够的能力感知真实的自我
- 所谓耐心，就是甘于把时间投入到简单、枯燥但是最终会意义非凡的重复当中去。
- “实际上，我敢打赌，在你自己不喜欢的事儿上专注2 个小时，未经训练的你肯定做不到——因为没有任何人不经训练就可以做到。你可以参照所谓的时间分割法。比如，你需要在这件事儿上专注2 个小时，即120 分钟。那你应该把当天的任务分解成6 块，而每一块用20 分钟完成。你把20 分钟当作你专注的基本时间单位，而每个时间单位过后，休息5 分钟，想办法犒劳一下自己——喝杯咖啡或者牛奶，不怕健康有问题那就抽一支烟，或者给你的男朋友或女朋友打个电话说点儿肉麻的话……在属于休息时间的5 分钟之内的最后一分钟，重新振作，尝试着恢复状态，之后进入下一个基本时间单位——另一个20 分钟。于是，在你规划时间的时候，你应该明白为了能够完全专注120 分钟，你最终需要规划出150 分钟左右的时间开销。这个方法非常简单，但非常有效。”
- 记住，一个人的幸福程度，往往取决于他多大程度上可以脱离对外部世界的依附。”
- 生活的智慧就在于，集中精力改变那些能够改变的，而把那些不能改变的暂时忽略掉。专心打造自己，把自己打造成一个优秀的人，一个有用的人，一个独立的人，比什么都重要。”这话的某些部分好像在某个地方见过，似曾相识。
- 当你把时间花在一个人身上的时候，相当于在他的身上倾注了你生命的一段，不管最终结果如何，反正，那个人、那件事都成了你生命的一部分——不管最后你是喜欢还是不喜欢。
- 往往并不是有兴趣才能做好，而是做好了才有兴趣——别说没有兴趣
- 学习上的成功，都只靠两件事：策略和坚持，而坚持本身就是最重要的策略。 坚持,其实就是重复;而重复,说到底就是时间的投入，我是说，大量的时间投入。
2、与其不停地找更好的方法，还不如马上开始行动，省的虚度更多的时间。 3、真正拥有耐心，甚至惊人的耐心，生活会从此发生翻天覆地的变化。 4、今天所有的轻松安逸，都可能是未来的成本。 5、真正最重要的任务永远只有一个：那个真正对你的目标实现有帮助的任务。
- 浪费生命、虚度年华的人，有个共同的特征——他们拼命想控制自己完全不能控制的，却在自己真正能掌控的地方彻底失控。
- 要提高自己的社交质量，最好将时间和精力更多的倾注在”把自己变得更加优秀”这件事情上——哪怕只在某一方面。
- 我们总是对短期收益期望过高，却对长期收益期望过低。
- 当我们不停地鼓励所有人的时候，最大的受益者其实是我们自己，因为最终我们会发现，自己开始进入一种他人无法想象的状态，成为一个不需要他人鼓励的人。
- 很多时候,人们的善良来自于软弱,而他们的残暴只不过来自于恐惧.
- 你比别人强一点根本没用，真正有用的是你比别人强很多很多。
- 成长比成功更重要，而且，这才是人人都可以做到的事情，才是人人都值得追求的事情。而成长其实只有一条路——积累。
- 学东西前不用定义一定要有用——别觉得知识现在没用就拒绝去学它
- 相信积累的力量，本质上就是相信复利的力量。
- 人是没办法管理时间的，时间也不听从任何人的管理，它只会自顾自一如既往地流逝。”管理时间”只不过是人们的一厢情愿而已。
其他
如果有带给你一丝丝小快乐，就让快乐继续传递下去，欢迎转载，点赞，顶，欢迎留下宝贵的意见，多谢支持！
更多推荐
优雅编程之做1个“正常”的程序员
发布评论