Laravel if文でデータが存在するかで条件分岐させたい。

Question

#解決したいこと
if文を用いて、データが存在しない場合にsaveメソッドを用いてデータを保存したい。

#試したこと
下機の記述を追加

public function send()
    {
    $today = date("Y-m-d");
    $date = \App\StandingUpRecord::where('user_id',Auth::id())->where('created_at', $today)->get();
    if ($date === null){
    
    
      $record = new StandingUpRecord();

      $record->number_of_times = 10;

      $record->user_id = Auth::id();

      $record->save();
    }else{
    $date2 = \App\StandingUpRecord::where('user_id',Auth::id())->where('created_at', $today)->increment('number_of_times', 10);
    
    }
      return view('standing-up-record.rest');

    }

試した結果

データの保存はできなかった。

#コード
app/Http/Controllers/StandingUpRecords.php

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\StandingUpRecord;
use Illuminate\Support\Facades\Auth;

class StandingUpRecords extends Controller
{
    public function index()
    {
        return view ('standing-up-record.index');
    }

    public function new()
    {
        return view ('standing-up-record.new');
    }

    public function rest()
    {
        return view ('standing-up-record.rest');
    }

    public function send()
    {
    $today = date("Y-m-d");
    $date = \App\StandingUpRecord::where('user_id',Auth::id())->where('created_at', $today)->get();
    if ($date === null){
    
    
      $record = new StandingUpRecord();

      $record->number_of_times = 10;

      $record->user_id = Auth::id();

      $record->save();
    }else{
    $date2 = \App\StandingUpRecord::where('user_id',Auth::id())->where('created_at', $today)->increment('number_of_times', 10);
    
    }
      return view('standing-up-record.rest');

    }

}

app/StandingUpRecord.php

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class StandingUpRecord extends Model
{
    protected $fillable = ['number_of_times', 'user_id'];
    protected $table = 'standing_up_records';

    public function user()
{
    return $this->belongsTo('App\User');
}
}

views/standing-up-record/new.blade.php

<form action="send" method="post">
@csrf
<input type="submit" value="10回立ち座りした">
</form>

views/standing-up-record/rest.blade.php

休憩中です

<div class="links">
                    <a href="new">もう10回頑張る</a>
                    </div>

                    <div class="links">
                    <a href="/standing-up-record">今日は終わりにする</a>
                    </div>

よろしくお願いします。

Comments

[FKM]

$date = \App\StandingUpRecord::where('user_id',Auth::id())->where('created_at', $today)->get();

これの結果はきちんとnullを返してくれているのでしょうか。

[退会済みユーザー]

created_atって通常タイムスタンプ（日付＋時刻）だから、$todayとぴったんこってのはないんじゃない？ せめて日付成分だけ抽出しての比較をしないと。

[kuromame6145]

ありがとうございます！！

[kai0310]

$date = StandingUpRecord::where('user_id', Auth::id())->whereDate('created_at', now())->get();

でよさそう。

Answer 1

get()だと対象データが存在しない場合、空のCollectionが返ってきます。

$dateに入る値がStandingUpRecordオブジェクト1つかnullを期待したいのであればfirst()を使うと良いです。

$date = \App\StandingUpRecord::where('user_id',Auth::id())->where('created_at', $today)->first();

$dateには複数のStandingUpRecordオブジェクトを期待しているのであれば、get()で取得したのちにCollectionのデータ件数をcount()で判定すると良いです。

$date = \App\StandingUpRecord::where('user_id',Auth::id())->where('created_at', $today)->get();
if ($date->count() > 0){

} else {

}