调度 Laravel 队列时遇到问题

Question

我目前正在尝试使用 Laravel 5.4 中的队列发送邮件，以加快一些请求。但由于某种原因，我就是不会解决。

我的工作如下所示：

<?php

namespace App\Jobs;

use Illuminate\Bus\Queueable;
use Illuminate\Support\Facades\Mail;
use Illuminate\Queue\SerializesModels;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Bus\Dispatchable;

class NotificationEmail implements ShouldQueue
{
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

    protected $doer, $does, $user;

    /**
     * Create a new job instance.
     *
     * @param  Podcast  $podcast
     * @return void
     */
    public function __construct($doer, $does, $user)
    {
        $this->doer = $doer;
        $this->does = $does;
        $this->user = $user;
    }

    /**
     * Execute the job.
     *
     * @param  AudioProcessor  $processor
     * @return void
     */
    public function handle()
    {
        $actions = [
            'accepted.invite' => 'accepted your invited.',
            'accepted.requesting' => 'accepted your request.',
            'denied.invite' => 'denied your invite.',
            'denied.requesting' => 'denied your request'
        ];
        Mail::send('emails.notification', [
            'doer' => $this->does,
            'action' => $actions[$this->action]
        ], function ($m)  {
            $m->from('noreply@bigriss.com', 'Bigriss');

            $m->to("myemail@gmail.com", 'Shawn')->subject('New Notification');

            echo "SENT";
        });
    }
}

通过以下方式将其分派到另一个类中：

NotificationEmail::dispatch($doer, $does, $user);

在侦听队列php artisan queue:listen 后，我一分派作业，侦听器就会无休止地运行，试图解析句柄函数。我收到“已发送”消息，但从未发送过电子邮件（正如我在电子邮件提供商上看到的那样），队列实际上从未被删除，尝试次数只会无限增加。我在这里错过了什么吗？这不是队列的好处吗？

Answer 1

您正在将字符串传递给您的 to 函数，而您的闭包中缺少一个变量。

当你有一个匿名函数时，你需要使用use 传递任何额外的变量。我在您的句柄方法中的任何地方都没有看到$user 变量。它需要作为单独的变量传入，因为您不能使用 $this->user 将其传递到闭包中。

现在你有

$m->to("$user->email", 'Shawn')->subject('New Notification');

这从字面上解释为一个字符串，表示 $user->email 因为你没有传递任何东西。（旁注：这里真的没有理由使用它，保存它带有文件路径等的内联变量。您不需要带有此字符串的内联变量。

您需要将其更改为

$user = $this->user;
Mail::send('emails.notification', [
        'doer' => $this->does,
        'action' => $actions[$this->action]
    ], function ($m) use ($user)  {
        $m->from('noreply@bigriss.com', 'Bigriss');

        $m->to($user->email, 'Shawn')->subject('New Notification');

        echo "SENT";
    });

您可能需要考虑使用Laravel Dusk 之类的东西来调试您的队列，并使用logging 来更好地控制这一点，而不是尝试在浏览器中查看“SENT”。

此外，请考虑清理您的网站地址，因为您要从中发布源代码。