给定条件使用子类的实例而不是父类

Question

我正在努力将一个庞大的整体类分解为几个子类，但是一次完成所有工作太多了，所以我希望在时间允许的情况下将它们一个一个地拆分为多个版本。它是一个授权某些通道的身份验证类，所以目前它看起来像这样：

$auth = new Auth($user, $data);
$output = $auth->authChannel($channelName);

在Auth里面，基本上是这样的：

public function __construct($user, $data)
{
    $this->user = $user;
    $this->data = $data;
}

public function authChannel($channel)
{
    $this->setUserData();

    if (isset(self::CHANNEL_AUTH_FUNCTIONS[$channel])) {
        $authFunction = self::CHANNEL_AUTH_FUNCTIONS[$channel];
        return $this->$authFunction();
    } else {
        // invalid channel
    }
}

所以self::CHANNEL_AUTH_FUNCTIONS基本上就是['channelA' => 'authChannelA', 'channelB' => 'authChannelB']等，所有这些功能都在这一类中。

现在我想做的，一次一个，是if $legacyChannel => callLegacyFunction() / else $newChannel => instantiate its own class and call auth()。

所以我将Auth.php 放入它自己的命名空间中，并在同一个命名空间中拥有新的Channel.php 类。和Channel extends Auth。

目前我有这个：

public function authChannel($channel)
{
    $this->setUserData();

    if (isset(self::CHANNEL_AUTH_LEGACY_FUNCTIONS[$channel])) {
        $authFunction = self::CHANNEL_AUTH_LEGACY_FUNCTIONS[$channel];

        if ($authFunction) {
            return $this->$authFunction();
        } else {
            $authClassName = __NAMESPACE__ . '\\' . ucwords($channel);
            $authClass = new $authClassName($user, $data);
            return $authClass->auth();
        }
    } else {
        // invalid channel
    }
}

有没有更好的方法来做到这一点？目前这似乎有点浪费，因为创建了两个不同的对象，并且我相信例如需要再次调用 setUserData() 函数。我也想知道除了__NAMESPACE__ . / . $className之外是否有更好的方法来获取动态类名。

Answer 1

在该代码开始看起来更好之前，您必须进行大量工作。我将尝试建议尽可能少的更改，以使“迁移”尽可能轻松，尽管您离干净的设计还有几步之遥。

首先，您可以为新的身份验证类创建一个AuthStrategyInterface。

interface AuthStrategyInterface
{
    public function supports(string $channel): bool;
    public function auth($user, $data);
}

您的每个新身份验证类都应实现此接口。 supports($channel) 方法很容易理解：如果一个认证类可以处理某个通道，它应该返回true。

您的Auth 课程需要一种方法来注入这些策略。通常你会在构造函数中这样做......但为了保持你的 API 不变，我们只需为此创建一个 setter 方法。

在执行authChannel()时，它会首先检查注入的策略，看是否有支持使用的$channel，并尽可能使用。如果没有，请返回检查您的旧实现。

这样，您在添加新的身份验证策略时无需接触任何旧代码。随着您添加新的实现，您逐渐strangling 遗留系统。有一次没有使用旧的实现，您可以进入新的代码重构阶段。

class Auth {
    private iterable $strategies = [];

    public function __construct($user, $data)
    {
        $this->user = $user;
        $this->data = $data;
    }

    public function setAuthStrategies(iterable $strategies)
    {
        $this->strategies = $strategies;
    }


    public function authChannel($channel)
    {
        $this->setUserData();

        // check if any of the new strategies supports  
        foreach ($this->strategies as $strategy) {
            if ($strategy->supports($channel) {
                return $strategy->auth($this->user, $this->data);
            }
        }

        // check "legacy" authentication methods.
        if (isset(self::CHANNEL_AUTH_FUNCTIONS[$channel])) {
            $authFunction = self::CHANNEL_AUTH_FUNCTIONS[$channel];
            return $this->$authFunction($this->user, $this->data);
        }

        // no valid authentication method
        return false;     
    }
}

要使用它，你可以这样做：

$fooAuthStrategy = new FooAuthStrategy();
$barAuthStrategy = new BarAuthStrategy();
$bazAuthStrategy = new BazAuthStrategy();

$auth = new Auth($user, $data);
$auth->setAuthStrategies(
    [
        $fooAuthStrategy,
        $barAuthStrategy,
        bazAuthStrategy
    ]
);

$auth->authChannel($channel);

具体情况会根据您的应用程序的设置方式而有所不同，但与您当前的方法相比，这样的方法会让您朝着更好的方向走得更远。

Answer 2

我不知道我是否正确理解了这个问题，但你不能那样做吗？

 public function authChannel($channel)
{
    $this->setUserData();

    if (!isset(self::CHANNEL_AUTH_LEGACY_FUNCTIONS[$channel])) {
        // Invalid channel
        return;
    }
    return  self::CHANNEL_AUTH_LEGACY_FUNCTIONS[$channel]
            ? $this->$authFunction()
            : parent::auth();
}