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

2016-07-07 本文已影响926人阿_毅

开心一笑

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

提出问题

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

解决问题

感悟

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

/**
 * 获得设计件当前编辑的章节名称
 * @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个“正常”的程序员（三）

开心一笑

提出问题

解决问题

读书感悟

其他

猜你喜欢

热点阅读