重构--有效设计代码

最近在读《重构-改善既有代码的设计》，对其简单进行了一下总结，但是对于里面大多数内容，认为都是无关紧要的。

姑且先简单写一下，如果后续有比较好的重构理念，再进行补充

重构：对软件内部结构的一种调整，目的是在不改变软件可观察行为的前提下，提高其可理解性，降低其修改成本。

原因：

当人们只为短期目的，或是在完全理解整体设计之前，就贸然修改代码，程序将逐渐失去自己的结构，程序员越来越难通过阅读源码而理解原来的设计。

1、代码经过很多人的手，时间变长，每个人的风格不同，是代码变得杂乱

2、写代码时为了短期目的，只为满足功能，没有考虑扩展性与可理解性

三次法则：事不过三，三则重构

第一次做某件事时只管去做；第二次做类似的事会产生反感，但无论如何还是可以去做；第三次再做类似的事，你就应该重构。

添加功能时重构；复审代码时重构；修补错误时重构。

代码的坏味道：

重复代码、过长函数、过大的类、过长的参数列、发散式变化、数据泥团、switch、过多的注释

提炼函数：针对过长的函数

可以将部门独立的功能抽取出来，作为单独的函数来调用。同时函数名要能代表抽取这部分代码的功能，使人容易理解。

同时要注意是否引用局部变量，处理好局部变量与抽取新函数之间的调用关系。

拆分时注意不要粒度过于细，否则会造成函数功能过于简单，同时数量太多，反而不利于维护
 

void printinfo(){

        printBanner();

        System.out.println("print test");

        System.out.println("print info");

    }

void printinfo(){

        printBanner();

        printInfo();

    }

    void printInfo(){

        System.out.println("print test");

        System.out.println("print info");

    }

2、引入解释性变量

表达式有可能非常复杂而难以阅读，这种情况下，临时变量可以帮助你将表达式分解为比较容易管理的形式

boolean volumeAndCommissionRate = (volume==0) && (commissionRate >=50) && userType.equals("0") &&

                        (null == dsr || dsr==50000 || dsr==0);

                

boolean volumeAndPrice = (volume==0) && (zkFinalPrice==0.1 || zkFinalPrice==0.01) && userType.equals("0");

               

boolean priceAndDsr = first && ( dsr==0 || null == dsr || dsr==50000 ) && (zkFinalPrice<=1) && userType.equals("0");

                

if(priceAndCommissionRate || volumeAndCommissionRate || volumeAndPrice || priceAndDsr){

     System.out.println("test info");

}

if((volume==0) && (commissionRate >=50) && userType.equals("0") &&

                        (null == dsr || dsr==50000 || dsr==0) || (volume==0) && (zkFinalPrice==0.1 || zkFinalPrice==0.01) && userType.equals("0") || first && ( dsr==0 || null == dsr || dsr==50000 ) && (zkFinalPrice<=1) && userType.equals("0")){

    System.out.println("test info");

}

3、移除对参数的赋值

当函数对传入的基本类型的参数，进行操作时，可能会造成语义不清，所以建议如果对变量进行修改，建议赋值给一个新的变量，然后进行操作，这样就能体现函数所带来的变化。

int count(int v1,int v2){

    return v1*89+100

}

int count(int v1,int v2){

    int temp = v1*89+100;

    return temp;

}