秒大刀博客

好好学习天天向上

日志

关于我

秒大刀

文章分类

[转]实践中的重构

2013-03-06 18:43:46| 分类：技术积累 | 标签： |举报 |字号大中小订阅

下载LOFTER 我的照片书 |

重构，早就不再是“奢侈品”，而是“日用品”。纵然如此，在自己的工作过程中，还是听到很多关于重构的误解。

首先，重构是日常工作。

许多人问过我这样的问题，要不要拿出专门的时间做重构。作为一个程序员，重构就应该和写代码、写测试一样，是我们日常工作的一部分，只要你写代码，就应该重构。特别是当你在做TDD的时候，如果你只写测试，编代码通过，对不起，那根本不叫TDD，那叫测试先行。目前为止，我还没有见过一个程序员，包括我自己在内，写代码是一遍就写得非常整洁，无需重构的。通常都是第一遍写出的代码只能实现功能，然后，把它重构得符合整洁的要求。这里要说的重点是，别讨论专门拿时间做重构，那就是你日常工作的一部分。

我知道为什么会有人问出这样的问题，这就是接下来要说的：重构不是重写。

对于很多人来说，他嘴上说的是重构，心里想的是重写，所以，这事很大，不是一蹴而就的，所以，要专门划拨时间来做。重写，事确实不小。但也是有技巧可言的。我们这里所说的重写，不是整个系统的重写，那专门立项就好了，这里所说的是要调整项目的某个模块，或是某个设计。

一旦设计到调整，实际上是对整个项目组编码风格的一种影响，因为影响比较大，所以，可以考虑以技术债的形式在项目组内部提出来。如果项目组内达成共识，并且业务人员也同意给这件事留出时间，那就可以着手来做了。想要说服业务人员，最好是告诉他们，花多长时间做这件事，能给未来省下多少时间。即便所有人都同意这个调整的方案，真正在实施的时候，我们也需要按照重构的思路来做，这也是需要一些功夫的。

没有测试的重构就是耍流氓。

对于ThoughtWorks内部的项目，测试多还是有保障的，但我周游天下时亲眼见过一些没有测试的巨大代码库，我能说的，赶紧花时间为你要写的代码补一些测试，然后再说重构的事。

具体到动手重构了，标准只有一个：重构可以随时随地停下来，不破坏任何测试。

这话说得很容易，但真要做到可绝非轻而易举。我看到很多大动干戈的修改，中间无法停下，一旦发觉不对，只好推翻重来。我只能说，对于重构，这种做法只是理解了形，并不理解神。站着说话不要疼，事实上，我个人对重构的理解也是看到了有人用小步重构把一段代码逐步改好之后，震惊之余，才理解原来重构应该这样做。要知道，我也曾经是大刀阔斧之辈，那一刻，我皈依于小步重构。

小步重构，是需要刻意练习的。我是花了不少时间去练习，才能做出一些当初见到的重构效果。时至今日，我依然不敢说自己有十足的把握能够小步重构每一段丑陋的代码，内心中总有一种躁动，想一路劈杀过去，只能不断与自己说，慢点，别急。

对于ThoughtWorker而言，内部总是有这样机会见到小步重构，对于外部的人来说，经常参加ThoughtWorks的活动，应该也可以见到这种做法。

重构易，程序员人人知重构，重构难，做好重构者寥寥无几。无它，刻意练习。

from: 梦想风暴一个小程序员的信口开河

评论这张

转发至微博

阅读(1117)| 评论(0)

历史上的今天

