在laravel中实现命令处理程序的问题

Question

我正在为我工作的办公室的人力资源系统实现一个小组件。我目前正在努力允许用户申请假期。我正在尽力使用命令处理程序（希望也使用事件），因为我的控制器总是非常臃肿。

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

首先，用户点击表单上的提交按钮，例如为假期"创建新请求"。这应该转到RequestController上的store（）方法，并启动一个自处理命令，即CreateRequestCommand。这个命令将把用户的请求存储在数据库中，然后继续使用我还没有时间使用的功能。

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

首先，我的表格有一个文件上传。正如你所知，如果我在控制器中添加（）文件，我会看到可以用来上传它的文件属性。然而，如果我将其添加到CreateRequestCommand的handle（）函数中，它会显示为null。所以，我很困惑我应该在哪里执行上传逻辑？

其次，我的表单有9个字段，除了user_id之外，这些字段与我存储在数据库中的内容非常匹配。所以我通常会做的，我知道这被认为是一种倒退的实践，因此我尝试使用命令的原因是，从表单中获取所有输入，然后将其全部放入模型的create（）函数中，将其存储在DB中。现在，我已经将该逻辑提取到了Command中，通过查看Taylor Otwell在laracasts和PurchasePodcastCommand的文档中的示例（或其他内容），我明白我需要将所有相关数据放置在Command的构造函数中，这意味着我在构造函数中有9个变量。现在请原谅我的无知，但构造函数中有9个变量看起来是错误的，所以我现在在问自己——我做得对吗？下面是我在上面提到的类中的一些相关代码。

app\Http\Controllers\RequestController.php

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

app\Commands\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,
        ]);
    }
}

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

Answer 1

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

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

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

更新：作为对具有单独命令处理程序对象的评论的回复，

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

要创建自处理命令对象，请执行以下操作：php artisan make:command NameOfCommand

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

要使用单独的处理程序轻松创建命令，只需使用：

php artisan make:command NameOfCommand --handler

这将在app/commands中创建一个命令对象，只需一个构造函数，还将在app/Handlers/commands 中创建命令处理程序

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

命令处理程序将有一个handle（）方法，其中CommandObject作为参数传递。通过这种方式，您可以使用处理程序的构造函数来注入其他依赖项，例如处理命令所需的存储库。

在命令处理程序handle（）方法中，只需键入$command->attachment等即可获得命令属性

这应该可以回答您的所有问题：）

更新：

更改控制器方法如下：

// 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));