0

Laravelでタスクスケジュール機能を利用してクイズがランダムに更新されるという処理を作ったのですが、タスクスケジュールが正常に動いているはずなのに全く変化がありません。TimeQuiz.phpのhandle()の記述がおかしいのでしょうか?
同じプログラムをController.phpに書いた場合普通に動きます。

php artisan schedule:runを実行した結果

Running scheduled command: '/usr/bin/php' 'artisan' command:quiz > '/dev/null' 2>&1

TimeQuiz.php

<?php

namespace App\Console\Commands;

use Illuminate\Console\Command;
use App\Quiz;

class TimeQuiz extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'command:quiz';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'TimeQuiz';

    /**
     * Create a new command instance.
     *
     * @return void
     */
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        $quiz = Quiz::with('answer')
            ->inRandomOrder()
            ->get();

            return $quiz;
    }
}

Kernel.php

<?php

namespace App\Console;

use Illuminate\Console\Scheduling\Schedule;
use Illuminate\Foundation\Console\Kernel as ConsoleKernel;
use Illuminate\Support\Facades\Log;

class Kernel extends ConsoleKernel
{
    /**
     * The Artisan commands provided by your application.
     *
     * @var array
     */
    protected $commands = [
        'App\Console\Commands\TimeQuiz'
    ];

    /**
     * Define the application's command schedule.
     *
     * @param  \Illuminate\Console\Scheduling\Schedule  $schedule
     * @return void
     */
    protected function schedule(Schedule $schedule)
    {
        $schedule->command('command:quiz')
        ->everyMinute()
        ->onSuccess(function () {
            Log::info('成功');
        });
    }
1
  • 1
    タスクでは結果を /dev/null に捨てていますが、タスク経由でプログラムが意図した通りに動いているのかを確認する必要がある気がします。 / 単に「タスクスケジュールが動く」と「タスクスケジュール経由で想定通りプログラムが動く」は別だと考えます。
    – cubick
    Commented 2021年3月11日 8:56

1 件の回答 1

1

今のhandle()では何もしてないのと同じ。

コマンド+スケジュールの便利な使い方はキャッシュに永久に保存。

$quiz = Quiz::with('answer')
            ->inRandomOrder()
            ->get();

cache()->forever('quiz', $quiz);

保存しておいて使う場所で取り出す。

$quiz = cache('quiz');

裏側で保存(スケジュールで定期的に更新)、使う時はキャッシュから取り出すだけなので速い。
これがメリットなのでこの質問の場合はそもそもコマンドで実行してる意味がない。
ランダムならユーザーのアクセスごとに取得したほうがいいのでキャッシュの出番はない。

1
  • 返信ありがとうございます。コマンドを使う場合キャッシュを使った方がいいのはわかりましたが、ユーザーのアクセスごとにキャッシュを取得するというのはコントローラーに別途保存処理を書くということでしょうか?
    – Tikka
    Commented 2021年3月12日 5:44

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。