首页 > 代码库 > 重构指南 - 分解复杂判断（Remove Arrowhead Antipattern）
重构指南 - 分解复杂判断（Remove Arrowhead Antipattern）
当代码中有多层嵌套时，会降低代码的可读性，对于以后的修改也增加难度，所以我们需要分解复杂的判断并尽快返回。
重构前代码
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 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); }}
重构后的代码，大大提高了可读性和以后的可维护性。
重构指南 - 分解复杂判断（Remove Arrowhead Antipattern）
声明：以上内容来自用户投稿及互联网公开渠道收集整理发布，本网站不拥有所有权，未作人工编辑处理，也不承担相关法律责任，若内容有误或涉及侵权可进行投诉： 投诉/举报 工作人员会在5个工作日内联系你，一经查实，本站将立刻删除涉嫌侵权内容。