我正在为我工​​作的办公室的 HR 系统实现一个小组件。我目前正在努力让用户申请休假。我正在尽最大努力使用命令处理程序(希望也是事件)，因为我的 Controller 总是非常臃肿。

我想在我的应用程序中执行以下操作:

例如，首先，用户点击表单上的提交以“创建新请求”以获取假期。这应该转到 RequestController 上的 store() 方法并触发一个自处理命令，即 CreateRequestCommand。然后，此命令会将用户发出的请求存储在数据库中，然后继续执行我尚未完成的功能。

在实现上述内容时我面临两个问题:

首先，我的表单有一个文件上传。如您所知，如果我在我的 Controller 中使用 dd() 文件，我将看到可用于上传它的文件属性。但是，如果我在 CreateRequestCommand 的 handle() 函数中 dd() 它，它显示为 null。所以，我对应该在哪里执行上传逻辑感到困惑？

其次，我的表单有 9 个字段，除了 user_id 外，它们几乎与我存储在数据库中的内容匹配。所以通常我会做的，我知道这被认为是回溯练习，因此我尝试使用命令的原因是从表单中获取所有输入，然后将其全部放入模型的 create() 函数中进行存储它在数据库中。现在我已经将该逻辑提取到命令中，并通过在 laracasts 和 PurchasePodcastCommand 的文档(或其他任何内容)中查看 Taylor Otwell 的示例，我明白我需要将所有相关数据放在命令的构造函数中，这意味着我在构造函数中有 9 个变量。现在原谅我的无知，但是在构造函数中有 9 个变量看起来是错误的，所以我现在问自己 - 我这样做对吗？下面是我在上面提到的类中的一些相关代码。

应用\Http\Controllers\RequestController.php

public function store(Requests\CreateRequestRequest $request)
{
    $this->dispatchFrom(CreateRequestCommand::class, $request);
}

应用\命令\CreateRequestCommand.php

class CreateRequestCommand extends Command implements SelfHandling {

    /**
     * Values from request
     * @var mixed
     */
    protected $requestType, $attachment, $manager, $noOfDays, $dateFrom, $dateTo, $location, $contactNumber, $cover;

    /**
     * Create a new command instance.
     *
     * @return void
     */
    public function __construct($requestType, $attachment, $manager, $noOfDays, $dateFrom, $dateTo, $location, $contactNumber, $cover)
    {
        $this->requestType   = $requestType;
        $this->attachment    = $attachment;
        $this->manager       = $manager;
        $this->noOfDays      = $noOfDays;
        $this->dateFrom      = $dateFrom;
        $this->dateTo        = $dateTo;
        $this->location      = $location;
        $this->contactNumber = $contactNumber;
        $this->cover         = $cover;
    }

    /**
     * Execute the command.
     *
     * @return void
     */
    public function handle()
    {
        // store to db here
        dd([
            'requestType'   => $this->requestType,
            'attachment'    => $this->attachment,
            'manager'       => $this->manager,
            'noOfDays'      => $this->noOfDays,
            'dateFrom'      => $this->dateFrom,
            'dateTo'        => $this->dateTo,
            'location'      => $this->location,
            'contactNumber' => $this->contactNumber,
            'cover'         => $this->cover,
        ]);
    }

}

非常感谢任何可以提供任何建议的人。如果您需要查看更多代码或任何内容，我很乐意与您分享。

最佳答案

在 Command 构造函数中有九个变量并没有错。命令对象是处理程序的 DTO(数据传输对象)，因此这不仅是完全可以接受的，而且也是推荐的方式。命令对象负责确保处理程序收到所需的 $command->vars，所以是的，它们应该在构造函数中。

更新: 抱歉忘记了关于文件上传的其他问题:

要在 FormRequestObject 中处理上传的文件，请使用 $request->file('inputname') 而不是 $request->input('name')

更新: 在回复关于有一个单独的命令处理程序对象的评论时，

在 Laravel 5 中，你可以有一个 CommandObject 来处理自己的命令，也可以有一个单独的处理程序。

要创建一个自处理命令对象: php artisan make:命令 NameOfCommand

这将在 app/Commands 中创建一个命令，它将有一个 constructor() 和一个 handle() 方法。所有命令参数都应在构造函数中启动。

要使用单独的处理程序轻松创建命令，您可以简单地使用:

php artisan make:command NameOfCommand --handler

这将在 app/commands 中仅使用一个构造函数创建一个命令对象，它还将在 app/Handlers/Commands 中创建一个命令处理程序

您应该在 Command 对象的构造函数中初始化您的命令属性。

命令处理程序将有一个 handle() 方法，其中 CommandObject 作为参数传递。这样您就可以使用处理程序的构造函数来注入(inject)其他依赖项，例如处理命令可能需要的存储库。

在命令处理程序 handle() 方法中，您可以通过简单地键入诸如 $command->attachment 等内容来获取命令属性...

这应该可以回答您所有的问题:)

更新:

如下更改您的 Controller 方法:

// assign the request properties to variables - note the file is handled differently. 
$requestType = $request->get('requestType'); 
$attachment = $request->file('attachment'); 
$manager = $request->get('manager'); 
$noOfDays = $request->get('noOfDays'); 
$dateFrom = $request->get('dateFrom'); 
$dateTo = $request->get('dateTo'); 
$location = $request->get('location'); 
$contactNumber = $request->get('contactNumber'); 
$cover = $request->get('cover'); 

// dispatch the command 
return $this->dispatch(new CreateRequestCommand($requestType, $attachment, $manager, $noOfDays, $dateFrom, $dateTo, $location, $contactNumber, $cover));

关于php - 在 laravel 中实现命令处理程序的问题，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/29490496/