this.p={  m:2,
              b:2,
              loftPermalink:'',
              id:'fks_087065081086086064085082085069072084081070082083081071',
              blogTitle:'[转]实践中的重构',
              blogAbstract:'<div\><div\><img title=\"[转]实践中的重构 - 秒大刀 - 秒大刀 博客\"   alt=\"[转]实践中的重构 - 秒大刀 - 秒大刀 博客\"   style=\"margin: 10px 0px 10px 10px; float: right;\"   src=\"http://img2.ph.126.net/xvZto_Ja-K8AJfI7pLztSw==/6597322654331197661.png\"   \></div\></div\><div\>    重构，早就不再是“奢侈品”，而是“日用品”。纵然如此，在自己的工作过程中，还是听到很多关于重构的误解。</div\><div\><br\></div\><div\>    首先，<b\>重构是日常工作</b\>。</div\><div\><br\></div\><div\>    许多人问过我这样的问题，要不要拿出专门的时间做重构。作为一个程序员，重构就应该和写代码、写测试一样，是我们日常工作的一部分，只要你写代码，就应该重构。特别是当你在做TDD的时候，如果你只写测试，编代码通过，对不起，那根本不叫TDD，那叫测试先行。目前为止，我还没有见过一个程序员，包括我自己在内，写代码是一遍就写得非常整洁，无需重构的。通常都是第一遍写出的代码只能实现功能，然后，把它重构得符合整洁的要求。这里要说的重点是，别讨论专门拿时间做重构，那就是你日常工作的一部分。</div\><div\><br\></div\><div\>    我知道为什么会有人问出这样的问题，这就是接下来要说的：<b\>重构不是重写</b\>。</div\><div\><br\></div\><div\>    对于很多人来说，他嘴上说的是重构，心里想的是重写，所以，这事很大，不是一蹴而就的，所以，要专</div\>',
              blogTag:'',
              blogUrl:'blog/static/205657420132664145196',
              isPublished:1,
              istop:false,
              type:0,
              modifyTime:1362566626866,
              publishTime:1362566626836,
              permalink:'blog/static/205657420132664145196',
              commentCount:0,
              mainCommentCount:0,
              recommendCount:0,
              bsrk:-100,
              publisherId:0,
              recomBlogHome:false,
              currentRecomBlog:false,
              attachmentsFileIds:[],
              vote:{},
              groupInfo:{},
              friendstatus:'none',
              followstatus:'unFollow',
              pubSucc:'',
              visitorProvince:'',
              visitorCity:'',
              visitorNewUser:false,
              postAddInfo:{},
              mset:'000',
              mcon:'',
              srk:-100,
              remindgoodnightblog:false,
              isBlackVisitor:false,
              isShowYodaoAd:false,
              hostIntro:'',
              hmcon:'1',
              selfRecomBlogCount:'0',
              lofter_single:'<iframe width="140" height="560" style="overflow:hidden;" src="http://www.lofter.com/mailEntry.do?blogad=1&blog" frameBorder="0"></iframe>'
            }

{list a as x}
    {if !!x}
    <div class="iblock nbw-fce nbw-f40">
      <a class="fc03 noul" target="_blank" hidefocus="true" href="http://blog.163.com/${x.visitorName}/">
      {if x.visitorName==visitor.userName}
      <img alt="${x.visitorNickname|escape}" onerror="this.src=location.f40" class="cwd bdwa bdc0" src="${fn1(x.visitorName)}&r=${visitor.imageUpdateTime}"/>
      {else}
      <img alt="${x.visitorNickname|escape}" onerror="this.src=location.f40" class="cwd bdwa bdc0" src="${fn1(x.visitorName)}"/>
      {/if}
      </a>
      <div class="cwd vname thide">
        {if x.moveFrom=='wap'}
          <a class="noul pnt" target="_blank" href="http://blog.163.com/services/wapblog.html?frompersonalbloghome"><span title="来自网易手机博客" class="iblock wapIcon"> </span></a>
        {elseif x.moveFrom=='iphone'}
          <a class="noul pnt" target="_blank"><span title="来自iPhone客户端" class="iblock iphoneIcon"> </span></a>
        {elseif x.moveFrom=='android'}
          <a class="noul pnt" target="_blank"><span title="来自Android客户端" class="iblock androidIcon"> </span></a>
        {elseif x.moveFrom=='mobile'}
          <a class="noul pnt" target="_blank" href="http://blog.163.com/services/emsblog.html?frompersonalbloghome"><span title="来自网易短信写博" class="iblock wapIcon"> </span></a>
        {/if}
        <a class="fc03 m2a"  target="_blank" hidefocus="true" href="http://blog.163.com/${x.visitorName}/">
          ${fn(x.visitorNickname,8)|escape}
        </a>
      </div>
    </div>
    {/if}
    {/list}

