使此代码更加面向对象的最佳方法是什么？

Question

我是 Java 和面向对象设计概念的新手。 我有一段我知道可以改进的代码，但我不确定最好的方法。

我有一个数据流（授权值）进来，我正在尝试将授权 ID 转换为值的有效性的映射。 为了测试有效性，我需要检查它的授权值类型，并将其发送到远程客户端以确保它有效。

我在这里创建授权有效性地图：

private Map<UUID, Boolean> getAuthorizationValidityMap(UUID accountId) {
        final Map<UUID, Boolean> authValidity = findCredentialsByAccountId(accountId)
                .parallelStream()
                .map((auth -> Pair.of(auth.getId(), isAuthValid(auth))))
                .collect(Collectors.toMap(res -> res.getLeft(), res -> res.getRight()));
        return authValidity;
    }

查找授权值是否有效的函数。 授权值类型有多种，每一种都需要检查不同的有效性。 我实现它的方式感觉很笨拙，因为随着更多的身份验证类型被添加，它只会导致更多的 else if 语句。

private Boolean isAuthValid(AuthorizationValue authValue) {
        if (authValue.getType().equals(AUTH_TYPE_A)) {
            return isAuthTypeAValid(authValue);
        } else if (authValue.getType().equals(AUTH_TYPE_B)) {
            return isAuthTypeBValid(authValue);
        } else if (authValue.getType().equals(AUTH_TYPE_C)) {
            return isAuthTypeCValid(authValue);
        } 
        return false;
    }

Answer 1

在运行时，您可以决定应该使用哪个Authorize对象来检查您的凭据。 它可以通过策略模式来完成。 授权策略将存储在集合中，您可以使用工厂模式获取授权实例。

所以让我们看一个实现的例子。 我将通过 C# 展示一个示例，但是我已经注释了在 Java 中可以使用哪些数据结构。 这是Auth的属性类型：

public enum AuthType
{ 
    A, B, C, D
} 

然后您可以创建抽象类Authorization ，它将定义所有派生类的行为。 这些派生类将验证值：

public abstract class Authorization 
{
    public abstract bool IsValid(string value);
}

public class Authorization_A : Authorization
{
    public override bool IsValid(string value)
    {
        // null check should be implemented
        return true;
    }  
}

public class Authorization_B : Authorization
{
    public override bool IsValid(string value)
    {
        // null check should be implemented
        return true;
    }
}

我们需要一个工厂，它将通过AuthType返回授权实例：

public class AuthorizationFactory 
{
    // HashTable in Java
    private Dictionary<AuthType, Authorization> _authByType
        = new Dictionary<AuthType, Authorization>()
    {
        { AuthType.A, new Authorization_A() },
        { AuthType.B, new Authorization_B() }
    };

    public Authorization GetInstance(AuthType authType)
    { 
       return _authByType[authType];
    }
}

这是获取身份验证类型的模拟方法：

AuthType GetAuthType() 
{
    // your logic here of getting AuthType
    return AuthType.A;
}

还有一种使用工厂的方法：

bool IsAuthValid(AuthType authType) 
{
    AuthorizationFactory authorizationFactory = new AuthorizationFactory();
    Authorization authorization = authorizationFactory.GetInstance(authType);
    return authorization.IsValid("some value");
}

你可以这样称呼它：

AuthType authType = GetAuthType();
bool isAuthValid = IsAuthValid(authType);

因此，我们正在获取授权实例以根据AuthType属性进行检查。 模式名称是 Strategy 。

所以我们的代码遵循SOLID 原则的Open Closed原则。 当您想要添加新的授权策略时，您将需要一个新的FooAuthorization类。