Tip1:
Bad practice
<a></a>
private void Method()
{
int Var1;
int Var2;
int Var3;
//Some more statements
Var1 = 10;
}
Good Practice
这个方法增加的代码的可读性。易于维护和扩展。
Tip 2 :
如果临时变量只是在一个地方使用,应该避免声明,如下所示。
private string GetData()
{
string temp;
temp = expression + ( var1 * var2).ToString();//Some expression which calculate the desired value
return temp;
Good practice
{
return (expression + (var1 * var2).ToString());
Tip 3
在一类,如果方法中一表达式在不同的重复。封装它,同时提供属性或方法,如下所示。
这样增加了代码的重复使用,易于扩展和维护
代码
Tip 4
基于一个设计原则 - SRP(单一职责原则)一个类,最好只做一件事,只有一个引起它变化的原因。不要在一个类中放置多个功能。
Bad practice:
不要在一个类中同时放置业务逻辑和持久性逻辑。
下面的 'Customer' 类既包含业务逻辑又包含持久性逻辑。因此,存在有两个理由去改造(业务和数据相联系)。这是违反了单一职责原则。
Good practice:
有两组类 - 一组业务和一组持久化的
在下面的片段中,我们有两项类设置一个独有的域逻辑(客户),从而获得通过库类的数据的能力(这是专为数据)。
Tip 5
总是使用抽象/接口规划程序而不是具体的类。这样有利于扩展、松散耦合、插件。
这也将是符合设计原则之一- 里氏替换原则 - 简单的总结的是,每个子类能替代的基类。
例如:如果要我写一个数据持久化类。
Tip 6
移除未使用的变量的代码
Bad practice
在下面的代码声明了var1,但没有使用
public class A
{
private int var1;
private void Method()
//Some statements...
Tip 7
从代码中删除未使用的方法。这有助于实现高度维护的代码在下面的函数方法片段,但声明没有使用。
public class A
private void Method1()
public void Method2()
//Some statements
public class A
//Some statements
Tip 8
删除没有使用类/类型的声明。删除代码中没有使用的命名空间-这通常发生在我们的移除没有使用的类或类型的过程中。
Tip 9
如果你的静态方法要在所有的实体/类型/类中使用,将这个静态方法到实体中。例如:
public class Utility
public static void Method1(Customer cust)
//Some processing on the Entity customer...
}
public static void Method2(some param)
public class Utility
public static void Method2(some param)
//Statements...
public class Customer
public void Method1()
//Do the processing...
Tip 10
Good practice
本文转自麒麟博客园博客,原文链接:http://www.cnblogs.com/zhuqil/archive/2010/03/03/Refactoring-Tips.html,如需转载请自行联系原作者