码迷,mamicode.com
首页 > 其他好文 > 详细

重构指南 - 分解复杂判断(Remove Arrowhead Antipattern)

时间:2017-01-20 10:02:24      阅读:163      评论:0      收藏:0      [点我收藏+]

标签:att   enum   color   需要   判断   返回   row   count()   ssi   

当代码中有多层嵌套时,会降低代码的可读性,对于以后的修改也增加难度,所以我们需要分解复杂的判断并尽快返回。

重构前代码

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)

标签:att   enum   color   需要   判断   返回   row   count()   ssi   

原文地址:http://www.cnblogs.com/hmloo/p/6321832.html

(0)
(0)
   
举报
评论 一句话评论(0
登录后才能评论!
© 2014 mamicode.com 版权所有  联系我们:gaon5@hotmail.com
迷上了代码!