<#--最新日志，群博日志--> <#--推荐日志-->

<p class="fc06">推荐过这篇日志的人：</p>
    <div>
      {list a as x}
      {if !!x}
      <div class="iblock nbw-fce nbw-f40">
        <a class="fc03 noul" target="_blank" hidefocus="true" href="http://blog.163.com/${x.recommenderName}/">
        <img alt="${x.recommenderNickname|escape}" onerror="this.src=location.f40" class="cwd bdwa bdc0" src="${fn1(x.recommenderName)}"/>
        </a>
        <div class="cwd thide">
          <a class="fc03 m2a" target="_blank" hidefocus="true" href="http://blog.163.com/${x.recommenderName}/">
            ${fn(x.recommenderNickname,6)|escape}
          </a>
        </div>
      </div>
      {/if}
      {/list}
    </div>
    {if !!b&&b.length>0}
    <p  class="fc06">他们还推荐了：</p>
    <ul>
    {list b as y}
      {if !!y}
        <li class="rrb"><span class="iblock">·</span><a class="fc03 m2a" target="_blank" href="http://blog.163.com/${y.recommendBlogPermalink}/?from=blog/static/205657420132664145196">${y.recommendBlogTitle|escape}</a></li>
      {/if}
    {/list}
    </ul>
    {/if}

<#--引用记录--> <#--博主推荐--> <#--随机阅读--> <#--首页推荐--> <#--历史上的今天--> <#--被推荐日志--> <#--上一篇，下一篇--> <#-- 热度 -->

{list a as x}
    {if !!x}
    <div class="hotItem iblock nbw-fce nbw-f40">
      <a class="fc03 noul" target="_blank" hidefocus="true" href="http://blog.163.com/${x.publisherUsername}/">
      {if x.publisherUsername==visitor.userName}
      <img alt="${x.publisherNickname|escape}" onerror="this.src=location.f40" class="cwd bdwa bdc0" src="${fn1(x.publisherUsername)}&r=${visitor.imageUpdateTime}"/>
      {else}
      <img alt="${x.publisherNickname|escape}" onerror="this.src=location.f40" class="cwd bdwa bdc0" src="${fn1(x.publisherUsername)}"/>
      {/if}
      </a>
      <div class="cwd vname thide">
        <a class="fc03 m2a"  target="_blank" hidefocus="true" href="http://blog.163.com/${x.publisherUsername}/">
          ${fn(x.publisherNickname,8)|escape}
        </a>
      </div>
      <a class="f-myLikeIcons hottype {if x.type==1} js-liketype{elseif x.type==2} js-reblogtype{elseif x.type==3} js-sharetype{else}{/if}" target="_blank" hidefocus="true" href="http://blog.163.com/${x.publisherUsername}/"> </a>
    </div>
    {/if}
    {/list}

<#-- 网易新闻广告 --> <#--右边模块结构--> <#--评论模块结构--> <#--引用模块结构--> <#--博主发起的投票-->

页脚

我的照片书 - 手机博客 - 下载LOFTER APP - 订阅此博客

秒大刀博客

导航

日志

[转]实践中的重构

历史上的今天

最近读者

热度

评论

页脚

秒大刀 博客

导航

日志

[转]实践中的重构

历史上的今天

最近读者

热度

评论

页脚

秒大刀博客