樱之花

叶散的时候,你明白欢聚;花谢的时候,你明白青春.

日志

关于我

yinzhihua2008

分类中“我的实验室”是我在日常工作中的一些知识总结,有些写的比较匆忙,可能大家在阅读时会产生困扰,后期有时间我会重新整理编辑,谢谢大家的到访,您们的支持是我前进的动力！

文章分类

代码重构-代码的坏味道

2012-03-04 12:26:52| 分类：代码重构 | 标签： |举报 |字号大中小订阅

下载LOFTER 我的照片书 |

1. 重复代码。
相关class具有相同代码：抽出代码，放入基类中。
无关class具有相同代码：抽出代码，放入一个独立class或函数中。

2. 过长的函数

动则上千行的函数，我的确见过很多，工作中这样的代码一点也不稀罕。
找出函数中逻辑和意义比较紧凑的部分，抽出来成为一个新的小函数，把局部变量作为参数或返回值。
往往需要对参数列表再次进行重构，如Replace Temp with query, Introduce Parameter等具体方法操作。

3. 过长参数。
6-7个参数就应当考虑传入一个结构体了。

4. 发散式变化。
如果需要增加一个新功能需要修改一些函数，如果需要增加另一个新功能需要修改另外几个函数，表明class在不同的方向上发生了变化了。
这个时候应当把相应的变化提炼到一个新的类中，使得增加的新功能只会改变这个单一的类。

5. 游击战的修改。
如果你要做出某种改变，你需要在不同的class内作出小修改，这个时候应当move method或move field将需修改的代码放到同一class中去。这种情况类似于发散式变化的情况。

6. 移情别恋
数据和加诸其上的操作行为封装在一起形成类。如果某些函数对于别人的class的兴趣高过自己class时，应当分手了。让他走吧，把他移到他所感兴趣的类中去。
这个往往可以通过看函数所用到的数据属于哪个类的更多。

评论这张

转发至微博

阅读(635)| 评论(0)

历史上的今天

this.p={  m:2,
              b:2,
              loftPermalink:'',
              id:'fks_087067092095082064082082080069072081088071087086080068',
              blogTitle:'代码重构-代码的坏味道',
              blogAbstract:'<P\>1. 重复代码。<BR\> <WBR\> 相关class具有相同代码： 抽出代码，放入基类中。<BR\> <WBR\> <WBR\>无关class具有相同代码：抽出代码，放入一个独立class或函数中。 <WBR\></P\>  <P\>2. 过长的函数</P\>  <P\> <WBR\> 动则上千行的函数，我的确见过很多，工作中这样的代码一点也不稀罕。<BR\> <WBR\>找出函数中逻辑和意义比较紧凑的部分，抽出来成为一个新的小函数，把局部变量作为参数或返回值。<BR\> <WBR\> 往往需要对参数列表再次进行重构，如Replace Temp with query, Introduce Parameter等具体方法操作。 </P\>  <P\>3. 过长参数。<BR\> <WBR\> <WBR\> 6-7个参数就应当考虑传入一个结构体了。 </P\>',
              blogTag:'',
              blogUrl:'blog/static/794306720122402652741',
              isPublished:1,
              istop:false,
              type:2,
              modifyTime:0,
              publishTime:1330835212741,
              permalink:'blog/static/794306720122402652741',
              commentCount:0,
              mainCommentCount:0,
              recommendCount:1,
              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:true,
              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/794306720122402652741">${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 - 订阅此博客

樱之花

导航

日志

代码重构-代码的坏味道

历史上的今天

最近读者

热度

评论

页脚