代码重构技巧方法

来源：互联网发布：asp.net 字符串转json 编辑：程序博客网时间：2024/05/17 23:31

我是一个程序员。相信很多的程序员也有过同样的的经历，在公司一边骂别人的代码烂，一边写着比别人更烂的代码。
代码并不是一开始就变坏，代码变坏有以下可能
1、需求变更
2、维护代码的人的水平参差不齐
3、维护人员流动大
4、开发周期短，交付时间紧急
5、个人习惯
6、管理者对代码的要求
以上都是有可能使代码变坏的可能，那么如何写出好的代码？除了个人编程能力外还需要有一定的技巧，以下就是总结前人的经验

得来的，需要在编程中养成一种好习惯，让写好代码成为一种习惯吧！！！！！

1、使用卫语句
在什么情况下可以使用卫语句呢，如下

public void add(Object element) {if (!readOnly) {int newSize = size + 1;if (newSize > elements.length) {Object[] newElement = new Object[elements.length + 10];for (int i = 0; i < size; i++) {newElements[i] = elements[i];}elements = newElements;}elements[size++] = element;}}

使用卫语句重构后，如下

public void add(Object element) {if (readOnly)return;int newSize = size + 1;if (newSize > elements.length) {Object[] newElement = new Object[elements.length + 10];for (int i = 0; i < size; i++) {newElements[i] = elements[i];}elements = newElements;}elements[size++] = element;}

这样嵌套就少了一层，降低复杂度
2、提炼方法
当看见一个过长的函数或者一段需要注释才能让人理解用途的代码，就应该将这段代码放进一个独立函数中，是最常用的重构手法
之一，那么什么情况下可以提炼方法呢
一、重复代码
二、使用助手方法，就是把注释部分提炼成一个函数，如下

public List getUsers(){List users = new ArrayList<User>();//.........//sort by most recently registered usersCollections.sort(users, new User.UserComparatorByDescendingRegistration());return users;}

提炼方法之后

public List getUsers(){List users = new ArrayList<User>();//.........sortByMostRecentlyRegistered(users);return users;}

三、函数结构层次单一，如下
重构前

void printOwing(double amount){        printBanner();        System.out.println("name:" + _name);        System.out.println("amount" + amount);    }

重构后

void printOwing(double amount){        printBanner();        printDetails(amount);    }

四、以查询取代临时变量，如果把临时变量替换为一个查询，那么同一个类中的所有函数都可以获得这份信息；如下
重构前代码

public double getPrice(){    double basePrice = _quantity*_itemPrice;        if (basePrice > 1000){            return basePrice * 0.95;        }else{            return basePrice * 0.98;        }    }

重构后代码

public double getPrice(){        if (BasePrice() > 1000){            return BasePrice() * 0.95;        }else{            return BasePrice() * 0.98;        }    }        private int BasePrice(){        return _quantity* _itemPrice;    }

3、变量作用单一，如下代码的avg有两个作用，意义不明确，容易产生岐义

private double calculateAverage(){Iterator grades = clines.iterator();double avg = 0.0;while(grades.hasNext()){grade = (CourseGrade)grades.next();if(!(grade.grade == "F")){avg = avg + grade.grade - 64;}}avg = avg / clines.size();return avg;}

4、重新组织代码，变量定义与使用的跨度不要过大，即在使用时再定义，更容易理解
重构前代码

MARKETING_DATA *marketingData = new MARKETING_DATA;    SALES_DATA *salesData = new SALES_DATA;    TRAVEL_DATA *travelData = new TRAVEL_DATA;        travelData.ComputeQuarterly();    salesData.ComputeQuarterly();    marketingData.ComputeQuarterly();        salesData.ComputeAnnual();    marketingData.ComputeAnnual();    travelData.ComputeAnnual();        salesData.print();    travelData.print();    marketingData.print();

重构后代码

MARKETING_DATA *marketingData = new MARKETING_DATA;    marketingData.ComputeQuarterly();    marketingData.ComputeAnnual();        SALES_DATA *salesData = new SALES_DATA;    salesData.ComputeQuarterly();    salesData.ComputeAnnual();    salesData.print();        TRAVEL_DATA *travelData = new TRAVEL_DATA;    travelData.ComputeQuarterly();    travelData.ComputeAnnual();    travelData.print();

参考资料：

1、http://www.cnblogs.com/matchcolor/archive/2010/08/02/1784888.html

0 0