Laravel 验证控制器存储中的变量

Question

我的问题找不到答案，希望有人能帮助我

我想验证我是否添加了一个新的约会，而所选员工在约会当天尚未被选中。所以我不能在一天内重复预订某人。 我正在使用 laravel 5.6 和 MySQL 与表约会使用以下行： id、day、employee_id 和 resource_id

我的控制器是一个资源控制器（具有索引、创建、存储等功能）。

因此，如果 $appointmentExists 为 1，我需要抛出错误并停留在创建表单的同一页面上。

public function store(Request $request)
{
    $appointmentExist = \DB::table('appointments')
        ->where([
            ['day','=',$request->day],
            ['employee_id','=',$request->employee_id],
        ])
        ->exists();
    $request->validate([
        'day' => 'required|min:1|max:10',
        'employee_id' => 'required',
        'resource_id' => 'required',
        $appointmentExist => 'in:0',
    ]);
    $appointment = Appointment::create(['day' => $request->day, 'employee_id' => $request->employee_id, 'resource_id' => $request->resource_id]);
    return redirect('/appointments/' . $appointment->id);
}

希望有人能帮忙

Answer 1

所以我自己找到了答案，也许其他人可以使用它：

if(\DB::table('appointments')
    ->where([
        ['day','=',$request->day],
        ['employee_id','=',$request->employee_id],
    ])
    ->exists()){
        return redirect()->back()->withErrors(['This employee is already busy for that day, select another employee or another day.']);
    };

所以现在我回复错误“该员工那天已经很忙了，...”。 我还没有找到如何从 $request->validate() 返回错误，但在这种情况下我不需要它。如果您知道，请随时告诉我。

Answer 2

你的问题是这一行：

$appointmentExist => 'in:0',

这是检查in_array($request->input($appointmentExist), [0])，但$request->input($appointmentExist) 将检查$request->input(0) 或$request->input(1)，这两种技术在技术上都不存在。

我会改为使用请求添加：

$exists = \DB::table(...)->exists(); // Same query, just assigned to a variable
$request->request->add(["exists", $exists]);

$request->validate([
  ...,
  "exists" => "in:0"
]);

通过将密钥 "exists" 添加到请求负载中，您可以像验证请求中发送的实际数据一样验证它，并立即返回所有错误。

根据@N.B. 的评论，以上内容只会防止这种情况下的重复预订；如果验证失败，则永远不会调用Appointment::create()，也不会插入数据。

考虑到这一点，如果验证意外通过，最好有一个回退，在这种情况下，如果您真的想防止重复预订，则对 employee_id 和 day 的组合使用 unique 约束，并像这样处理：

try {
  Appointment::create(...);
catch (\Illuminate\Database\QueryException $qex){
  \Log::error("Unable to Create Appointment: ".$qex->getMessage());

  // Handle individual codes
  if($qex->getCode() == "23000"){
    return redirect()->back()->withErrors(...);
  }
}

Answer 3

$request->validate([
        'day' => 'required|min:1|max:10',
        'employee_id' => 'required',
        'resource_id' => 'required',
        $appointmentExist => 'in:0',
    ]);

此无效代码。验证器将在请求数据中搜索 1 或 0 ($appointmentExist)。这些键永远不会包含在此请求中。

尝试使用规则类。示例：

$day = $request->day;

$request->validate([
    'day' => 'required|min:1|max:10',
    'employee_id' => [
        'required',
            Rule::unique('appointments')->where(function ($query) use ($day) {
               return $query->where('day', $day);
            })
        ],
    'resource_id' => 'required'
]);