Laravel：控制器或中间件

Question

我正在使用 Laravel 6.9.0。这是我的付款控制器：

class PaymentController extends Controller
{
    public function __construct(){
        $this->middleware('payment.test');
        $this->middleware('payment.check');
    }

    public function pay(){
        $this->payment->pay();
    }

    public function refund(){
        $this->payment->refund();
    }

    public function checkOrder(){
        $this->payment->checkOrder();
    }
}

payment.test 中间件执行以下操作：

1. 写请求日志
2. 检查商家是否存在
3. 解密请求

payment.check middlware 用于检查付款方式，例如 applepay 或 googlelpay。

但是从官方文档来看，中间件是用来过滤HTTP请求的， 验证您的应用程序的用户是否已通过身份验证。

看来payment.check 不属于这个。因此，我将payment.check 中间件更改为控制器。因为所有的方法都要检查支付，所以我把它放在了构造函数中。但是，我必须在检查之前解密请求， 所以我的构造函数现在是

$this->middleware('payment.test');

$this->middleware(function ($request, $next) {
    $this->checkPayment($request);

    return $next($request);
});

checkPayment 看起来像这样：

private function checkPayment($request){
    if($request->aaa == 'aaa'){
        switch($request->type){
            case '001':
                $type = 'apple';
                break;
            case '111':
                $type = 'google';
                break;
            ...

        }
    }else{
        switch($request->code){
            case 'android':
                $type = 'android';
                break;
            ...
        }
    }


    $this->payment = app($type);
}

这让我的控制器很丑。我觉得将它写入中间件看起来更加模块化和清晰。将它写入控制器真的比中间件更好吗？或者有什么其他建议？

Answer 1

我认为你的整个问题是，即使你的逻辑最终通过调用$this->payment = app($type); 来实例化正确的实现。核心概念还是在这个上下文中，你需要哪个提供者。对此有很多选择，服务、工厂等。但是，如果你在Laravel 中做一个容器提供者版本，它真的很强大而且很干净。

想象一下拥有您的PaymentProviderInterface，我将绑定它以解决要使用的支付提供商。将它与您在任何情况下都可以解析请求对象相结合，您可以将所有这些移至提供者，我会创建一个新的。 PaymentProvider 并将其注册为任何其他提供者。

class PaymentsProvider {
    public const APPLE_PAYMENT_PROVIDER = 'apple';

    public const GOOGLE_PAYMENT_PROVIDER = 'google';

    public const ANDROID_PAYMENT_PROVIDER = 'android';

    public function boot() {
        $this->app->bind(PaymentProviderInterface::class, function () {
           /** @var Request $request **/
           $request = resolve(Request::class);

           $type = null;

           if($request->aaa == 'aaa') {
               $type = $this->resolvePaymentByType($request->type);
           } else {
               $type = $this->resolvePaymentByCode($request->code);
           }

           return resolve($type);
        });
    }

    private function resolvePaymentByType(string $type): string
    {
        if ($type === '001') {
            return static::APPLE_PAYMENT_PROVIDER;
        }

        if ($type === '111') {
            return static::GOOGLE_PAYMENT_PROVIDER;
        }

        throw new Exception('Invalid state');
    }

    private function resolvePaymentByCode(string $code): string
    {
        if ($type === static::ANDROID_PAYMENT_PROVIDER) {
            return static::ANDROID_PAYMENT_PROVIDER;
        }

        throw new Exception('Invalid state');
    }
}

总的来说，我不喜欢开关盒，如果你喜欢，可以使用它们，我认为这样更干净。另一个小的优化，而不是让魔术字符串使用常量，它更具可读性并且有很多好处。由于我们将所有内容绑定到PaymentProviderInterface，因此通过像这样解析您的提供者，这将使您的控制器真正干净。为了避免您没有请求的情况，我会在每个方法中注入。

class PaymentController extends Controller
{
    public function pay(PaymentProviderInterface $payment){
        $payment->pay();
    }