最近重构一套代码，运用以下几种方法，供参考。

1  公共方法尽可能的使用缓存

 public static List<string> GetRegisteredCompany(){String sql = "SELECT CompanyCode FROM dbo.Company WHERE Suspended='N' ORDER BY CompanyCode";DataTable table = SqlHelper.ExecuteDataset(BaseCommon.GetSystemConnectionStringBuilder().ConnectionString, sql).Tables[0];return (from item in table.AsEnumerable()select item.Field<string>("CompanyCode")).ToList();}

这个方法返回的值，可能会被多个地方用到。应该把这个方法的值缓存起来，供程序多次使用。

2  使用NArrange格式化代码，这个工具，可以将代码格式化，用region分隔开来。

NArrange 0.2.9.0
____________________________________________________________
Copyright (C) 2007-2009 James Nies and NArrange contributors.
All rights reserved.
http://www.NArrange.net

Zip functionality courtesy of ic#code (Mike Krueger, John Reilly).

经过格式化之后，如上图所示，被分成几个区域，看起来条理更清晰。

3  查找重复的代码，提取到公共方法库中。

关于查找程序中重复的代码，有如下工具可以检测重复的代码

1) Simian   http://www.harukizaemon.com/simian/

2)  Team City  http://www.jetbrains.com/teamcity/

3) Resharper plugin  AgentRalph  http://agentralphplugin.googlecode.com/

4)  DevExpress CodeRush

5)  Resharper  Find Similar Code 查找相似的代码

6) CCFinder

7)  Duplo  http://sourceforge.net/projects/duplo/

8) Visual Studio 2012

9) Clone Detective for Visual Studio   http://www.codeplex.com/CloneDetectiveVS

10) Atomiq    http://getatomiq.com

11)  NDepend  http://www.ndepend.com/

12)  CloneDR  http://www.semanticdesigns.com/Products/Clone

工具相当多，如果使用VS2012开发，已经内置此功能。

4 增加测试代码，测试各种情况下公共代码的功能是否正确。

[TestMethod]
public void EmpLicenseTestMethod(){DateTime expired=new DateTime(2013,12,1);string licensee = "Administrator";string company="Microsoft Corporation";string license=GenerateLicense(expired, licensee, company);//保存许可文件XmlDocument doc = new XmlDocument();doc.LoadXml(license);SaveFileDialog dlg = new SaveFileDialog();dlg.RestoreDirectory = true;dlg.Filter = "License file (*.lic)|*.lic|All files (*.*)|*.* ";if (dlg.ShowDialog() == DialogResult.OK){doc.Save(dlg.FileName);}
}

Visual Studio内置的Test工具与技术，再配合Resharper的工具，测试一个方法相当容易实现。

5  应用改善效率的代码替换不合适的代码，或是应用.NET框架的方法重写自定义的方法。

效率改善代码，如下举例

1  不要用string ax=”” 判断ax是否为空, ax.Length==0的效率高于ax==string.Emptyu 一般用as.IsNullorEmpty
2  对于不改变的变量，用常量代替
3  用Linq代替大量的foreach查询
4  MyType t=(MyType) t和t as MyType的效率比较，后者效率好，而且转换失败后不会抛出异常
5  List<T> 代替Array ，Haset<T> ,Dictionary<K,V> 代替Hashtable

.NET的内置的方法，string.Join可以将字符串数组连接成由一个分隔符号连接的一个字符串。比如

例如，如果 separator 为“,”且 value 的元素为“apple”、“orange”、“grape”和“pear”，则 Join(separator, value) 返回“apple, orange, grape, pear”。

如果 separator 为 nullNothingnullptrnull 引用（在 Visual Basic 中为 Nothing），则改用空字符串 (Empty)。

这个方法，我们常常自己实现。应该改用.NET内置的方法。

6  使用并行技术改善系统效率

Parallel.ForEach可以充分利用多核的，提供系统性能。对于一些不需要考虑对数据的前后顺序操作的地方，均可以用这个方法改善。比如删除临时文件。对于需要考虑操作顺序的数据，比如清空数据表数据，要先清除子表，再删除主表，否则会引起外键冲突。应当用lock锁定要删除的数据表，提取出数据后，再进行删除。

7  使用Linq和Lambda表达式改善系统的写法

对于只有几行的方法，可以用Lambda来替代。如果方法很通用，可以写成通用的方法。对DataTable的循环遍历操作，首选Linq技术，可减少出错的机率，也使代码简化。

8 字符串操作

简单的操作不改变，复杂的匹配与查找改用正则表达式，同时要考虑Encoding的问题。

9 公共方法的提取与扩展方法

static类型的公共方法如果过多，因为GC无法回收内存，会影响系统的效率。可以考虑用扩展方法替代，编译时直接被替换为原始类型的静态方法调用。一般添加扩展方法的类型为.NET系统类型，自定义的类型添加扩展方法很少。

10  尽量多的给方法注释

使用vsDocman工具，尽可能的为公共方法书写注释，添加用例