php - 在 Laravel 中从另一个 Controller 调用 Controller 是一个好习惯吗？

Question

我能够实现一个 PaypalController，它有一个可重用的 postPayment() 方法，它接受元素及其价格，创建一个 Paypal 支付，并重定向到一个 Paypal 支付页面。

class PaypalController extends Controller {

    private static $_api_context;

    private static function initialize() {
        //initialize api context
    }

    public static function postPayment($items, $currency, $description) {
        self::initialize();

        //create item list, transaction, payment objects, etc

        $payment->create(PaypalController::$_api_context);
        ...
        return redirect()->away($redirect_url); // redirect to paypal
    }
}

PaypalController 由其他 Controller 静态调用。例如，AuthController 可能会在用户注册到我的网站后立即调用它来请求用户付款:

class AuthController extends Controller {
    public function postRegister(Request $request) {
        return PaypalController::postPayment($items, 'JPY', 'description');
    }
}

基本上，PaypalController 返回一个 Redirect 到 AuthController，后者也返回它，以执行到 Paypal 支付页面的重定向。

我想知道这是不是一个好的设计 - 一个 Controller 调用另一个 Controller ，是吗？

如果不是，那么执行此操作的更好方法是什么？也许将我的代码从 PaypalController 移到自定义服务提供程序、自定义帮助程序或其他东西中？我是 Laravel 的新手，希望得到一些指导。

Answer 1

不，这不是一个好的做法。您应该将业务逻辑抽象为服务/存储库类。例如:

创建一个接口(interface)作为契约(Contract):

namespace App\Services\Paypal;

interface PaypalInterface {

     public function PostRegister(Array $array, /*More $params if necessary*/); 
}

然后执行合约:

namespace App\Services\Paypal;

class PaypalService implements PaypalInterface {

    // Must match the method signature declared in the interface
    public function PostRegister(Array $array, /*$More $params if necessary*/) {

        // Do the process here
    }
}

然后使用契约/接口(interface)作为依赖。因此，在您的 PaypalController 或任何其他 Controller 中，您可以(重新)使用它:

namespace App\Http\Controllers;

use App\Http\Request;
use App\Services\Paypal\PaypalInterface;

class AuthController extends Controller {
    public function postPayment(Request $request, PaypalInterface $paypalService) {
        return $paypalService->postRegister($request->all());
    }
}

在这种情况下，register the binding (实现的接口(interface))在服务提供商中(基本上在 AppServiceProvider 中)。这是基本的工作流程。为什么是接口(interface)，因为 Controller (客户端/消费者类)应该与 Contract/Interface 对话，而不是具体的实现。

This article of mine may help you但请记住，这不是 100% 解耦，它仍然与 Laravel 框架耦合，您甚至可以进一步解耦服务。

---

注意:这是一个最佳实践，但不要盲目地对每个项目/问题都遵循这种方法，只要明智地选择你应该做的时候，它真的取决于上下文，但不要为此而死。当前上下文可以遵循此。