2015-09-19 14 views
0

我建立了寧靜的web服務。該服務端點有一個類似的邏輯:如果隱藏程序的流程,我應該重構代碼嗎?

如果輸入驗證成功,執行數據庫操作並返回結果。

如果數據庫操作失敗,返回數據庫錯誤。

如果驗證失敗,返回與在驗證失敗

我的代碼是在這個意義上明確的,你可以閱讀代碼和理解程序的流量信息的響應。

甲同事重構它使得該基本流在一個工具類它主要功能接受請求機構,分貝操作和處理程序重構。

現在的代碼不將流動但基本的邏輯重構。

最佳做法是什麼?你會犧牲重構​​的可讀性嗎?

是的單元測試會減少一點。

謝謝。

+1

我建議在發佈前和http://codereview.stackexchange.com/版本之後。 – jaco0646

+0

也請查看[refactoring]的定義(https://en.wikipedia.org/wiki/Code_refactoring)。將重構與可讀性進行對比是沒有意義的。重構不是一個功能。 – jaco0646

+0

謝謝,我在這裏發佈的問題:http://codereview.stackexchange.com/questions/105125/code-readability-vs-refactoring – ontk

回答

0

你認爲公用事業類,其中包含重構的代碼,將在運營許多其他類似的場景,例如可重複使用的,

  • 驗證輸入,
  • 運行的數據庫操作,
  • 返回結果

如果它可以在其他類似場景中重用,那麼可以重構。

我想這將是更好的重構,你可以從圖中maintanable代碼,重構服務器更好的目的點見。 (如果時間允許,我想,你可以把新的代碼可讀性也,使流動明確。)

相關問題