避免複製與貼上

《重構之美》之一

在我的程式設計生涯中，碰到類似copy-paste的情況簡直不勝列舉。在一次專案中，我們對開源專案jasper report進行了擴充套件。我們加入了對新報表型別(cjt_report)的支援。在reportparameteraction類中，我們需要對報表物件reportunit判斷報表型別。於是，我在reportparameteraction類中定義了如下的私有方法：

privatevoidsetreportunittypebyfileresource(reportunit reportunit)else

} }

protected

reportunit reportunit = loadreportunit(context);

setreportunittypebyfileresource(reportunit);

initialrequestcontext(context, reportunit);

intreporttype = reportunit.getreporttype();

if(reporttype == reportunit.

cjt_report)

//……

}

後來，我發現在engineserviceimpl類中同樣需要判斷報表的型別。然而,setreportunittypebyfileresource()方法卻被定義為reportparameteraction的私有方法，無法被engineserviceimpl物件呼叫。最簡單的做法是採用複製的方式，將這段**複製到engineserviceimpl中。好了，如果此時我們不能忍住copy-paste的簡便所帶來的**，或許就會陷入重複的泥沼了。copy的動作絕對應該鳴起刺耳的警聲。我們需要對這一做法保持足夠的警惕。

通過分析setreportunittypebyfileresource()方法，我發現該方法需要操作的物件均與reportunit有關，而本身該方法的職責就是要獲得reportunit的型別，並對其欄位reporttype進行設定，因此，完全可以將它搬移到reportunit中（move method重構，不是嗎？）。由於reportunit已經增加了對reporttype欄位的get和set訪問方法，我發現它們與我要重構的方法實有異曲同工之處，因而決定將其搬移到getreporttype()方法中：

publicintgetreporttype()

returngetreporttypebyfileresource();

} privateintgetreporttypebyfileresource() }

} }

returnreporttype; }

privatebooleanhasreporttypebeenset()else}

注意，在以上過程中，除了使用move method之外，我還運用了rename method以及extract method等重構手法。

protected

reportunit reportunit = loadreportunit(context);

setreportunittypebyfileresource(reportunit);

initialrequestcontext(context, reportunit);

intreporttype = reportunit.getreporttype();

if(reporttype == reportunit.

cjt_report)

//……

}

避免複製與貼上

避免複製與貼上

複製與貼上

貼上複製Android複製和貼上的實現

相關推薦