优雅编程之做1个“正常”的程序员(三)

开心一笑

他喜欢上一个在自习室一直坐在他前面的女孩。经过无数次挣扎,他递给女孩一张字条:你好!我注意你很久了,
能和你做个朋友吗?就又回去做题。女孩看完,收拾了书本到他座位前问:我要走了,你要不要和我一起走?
接下来,他说了一句可能是他一生中说过的最经典的话:你先走吧,我还有道题没做……

提出问题

项目中碰到的几个代码优化问题???

解决问题

优雅编程之做1个“正常”的程序员(三)_第1张图片
感悟

先看看下面的代码,不知道给你的第一感觉是什么?

/**
 * 获得设计件当前编辑的章节名称
 * @param deView
 * @return
 */
@Override
public List getEditChapter(DeView deView,List 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> treeModels = (List>)resultData.getEntities().get("titleTreeModel");
            /** 判空 **/
            if(null != treeModels && treeModels.size() >0){
                AllBreak:
                for(TreeModel treeModel:treeModels){
                    List 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 map = (Map)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 getEditChapter(DeView deView,List 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

3.方法过长

代码整洁之道,对于函数有这么一句话:函数只应该做一件事情,把一件事情做好,而且只由它来做这一件事情

所有上面的代码其实可以抽出很多方法出来:

既:

/**
 * 获得设计件当前编辑的章节名称
 * @param deView
 * @return
 */
@Override
public List getEditChapter(DeView deView,List 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> treeModels = (List>)resultData.getEntities().get("titleTreeModel");
        /** 判空 **/
        if(null != treeModels && treeModels.size() >0){
            AllBreak:
            for(TreeModel treeModel:treeModels){
                List 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 map = (Map)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
}

一切都验证了代码简洁之道的那句话!

再看下面的代码:

优雅编程之做1个“正常”的程序员(三)_第2张图片
代码

代码开始new一个PpProject对象,但是在下面的if逻辑中才使用上,假如if逻辑没进入的话,
那new出来的对象就使用不上,针对这种情况,就该把上面的代码移到if逻辑判断中,这样才是合理的

读书感悟

来自《把时间当作朋友》

  • 心理健康的人不仅应该有能力看清真实的世界,还应该有足够的能力感知真实的自我
  • 所谓耐心,就是甘于把时间投入到简单、枯燥但是最终会意义非凡的重复当中去。
  • “实际上,我敢打赌,在你自己不喜欢的事儿上专注2 个小时,未经训练的你肯定做不到——因为没有任何人不经训练就可以做到。
    你可以参照所谓的时间分割法。比如,你需要在这件事儿上专注2 个小时,即120 分钟。
    那你应该把当天的任务分解成6 块,而每一块用20 分钟完成。你把20 分钟当作你专注的基本时间单位,而每个时间单位过后,
    休息5 分钟,想办法犒劳一下自己——喝杯咖啡或者牛奶,不怕健康有问题那就抽一支烟,
    或者给你的男朋友或女朋友打个电话说点儿肉麻的话……在属于休息时间的5 分钟之内的最后一分钟,
    重新振作,尝试着恢复状态,之后进入下一个基本时间单位——另一个20 分钟。于是,在你规划时间的时候,
    你应该明白为了能够完全专注120 分钟,你最终需要规划出150 分钟左右的时间开销。这个方法非常简单,但非常有效。”
  • 记住,一个人的幸福程度,往往取决于他多大程度上可以脱离对外部世界的依附。”
  • 生活的智慧就在于,集中精力改变那些能够改变的,而把那些不能改变的暂时忽略掉。
    专心打造自己,把自己打造成一个优秀的人,一个有用的人,一个独立的人,比什么都重要。
    ”这话的某些部分好像在某个地方见过,似曾相识。
  • 当你把时间花在一个人身上的时候,相当于在他的身上倾注了你生命的一段,
    不管最终结果如何,反正,那个人、那件事都成了你生命的一部分——不管最后你是喜欢还是不喜欢。
  • 往往并不是有兴趣才能做好,而是做好了才有兴趣——别说没有兴趣
  • 学习上的成功,都只靠两件事:策略和坚持,而坚持本身就是最重要的策略。
    坚持,其实就是重复;而重复,说到底就是时间的投入,我是说,大量的时间投入。
    2、与其不停地找更好的方法,还不如马上开始行动,省的虚度更多的时间。
    3、真正拥有耐心,甚至惊人的耐心,生活会从此发生翻天覆地的变化。
    4、今天所有的轻松安逸,都可能是未来的成本。
    5、真正最重要的任务永远只有一个:那个真正对你的目标实现有帮助的任务。
  • 浪费生命、虚度年华的人,有个共同的特征——他们拼命想控制自己完全不能控制的,却在自己真正能掌控的地方彻底失控。
  • 要提高自己的社交质量,最好将时间和精力更多的倾注在"把自己变得更加优秀"这件事情上——哪怕只在某一方面。
  • 我们总是对短期收益期望过高,却对长期收益期望过低。
  • 当我们不停地鼓励所有人的时候,最大的受益者其实是我们自己,因为最终我们会发现,
    自己开始进入一种他人无法想象的状态,成为一个不需要他人鼓励的人。
  • 很多时候,人们的善良来自于软弱,而他们的残暴只不过来自于恐惧.
  • 你比别人强一点根本没用,真正有用的是你比别人强很多很多。
  • 成长比成功更重要,而且,这才是人人都可以做到的事情,才是人人都值得追求的事情。而成长其实只有一条路——积累。
  • 学东西前不用定义一定要有用——别觉得知识现在没用就拒绝去学它
  • 相信积累的力量,本质上就是相信复利的力量。
  • 人是没办法管理时间的,时间也不听从任何人的管理,它只会自顾自一如既往地流逝。"管理时间"只不过是人们的一厢情愿而已。

其他

如果有带给你一丝丝小快乐,就让快乐继续传递下去,欢迎转载,点赞,顶,欢迎留下宝贵的意见,多谢支持!

你可能感兴趣的:(优雅编程之做1个“正常”的程序员(三))