01
把3000行代码重构成15行
那年我刚毕业，进了现在这个公司。公司是搞数据中心环境监控的，里面充斥着嵌入式、精密空调、总线、RFID的概念，我一个都不懂。
还好，公司之前用Delphi写的老客户端因为太慢，然后就搞了个Webform的替代，恰好我对Asp.Net还算了解，我对业务的不了解并不妨碍我称成为这个公司的一个程序员。
小公司也有小公司的好，人少，进去很快负责代码开发。我当然也就搞这个数据中心智能管理系统啦。
这个系统非常的庞大，尤其牛逼的是支持客户端组态，然后动态生成网页，数据还能通过Socket实时监控(那时我还真就不懂网络编程)。
这个对于当时的我来说，真真是高、大、上呐！！当时跟着了解整个系统大半个月才算能够调试，写一些简单的页面。
在维护系统的过程中，时不时要扩展一些功能，也就接触了下面这个类：
看到没有，就是当年最最流行的三层架构的产物，对于刚出茅庐的毛头小子来说，这是多么专业的文件头注释，还有反射也就算了，这构造函数还能静态的，还能私有的？那时刚接触这么高大上的代码的我，瞬间给跪了！
但是，类写多了，我就感觉越来越别扭，就是下面这段代码：
每增加一个表，除了要改接口、要改DAL、要改BLL之外，还得在这个工厂类添加一个方法，真真是累到手抽筋，即使有当时公司了的G工给我推荐的神器——动软代码生成器，这粘贴复制的几遍，也是让我感觉到异常繁琐。
有时候打键盘稍微累了点，还把复制出来代码改错了，你妹的，难道这就是程序员该干的事情，不，绝对不是！
我想起了一句至理名言：当你觉得代码重复出现在程序中的时候，就应该重构了。是的，在这句话的指导下，我开始了折腾，决定挑战这个高大上的代码，事实证明，思想的力量是无穷的。
那么，怎么修改呢，仔细观察之后，发现其中className的生成跟返回的类型非常类似，只是一个是类名，一个是字符串，这两者之间应该能够关联起来。
于是google了一下(当时GFW还没猖獗起来哈)，隐隐约约就找到了“反射”这两个字，深入了解之后，确定可以完成。
接下来，就是返回的类型了，返回的类型并不固定，但是似乎很有规律……这个似乎好像在哪里见过，对了，模板，C++课程上有讲过的，于是再次google，了解到了C#中使用了泛型代替了C++中的模板。
在学习完泛型和反射之后，并参考了网上的一些文章，我捣鼓出了下面的代码：
没错，就是它了，三层架构年代最流行的工厂类……
看着原来滚十几屏幕的代码，变成了十多行的代码，真是爽到了骨子里去了，太干净了！
唯一让我担忧的是，我进公司的时候，帮忙整理公司申请软件著作权都是需要代码量的，根据代码多少行来评估软件的大小，万一老板知道了我非但没有帮公司增加代码量，还减少了，会不会立即把我开掉？我没敢给我们老板展示我优秀的成果。
所幸，这段代码非但没有出过任何问题，还避免了以前同事老是在新增一个类之后，把代码复制过来，但是没有正确修改的问题，大大提高了效率。
虽然，我没敢大事宣布我的劳动成果，但是这次成功的修改，则彻底让我走上了代码重构的不归路。
看到这里，大家应该知道这个案例是否真实的了吧。我相信，从08年开始的码农们，看到这种类似的代码绝对不比我少。
那么，我想告诉你们的是什么呢？
1.要在编程过程中多思考！
2.编程的思想很重要，请多看点经典的书！
3.从小处着眼，慢慢重构，尤其在应对一个大型的系统！
4.当重复出现的时候，你应该考虑重构了！
5.粘贴复制的代码越少，你的系统越稳定！
02
少用代码生成器
我们来分析一下，为什么我之前的前辈会写出上面的代码。我归结起来有以下几点：
1.因为使用了动软代码生成器，生成代码方便，就没多想了。
2.三层架构的概念倒是了解了，但是没有去深入思考就拿来应用。
3.遇到重复的代码，没有重构的概念，这是思想的问题——思想比你的能力重要。
至今为止，还是很多人使用代码生成器，那么我们应该怎么对待这个问题呢。
我认为，代码生成器确实可以减少你不少工作，但是少用，那些重复性的工作，除了部分确实是没有办法的，其他大部分都是可以通过框架解决的。
举例来说，像三层架构，真正需要用到代码生成器的，也就是Model类而已，其他的完全可以在框架中完成。
因此你要竭尽全力的思考怎么在框架中来减少你的重复性工作，而不是依赖于代码生成器。
另外，如果你还是在用相关的代码生成工具，请重新定义“动软代码生成器”的代码模板，自己写一个模板；或者使用CodeSmith来完全制定自己的代码生成，因为动软给的代码模板真心乱，比如下面这段代码：
for (int n = 0; n < rowsCount; n++)
{
model = new DBAccess.Model.eventweek();
if(dt.Rows[n]["GroupNo"].ToString()!="")
{
model.GroupNo=int.Parse(dt.Rows[n]["GroupNo"].ToString());
}
if(dt.Rows[n]["Week0"].ToString()!="")
{
model.Week0=int.Parse(dt.Rows[n]["Week0"].ToString());
}
if(dt.Rows[n]["Week1"].ToString()!="")
{
model.Week1=int.Parse(dt.Rows[n]["Week1"].ToString());
}
}
首先，你就不能用 var row=dt.Rows[n] 替代吗?其次，直接用int.Parse如果抛出了异常性能得有多低？再次，这段代码要是有点修改，我不是要每个dt.Rows[n]得改一遍？
03
不要重复发明轮子
我们再来看看其他的一些代码：
public List
List
int start=0;
for(int i=0;i if(dev[i]=='^'){ devs.Add(dev.SubString(start,i)); start=i+1; } } return devs; } 有没有很眼熟，没错，这就是对String.Split()函数的简单实现。我的前辈应该是从c++程序员转过来的，习惯了各种功能自己实现一遍，但是他忽略了C#的很多东西。 我们不去评判这段代码的优劣，而实际上他在很长一段时间都运行得很好。我们来看看使用这一段代码有什么不好的地方： 1.重复发明轮子。花费了额外的时间，函数的健壮性和很差。 2.可读性差。其实是一个很简单的功能，但是用上了这么一段函数，起初我还以为有什么特别的功能。 那么，我们应该怎样去避免重复发明轮子呢？我从个人的经历来提出以下几点，希望能够对各位有所帮助： 了解你所学的编程语言的特性。你可以看一本基础的入门书籍，把所有的特性浏览一遍，或者上MSDN，把相关的内容过一遍。 在你决定动手发明一个轮子之前，先搜索一下现成的解决方案。你还可以到CodeProject、GitHub之类的网站搜索一下。 在知乎上有很多人都在批评这么一种现象，老是问一些重复性的问题，然后又职责知乎没落了，没有人回答他的问题，实际上相关问题已经有了很详细的解答，那提问之前，不能首先去搜一下是否有现成的答案，反而指责没有回答他的问题呢？ 你有一定的基础之后，还应该去读一下相关的经典书籍，深入了解其中的原理。比如，你觉得你有一定的基础了，我建议你去把《CLR Via C#》多读几遍，你了解原理越多，你越是能够利用这编程语言的特性，从而来实现原本那些你认为要靠自己写代码的功能。 这里我再举一个我自己的例子。在我现有的程序中，我发现我需要越来越多的线程来执行一些简单的任务，比如在每天检测一下硬盘是否达到90%了，每天9点要控制一下空调的开启而在网上6点的时候把空调关掉。 线程使用越来越多，我越是觉得浪费，因为这些现场仅仅只需完成一次或者有限的几次，大部分时间都是没有意义的，那么怎么办呢？ 我决定自己写一个任务类，来完成相关的事情。说干就干，我很快把这个类写出来了。 public abstract class MissionBase : IMission { private DateTime _nextExecuteTime; protected virtual DateTime[] ExecuteTimePoints { get; private set; } protected virtual int IntervalSeconds { get; private set; } protected IEngine Engine { get; private set; } public bool IsCanceled{get{……}} public bool IsExecuting{get{……}} public bool IsTimeToExecute{get{……}} public abstract bool Enable { get; } public abstract string Name { get; } protected MissionBase(IEngine engine) { ExecuteTimePoints = null;//默认采用间隔的方式 IntervalSeconds = 60 * 60;//默认的间隔为1个小时 Engine = engine; } /// 任务的执行方法 public void Done() { if (Interlocked.CompareExchange(ref _isExecuting, 1, 0) == 1) return; try { …… } finally { Interlocked.CompareExchange(ref _isExecuting, 0, 1); } } ///实际方法的执行 protected abstract void DoneReal(); } 但是，实际上这个任务方法，并不好用，要写的代码不少，而且可靠性还没有保障。当然，我可以继续完善这个类，但是我决定搜索一下是否还有其他的方法。 直到有一天，我再次阅读《CLR Via C#》，看到线程这一章，讲到了System.Threading.Timer以及ThreadPool类时，我就知道了，使用Timer类完全可以解决我的这个用尽量少的线程完成定时任务的问题。 因为从原理上来说，Timer类无论你声明了多少个，其实就只有一个线程在执行。 当你到了执行时间时，这个管理线程会用ThreadPool来执行Timer中的函数，因为使用的ThreadPool，执行完成之后，线程就马上回收了，这个其实就完全实现了我所需要的功能。 04 等你无法重构的时候再考虑重写 我带过很多优秀的程序员，也与很多优秀的程序员共事过。有一大部分的程序员在看到一套系统不是那么满意，或者存在某些明显的问题，就总是忍不住要把整套系统按自己觉得可以优化的方向来重写，结果，重写结构往往并不令人满意。 系统中确实存在很多不合理的地方，但是有不少的这种代码，恰恰是为了解决一些特定场景下的问题的。 也就是说，所有的规范以及编程的原则，其实也是有条件限制的，他可能在大部分的时候是正确的，能够指导你完成你的任务，但是，并不是在所有地方都是适用的。 比如数据库范式，但实际中我们的设计往往会考虑冗余，这是违背范式的，但是为什么还有那么多人趋之若鹜呢？因为我们可能需要用空间换时间。 如果我们一开始就考虑重写，那么你可能会陷入以下的困境： 1.需要花更大的精力来完成一些看似简单的BUG！ 你要知道，有一部分看似错误或者非常不优美的代码，其实恰恰是为了解决一些非常刁钻的问题的。 2.再也无法兼容老的系统了！ 你急于把原有系统重写，却往往忽略了对原有系统的兼容，那么你新的系统的推进则会十分缓慢。而老系统的维护，又会陷入及其尴尬的情况。 3.过度设计，导致重写计划迟迟无法完成！ 有重写冲动的程序员往往是在架构设计上有一些读到的见解，他们善于利用所学的各种设计模式和架构技巧来建立系统，但是越是想尽可能的利用设计模式，越是陷入过度设计的困局，导致重写的计划迟迟都无法完成。 4.无法有效利用现有系统已经完成并测试的代码！ 如果你确实有必要进行重写，我还是建议你把代码尽可能的重构。因为重构之后的系统，能够让你更轻易的重写，又最大限度了保留以前可用的业务代码。 我举个例子，说明如何通过重构更好的利用现有代码的。 我有一个非常庞大的系统，其中有一块功能是用于数据采集、存储、告警管理以及电话、短信等告警通知。大致的结构如下： class MainEngine:IEngine{ public MainEngine(ConfigSettings config){ } public void Start(); public void Stop(); } 需要增加新的业务功能时，程序员写的代码往往是这样的：首先时修改配置类！ class ConfigSettings{ public bool NewFuncEnable{get;private set;} public ConfigSettings(){ NewFuncEnable=xx;//从配置文件读取 } } 接着修改主程序： class MainEngine:IEngine{ private NewFuncClass newCls=new NewFuncClass(); public MainEngine(ConfigSettings config){ } public void Start(){ if(config.NewFuncEnable) newCls.Start(); } public void Stop(){ if(config.NewFuncEnable) newCls.Stop(); } } 在修改的过程中，往往是根据配置文件来判断新功能是否启用。上面代码会造成什么问题呢： 1.主程序代码和扩展功能耦合性太强，每增加一个功能都要修改主程序代码，这里非常非常容易出错。尤其是新的人进度开发组，很容易就忘主程序中增加了一些致命性的代码。 比如上述的扩展功能，可能是在特定的项目中才会有这个扩展功能，但是，写代码的人忘记增加是否启用的配置选项了，导致所有的项目都应用了这个功能，而这个功能需要特定的表，这样就悲剧了。 即使是你增加了配置，也是非常的不美观，因为在通用的版本中使用了这个配置，往往会让定制项目以外的人员感到困惑。 2.增加扩展功能的人还需对整个MainEngine代码有一定的熟悉，否则，他根本就不知道在Start方法和Stop方法进行newClas的对应方法的调用。 3.如果你打算对这段代码进行重写，那么，你会感到非常的困难，因为你分不清楚newCls这个新实例的作用，要么你花大精力去把所有代码理清楚，要么直接就把这段新增的业务代码去掉了。 那么我们如何对这段代码进行重构呢？ 首先，我们把新功能注册的代码抽取出来，通过反射来实现新的功能的注册。 private void RegisterTaskHandlerBundles() { var bundles = xxx.BLL.Caches.ServiceBundleCache.Instance.GetBundles("TaskHandlerBundle"); if (bundles != null && bundles.Count > 0) { var asmCache = new Dictionary foreach (var bundle in bundles) { try { if (!asmCache.ContainsKey(bundle.Category)) asmCache.Add(bundle.Category, Assembly.Load(bundle.AssemblyName)); var handler = (ITaskHandler)asmCache[bundle.Category].CreateInstance(bundle.ClassName, false, BindingFlags.Default, null, new object[] { this, bundle }, null, null); _taskHandlerBundles.Add(bundle, handler); } catch (Exception e) { NLogHelper.Instance.Error("加载bundle[Name:{0},Assembly:{1}:Class:{2}]异常：{3}", bundle.Name, bundle.AssemblyName, bundle.ClassName, e.Message); } } } } 修改MainEngine代码： class MainEngine:IEngine{ private NewFuncClass newCls=new NewFuncClass(); public MainEngine(ConfigSettings config){ RegisterTaskHandlerBundles(); } public void Start(){ _taskHandlerBundles.Start(); } public void Stop(){ _taskHandlerBundles.Stop(); } } OK，现在我们再来看看怎么实现原来的新增功能：你只需按规范新建一个类，继承ITaskHandler接口，并实现接口的方法。 最后在XTGL_ServiceBundle表中新增一条记录即可。我们再来看看这么做有什么好处： 1.新增的类只需按规范写即可，完全对MainEngine代码没有任何影响。你甚至可以把这个MainEngine代码写在一个新建的Dll中。 2.新增功能的这个业务类跟原来的代码解耦，非常方便进行新功能的业务测试，而无需考虑原有框架的影响 3.新增功能的业务类与架构完全分离，我们在重写代码中只要保证接口的稳定性，无论我们怎么把系统架构重写，我们可以马上就重用上原有的业务功能代码。 重构的目标之一，就是把框架和业务完全分离。 有志于深入了解的同学，可以了解下反射、Ioc和插件话编程等。 05 学会单元测试，培养你的重构意识 可能上面说了这么多，还是有很多人并不理解重构。没关系，在这里我教你们一个快速入门的办法，就是单元测试。什么是单元测试，请自行google。单元测试有什么要求？ 就是要求你要把每个方法都弄成尽量可以测试的。尽量让你的方法变成是可测试的，就是培养你重构意识的利器。 在你要求把方法变成可测试的过程，你就会发现你必须得不断的修改你的方法，让它的职责尽量单一，让它尽量的与上下文无关，让它尽可能通过方法参数的输入输出就能完成相关的功能，让依赖的类都尽量改为接口而不是实例。 最终，你就会发觉，这就是重构！而且是在不知不觉中，你重构的功力就会大大提升，你编程的水平也会大大提升！ 看到这里，有经验的程序员就会问，你这是在鼓励我使用TDD吗？不，不是的。 TDD(Test-Driven Development)鼓励的是测试驱动开发，未开发之前先编写单元测试用例代码，测试代码确定需要编写什么产品代码。 这是一种比较先进的开发方法，但是在编程的实践过程中，我认为它过于繁琐，很多中小企业很难实施，更别提我们个人开发者。 我这里提倡你用单元测试培养你的重构意识，可以说是一种后驱动，用于提高你的重构能力和重构愿望，你完全可以把我的这个方法称为“TDR(Test-Driven Refactoring)——测试驱动重构”。 当然，在开发之前如果你有意识的让方法可测试，那么你写出来的函数将会是比较高质量的代码。 当你的函数都是一个个可重用性高的函数之时，你将会发现，写代码其实就像堆积木一样，可以把一个大型的需求分解成无数细小的功能，很快的把需求实现。 以下是一个超大方法中的一段代码，如果你懂得怎样让这段代码编程一个可测试的方法，那么，恭喜你，你入门了。 06 所谓重构 如果你有耐心看到这里，你应该知道，我并非一个标题党，而这篇文章也许称为“如何在编程中应用重构的思想”更为贴切，但是我不想用这么严肃的标题。 很多编程初学者，或者有多年编程经验的人都觉得阅读别人的代码非常困难，重构更是无从谈起，他们要么对这些代码望洋兴叹，要么就是推翻从来。 但是，如果我们有重构的意识，以及在编程的过程中熟悉一些代码调整和优化的小技巧，你自然而然就会培养出重构的能力。 重构，其实很简单： 把基础打牢固！ 多看点优秀的代码！ 避免复制粘贴，如果看见重复代码时应该有意识要消灭它！ 减少对代码生成器的依赖！ 在处理现有代码时尽量用重构代替重写，在重写之前一定要先重构！ 尽量让所有的方法都是可测试的！ 如果你坚持这么去做了，一段时间之后感觉自然就出来了。 重构的目的，是让你的代码更为精简、稳定、能够重用，是最大程度的让功能和业务分离。 在重构的过程中，你的阅读代码的能力、写出优秀代码的能力以及系统架构能力都会稳步提升。 版权说明：本文内容来自网络、版权归原作者所有，如有侵权请联系删除。 相关： 西媒预测2020年将进一步发展的九大技术 西班牙《阿贝赛报》网站12月23日发表一篇题为《2020年的技术趋势》的文章，预测了2020年或将得到进一步发展和普及的技术。现将文章内容摘要如下：一、扩展融合现实虚拟现实经历了一段调整时期。它在市场上的受欢迎程度不如预期，但确实标志着该行业的转折点。越来越多的数字内容开发人员采用鼓励更大沉浸感的技术。但是从现在开始，增强现实和虚拟现实这两项互相联系又有所不同的技术将结合起来。增强现实技术是在静态图像上呈.. 中国15位明星登全球最美最帅榜单，看旧照有些人名不符实每一年，一个叫TC Candler的美国网站都会发布“全球最美/最帅面孔榜单”，今年的评选依旧有不少中国明星上榜。他们现在的样子我们肯定都不陌生，可是如果看旧照的话，不知道还担不担得起“全球最美面孔”的称号呢？最近因为《精英律师》获得了不少关注的朱珠，已经连续好几年上榜了，今年她的排名是72名▼9年前她入行当MTV主持人的时候，脸还是有点稚嫩的，妆容也不走熟女风，但五官跟现在差不多▼可是一旦画上浮夸的浓妆，那时.. 张咪抗癌成功！2岁外孙女首曝光，笑眯眯呆萌可爱今年11月，张咪公布了自己患癌晚期的消息，并乐观表示自己会积极治疗，很多网友纷纷送上祝福，希望她早日康复。日前，张咪的女儿李洛伊表示妈妈目前已经抗癌成功，正在恢复中，自己和舅舅一起照顾妈妈。同时，李洛伊还透露自己已经结婚的消息，并且有了一个两岁的女儿。在与女儿的合影照中，李洛伊母女身穿亲子装出镜，画面温馨。小宝宝笑眯眯的，呆萌可爱。 最新回应！新房水泥一抠就掉，这家开发商“同意退房” 中新经纬客户端12月28日电 近日，浙江义乌在建楼盘金地中梁·西江雅苑被曝房子存质量问题。28日，该楼盘开发商回应，对于所购房屋存在质量问题且不接受整改方案的业主，公司同意退房。 公开报道显示，今年10月底，部分业主代表在西江雅苑项目工地参观时发现，多幢在建新房的不同楼层存在“梁柱轻轻一抠，混凝土就会掉落”“天花板、墙壁、露台、飘窗等处有不同程度开裂”等情况。 据了解，西江雅苑项目位于义乌双江湖.. 华为裁员7000人背后：跟亿万富豪谈感情是你的不幸人生最怕的就是：“机遇来临时，抓不住，危险来临时，看不清”近期华为公司辞退7000员工这件事情震惊各大网络，其实这种事情早就见怪不怪了，早在之前，京东裁员10%的高级副总裁，滴滴（上海区域全部裁员，全国裁员2000人）摩拜（裁员20%-30%）知乎裁员，小米裁员，星巴克裁员……这背后到底隐藏一些什么？借用华为任正非的话：对于一个企业而言，想要长远的发展下去，企业的员工不能太过安逸，这样公司的氛围会非常懒散，从而失.. 华为员工1天的工作曝光，这才是狼性文化来源：商业思维圣经 华为，一直是企业界的标杆。很多企业都主张学习华为，学习华为的“狼性”。甚至同样的员工，在其他公司里是“小白兔”，到了华为，就变成了“狼”。华为员工的1天，工作流程化且紧凑从8：10开始早餐，8：30投入上午的工作，12：00午饭，13：00-14：00午休，大多数人在晚上20：30下班，领取免费宵夜。按照这个时间轴，至少一天12小时需要待在办公室。虽然时间不能代表业绩，但不投入时间是绝对不会有业绩的。.. 刚毕业就年薪201万，阿里腾讯抢着要，这群90后为何这么牛转眼2019年就过去了，今年想必大家对 知识创造财富这句话有了一个深刻的理解，身为90后的钟钊、秦通、左鹏飞等人刚毕业就入选了华为的“天才少年”计划，年薪达到了201万，即使不去华为，腾讯、阿里等各大企业也都向他们抛出了橄榄枝。有一些网友就质疑，他们为什么这么抢手呢，是不是有关系走后门呢？今天我们就来了解一下他们。1991年出生的钟钊本科就读于华中科技大学，学的是软件工程，华中科技大学数学与统计学院2012年消.. 你已被好友删除，微信是这样偷偷提示你的，太扎心了今天在这里，我就要向微信正个名！其实不是微信不出这个功能，而是当你被好友删除的时候，微信已经偷偷在提示你了，只是你没发现而已...所以，千万别再怪微信不出这个功能了。那么，微信是以什么方式提示你，你已经被好友删除了呢？无法向好友转账微信的转账和发红包有什么区别，相信还有很多人不知道，有兴趣的可以自己了解。这里想说的是：如果好友已经将你删除，你是无法向他进行转账的。测试方法：转账0.01元给好友若弹出支.. 泰国8台2020年新剧播报喜欢泰剧的小伙伴看过来，除了3台、7台、GMM等泰国众多优秀电视台的好剧之外，2020年泰国8台好剧也不容错过哦！日前，泰媒对外公布泰国8台2020年新剧，有哪些好作品呢，一起来看看吧！1、《เรือนสายสวาท/情妇之家》（Kate & YT主演）2、《ปอปผีเจ้า/Pop鬼》（Fai & Natrika主演）3、《สะไภ้ไร้ศักดินา/不封建的儿媳》（JakJaan & Nike主演）4、《ภูตรัตติกาล/情迷幽灵》（.. 这期《奇葩说》，请在父母陪同下观看今晚《奇葩说》最让人印象深刻的或许不是哪一方的发言，而是本场“观众沉浸赛”专门请来的50位“爸妈”。诞总和康永哥同时观察到，无论正反双方谁发言，在座的叔叔阿姨们都会点头、微笑，给予眼神肯定。有趣的是，肯定归肯定，但他们几乎不会拍下“葩葩钮”改变最初的持方。这难道就是传说中的“十动然拒”（十分感动然后拒绝了）吗？说服父母真的是一件看似简单实际艰难的事。比如，在拒绝父母逼你穿秋裤这件事上，你是不是难听..