概述

if else, for, while等是程序中最常用的語句，這些語句有一個共同點——它們的邏輯都封裝在一對“{}”包圍的代碼塊中。在實現復雜的業務邏輯時，會較多地用到這些語句，可能會形成多層的代碼嵌套。代碼的嵌套層數越大，代碼的縮進層次就越深，這將會降低代碼的可讀性。
如下圖所示，如果我們想理解綠色if代碼塊的邏輯，需要先了解前3個代碼塊是如何工作的。

N層嵌套的代碼不僅可讀性差，也難以維護。當需要變更某一層的代碼時，因前后層次的邏輯制約，很容易改出有問題的代碼。
本文要講的“分解大括號”策略正是為了避免這種問題，它也可以使我們的代碼變得更加整潔。

示例

重構前

Security類的HasAccess方法為了判斷用戶是否有權限訪問，它用了4層if嵌套。

隱藏代碼 public class Security {public ISecurityChecker SecurityChecker { get; set; }public Security(ISecurityChecker securityChecker){SecurityChecker = securityChecker;}public bool HasAccess(User user, Permission permission, IEnumerable<Permission> exemptions){bool hasPermission = false;if (user != null){if (permission != null){if (exemptions.Count() == 0){if (SecurityChecker.CheckPermission(user, permission) || exemptions.Contains(permission)){hasPermission = true;}}}}return hasPermission;} }public interface ISecurityChecker {bool CheckPermission(User user, Permission permission); }

HasAccess方法的邏輯可以分解為3個步驟：

當user或permission為null時，返回false

當參數permission是參數exemptions的一個元素時，返回true

返回SecurityChecker.CheckPermission(user, permission)調用的結果

下圖標示了這3個步驟。

這3個步驟都能決定方法的返回結果，所以沒有必要將這些步驟用“{}”嵌套在一起。
我們可以將這3個步驟分離開來，然后指定這3個步驟的執行順序，并盡快返回結果，就能解除這些嵌套的大括號。

重構后

重構后，我們再次閱讀讀HasAccess方法，一眼就知道它所表達的邏輯。
這段代碼也用到了我后面要講的一個重構策略——“盡快返回”。

隱藏代碼 public class Security {public ISecurityChecker SecurityChecker { get; set; }public Security(ISecurityChecker securityChecker){SecurityChecker = securityChecker;}public bool HasAccess(User user, Permission permission, IEnumerable<Permission> exemptions){if (user == null || permission == null)return false;if (exemptions.Contains(permission)){return true;}return SecurityChecker.CheckPermission(user, permission);} }public interface ISecurityChecker {bool CheckPermission(User user, Permission permission); }

小結

較多層次的嵌套代碼，會增加閱讀代碼的難度。
在編碼時，一個方法的嵌套層次應該盡量少。
多少層最合適呢？一般不應該多于一層或兩層。

小酌重構系列目錄匯總 關注keepfool

總結

以上是生活随笔為你收集整理的小酌重构系列[19]——分解大括号的全部內容，希望文章能夠幫你解決所遇到的問題。

如果覺得生活随笔網站內容還不錯，歡迎將生活随笔推薦給好友。