(Laravel)ラジオボタンの値をDBから取得して編集したい

Question

#システム概要
現在の日付から1週間分の日付を取得して、データベースにデータがあればラジオボタンにチェックを入れる、データがなければ、ラジオボタンにチェックを入れないという仕様のスケジュール登録機能を実装しています。
現在は新規で登録することはできています。

#解決したいこと
データの有無判定をしつつ、常に1週間分の最新の日付で表示したいです。(7行だけ)

現在は欲しいデータは取れています。
そしてデータの有無の判定まではできていますが、viewでforとforeachを使っているため、すごい数の行が出力されます。

#コード

<form action="/cast/mypage/schedule" method="POST">
    @csrf
    <table class="w-100 mb-5">
      <tr class="text-center">
         <th width="70"></th>
         <th width="70"><small>ダメ</small></th>
         <th width="70"><small>OK</small></th>
         <th width="70"><small>夜遅くはOK</small></th>
      </tr>
       
        //1週間の日付とラジオボタンを繰り返しで表示        
      @foreach($schedules as $schedule)
      @for($i=0; $i<7; $i++)
                                
     <tr class="datelist text-center">
        <th class="" width="50"><small>{{ $week[$i] }}<input type="hidden" name="date_at[]" value="{{ $week[$i] }}"></small></th>
                                    
      　<td class=""><input type="radio" name="time_at[{{$i}}]" value="1" {{ $schedule->time_at == 1 ? 'checked' : ''}}></td>
       <td class=""><input type="radio" name="time_at[{{$i}}]" value="2" {{ $schedule->time_at == 2 ? 'checked' : ''}}></td>
       <td class=""><input type="radio" name="time_at[{{$i}}]" value="3" {{ $schedule->time_at == 3 ? 'checked' : ''}}></td>
     </tr>
                                
    @endfor
    @endforeach                     
      </table>

      <div class="profile-store">
          <input type="submit" value="保存する"class="btn btn-lg btn-block btn-primary align-items-center" />
     </div>
</form>

<?php

namespace App\Http\Controllers\Cast;

use App\Http\Controllers\Controller;
use Illuminate\Http\Request;
use App\Models\Cast;
use App\Models\CastAvailableDay;
use Carbon\Carbon;
//use Illuminate\Support\Facades\DB;

class AvailableDayController extends Controller
{
	/**
    * Create a new controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->middleware('auth:cast');
    }

    //create画面でデータの登録と編集を一括で行いたい
    public function create(){
　　　　　//ユーザーに紐づいたスケジュールを取得
        $cast = auth()->user();
        $schedules = $cast->castAvailableDays;
　　　　　
　　　　　//1週間分取得
        $numOfDays = 7; 
        $dt = new Carbon();
        $format = 'YYYY-MM-DD';
        $week[0] = $dt->today()->isoFormat($format);
        for ($i=1; $i < $numOfDays ; $i++) {
          $week[$i] = $dt->copy()->addDay($i)->isoFormat($format);
        }
        return view('cast.mypage.schedule',['week'=>$week, 'schedules'=>$schedules]);
    }

    
    public function store(Request $request){
        
        // $schedules = $request->all();
        // unset($schedules['_token']);

　　　　 //スケジュール新規登録の処理
        foreach($request->date_at as $i => $val){
            $schedule = new CastAvailableDay;
            $schedule->time_at = $request->time_at[$i];
            $schedule->cast_id = auth()->user()->id;
            $schedule->date_at = $request->date_at[$i];
            $schedule->save();

        }

        return redirect('/cast/mypage/schedule')->with('flash_message', '登録に成功しました');
    }
}

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Model;

class CastAvailableDay extends Model
{
    protected $fillable = [
        // 'id',
        'cast_id',
        'date_at',
        'time_at',
        // 'created_at',
        // 'updated_at',
    ];

    protected $casts = [
        'date_at' => 'date',
    ];

    /**
     * リレーション Cast >> CastImage
     *
     * @return App\Models\Cast
     */
    public function cast()
    {
        return $this->belongsTo('App\Models\Cast', 'cast_id', 'id');
    }
}

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

class CreateCastAvailableDaysTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('cast_available_days', function (Blueprint $table) {
            $table->bigIncrements('id')->comment('対応可能日ID');
            $table->bigInteger('cast_id')->comment('ユーザーID');
            $table->date('date_at')->comment('日付');
            $table->integer('time_at')->default('0')->comment('時間帯');
            $table->timestamps();
        });
    }

    
}

#環境
laravel6
php7.2

Comments

[m.ts10806]

>現在のコードでデータベースにデータがあれば、ラジオボタンにチェック、データがなければ、ラジオボタンにチェックを入れないという仕様を実装できるものでしょうか？

これが質問でしたら「出ている結果が全て」という回答になります。
できてないなら「実装できない」　できているなら「実装できている」

ひとまず「データベースのデータ」は正しく画面出力はできている（取得できている）状態ですか？
質問を整理されたほうが良いと思います。

[m.ts10806]

いずれにしても checked が書かれてない時点で「できているわけがない」というのは分かります。

[zlk1985]

ご指摘ありがとうございます。
質問内容の訂正をさせて頂きました。

Answer 1

色々やり方はあるとは思うのですが、例えば以下のような感じはいかがでしょうか。

編集ビュー

    {{-- 1日1行で表示 --}}
    @foreach ($days as $i => $day)
        <tr class="datelist text-center">
            <th class="" width="50">
                <input
                    type="hidden"
                    name="days[{{ $i }}][id]"
                    value="{{ $day->id }}"
                >
                <small>
                    {{ $day->date_at->format('Y-m-d') }}
                    <input
                        type="hidden"
                        name="days[{{ $i }}][date_at]"
                        value="{{ $day->date_at->format('Y-m-d') }}"
                    >
                </small>
            </th>
            @foreach (range(1, 3) as $number)
                <td class="">
                    <input
                        type="radio"
                        name="days[{{ $i }}][time_at]"
                        value="{{ $number }}"
                        @if ($number == $day->time_at) checked @endif
                    >
                </td>
            @endforeach
        </tr>
    @endforeach

AvailableDayController

    //編集ビューを表示する
    public function edit(Request $request)
    {
        $start = today(); // 現在の日付
        $end = today()->addDays(6); // 6日後

        //ユーザーに紐づいたスケジュールを取得
        $cast = auth()->user();
        $schedules = $cast->castAvailableDays()
            ->whereBetween('date_at', [$start, $end])
            ->get();

        //スケジュールを準備
        $days = [];
        foreach ($start->toPeriod($end) as $date) {
            $day = $schedules->firstWhere('date_at', $date);
            if (!$day) {
                $day = new CastAvailableDay(['date_at' => $date]);
            }
            $days[] = $day;
        }

        return view(編集ビュー, ['days' => $days]);
    }

    //更新する
    public function update(Request $request)
    {
        $cast = auth()->user();

        //スケジュールを更新（または作成）
        foreach ($request->days as $day) {
            $cast->castAvailableDays()->updateOrCreate([
                'date_at' => $day['date_at'],
            ], $day);
        }

        //更新後はリダイレクト
        return ...
    }

Comments

[zlk1985]

ご回答ありがとうございます。

データベースから習得して、viewに反映まではできました。
更新あるいは登録で下記エラーが出ます。
「Undefined index: date_at」
dd($request->all())でデバックしてみるとdate_atが取れていなかったので、それが原因だと思われます。自分でも再度確認してみたいと思います。

【デバック内容】
"days" => array:7 [▼
0 => array:2 [▼
"id" => null
"time_at" => "1"
]

[Lulucom]

そうですか・・・Bladeの次の部分にタイプミスなど無いでしょうか？
name="days[{{ $i }}][date_at]"

[zlk1985]

特にスペルミスなどは見あたりませんでした。。。
ctrl+Fで確認もしましたが、完全一致でした。。。

[zlk1985]

すみません。。。
以下の日付表示用のjavascriptが効いて値が送られていないことが原因でした。
コメントアウトすることを失念しており、削除すると無事に動きました。
ご丁寧にありがとうございました！

自己解決できるよう頑張って勉強したいと思います。

<script>
//現在の日時と曜日を取得
// $(function(){
// var now = new Date();
// var wd = ['日', '月', '火', '水', '木', '金', '土'];

// $('.datelist small').text(function(){
// var m = now.getMonth()+1;
// var d = now.getDate();
// var w = wd[now.getDay()];
// now.setDate(now.getDate()+1);
// return m + "/" + d + "(" + w + ")";
// });
// });
</script>