質問編集履歴

コントローラーを調整

2020/01/14 18:21

投稿

kamille-mio

スコア24

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -236,7 +236,7 @@
-#### Controller(自分でtry-catchを組んでみました、全体としては未完成なので抜粋してます)
+#### Controller(自力で予約確定まで組んでみたものです)
 ```php
@@ -244,24 +244,6 @@
-namespace App\Http\Controllers;
-use App\Reserve;
-use App\Exceptions\ReserveDuplicationException;
-use App\Http\Requests\CreateReserveRequest;
-use Carbon\Carbon;
-use Illuminate\Http\Request;
 class ReserveController extends Controller {
@@ -412,7 +394,7 @@
         }
-        // try-catchで例外が出たら処理中止する。
+        // try-catchで例外が出たら入力フォームまでロールバックする。
@@ -444,7 +426,7 @@
-            // 入力確認ページのviewにinputsを渡す
+            // 入力確認ページのviewにdataを渡す
             return view('reserve-confirm', [
@@ -470,220 +452,244 @@
+        // 予約完了メールの発送
+        public function send(Request $request) {
+            //フォームから受け取ったactionの値を取得
+            $action = $request->input('action');
+            $inputs = $request->except('action');
+            //actionの値で分岐
+            if($action !== 'submit'){
+                return redirect()
+                    ->route('reserve.index')
+                    ->withInput($inputs);
+            } else {
+                $user = Auth::user();
+                $email = $user->email;
+                \Mail::to($email)->send(new ContactSendmail($inputs));
+            // トークンを再発行して再送信防止
+                $request->session()->regenerateToken();
+            }
+    }
+// 予約確定による、データベースへの情報の追加
+    public function store(Request $request) {
+        // バリデーション
+        $request->validate([
+             'facility_name' => 'required',
+             'dateinfo' => 'required',
+             'start_time' => 'required',
+             'end_time' => 'required',
+        ]);
+        // 再度ダブルブッキングのチェックを行う。
+        $this->confirm($request);
+        // 以下、ダブルブッキングなしの場合の更新処理
+        //フォームから受け取ったactionを除いたinputの値を取得
+        $inputs = $request->except('action');
+        // 検索用にfacility_nameをのみ別に変数に取り出しておく
+        $facility_name = $inputs['facility_name'];
+        // facility_idを抽出するメソッド
+        $facility_id = Facility::SearchFacility_id($facility_name);
+        // dateinfoとstart_time及びend_timeを組み合わせてdatetime型にする。
+        $start_time =$inputs['dateinfo'] .' '. $inputs['start_time'];
+        $end_time =$inputs['dateinfo'] .' '. $inputs['end_time'];
+        // 予約番号生成
+        $reserve_number =  uniqid(bin2hex(random_bytes((1))));
+        // ユーザー情報取得
+        $user = Auth::user();
+        $user_id = $user->id;
+        // データベースに追加
+        $reserve = new Reserve();
+        $reserve->user_id = $user_id;
+        $reserve->facility_id = $facility_id;
+        $reserve->start_time = $start_time;
+        $reserve->end_time = $end_time;
+        $reserve->reserve_number = $reserve_number;
+        $reserve->save();
+        \Debugbar::info();
+        $this->send($request);
+        // viewへ遷移
+        return view('reserve-complete' ,[
+                'reserve_number' => $reserve_number,
+            ]);
+    }
 ```
+## 質問に際してやったこと・作ったもの
-#### View(formの部分だけ抜粋)
+####console.log一覧
+![イメージ説明](bf9d85eeb4899a2eb321994921a1f7d8.png)
+####定義しているテーブル
+![イメージ説明](3ea1e3d5e942984d320f517afee539f5.png)
+## 追記
+####Handler.php
 ```php
-<form method="POST" id="test">
-							@csrf
-							<div class="form-group row">
-								<label for="facility_name" class="col-md-4 col-form-label text-md-right">{{ __('facility_name' )}}</label>
-								<div class="col-md-6">
-									<select class="form-control" id="facility_name" name="facility_name">
-										<option value="blank" selected="selected">選択してください</option>
-										<option value="test1">test1</option>
-										<option value="test2">test2</option>
-										<option value="test3">test4</option>
-									</select>
-								</div>
-							</div>
-							<div class="form-group row">
-								<label for="dates" class="col-md-4 col-form-label text-md-right">{{ __('dates' )}}</label>
-								<div class="col-md-6">
-									<input type="text" class="form-control dateinfo" placeholder="Select Date.." id="dateinfo" name="dateinfo" readonly="readonly">
-								</div>
-							</div>
-							<div class="form-group row">
-                            <div class="col-md-8 offset-md-4">
-                                <button type="submit" class="btn btn-primary" data-action='javascript:void(0);' id="date-select">
-                                    {{ __('submit') }}
-                                </button>
-							</div>
-							</div>
-							<div class="form-group row js-timeselectform" >
-								<label for="start_time" class="col-md-4 col-form-label text-md-right">{{ __('start_time' )}}</label>
-								<div class="col-md-6">
-									<input type="text" class="form-control time" placeholder="Select Time.." id="start_time" name="start_time" readonly="readonly">
-								</div>
-							</div>
-							<div class="form-group row js-timeselectform">
-								<label for="end_time" class="col-md-4 col-form-label text-md-right">{{ __('end_time' )}}</label>
-								<div class="col-md-6">
-									<input type="text" class="form-control time" placeholder="Select Time.." id="end_time" name="end_time" readonly="readonly">
-								</div>
-							</div>
-							<div class="form-group row js-timeselectform">
-                            	<div class="col-md-8 offset-md-4">
-                                	<button type="submit" class="btn btn-primary" data-action="{{ route('reserve.confirm') }}" id="reserve-settle">
-                                   		 {{ __('reserve-settle') }}
-                                	</button>
-								</div>
-							</div>
-						</form>
-					</div>
-				</div>
-				<!-- 予約時間重複によるエラーメッセージ表示。仮置してます。 -->
-				@if ($errors->any())
-				<ul id = "error" class="error">
-				@foreach ($errors->all() as $error)
-				<li>{{ $error }}</li>
-				@endforeach
-				</ul>
-				@endif
-(後略)
+ public function prepareResponse($request, Exception $e) {
+        // 競合違反を条件分岐
+        if($e instanceof ConflictHttpException) {
+            return $this->invaildHttpRequest($request, $e);
+        }
+        // 予約時間が競合していた場合エラーハンドリング
+        if($e instanceof ReserveDuplicationException) {
+            return redirect()->back()->withInput()->withErrors('その時間帯はすでに予約が入っています');
+        }
+        return parent::prepareResponse($request, $e);
+    }
 ```
-## 質問に際してやったこと・作ったもの
-####console.log一覧
-![イメージ説明](bf9d85eeb4899a2eb321994921a1f7d8.png)
-####定義しているテーブル
-![イメージ説明](3ea1e3d5e942984d320f517afee539f5.png)
-## 追記
-####Handler.php
-```php
- public function prepareResponse($request, Exception $e) {
-        // 競合違反を条件分岐
-        if($e instanceof ConflictHttpException) {
-            return $this->invaildHttpRequest($request, $e);
-        }
-        // 予約時間が競合していた場合エラーハンドリング
-        if($e instanceof ReserveDuplicationException) {
-            return redirect()->back()->withInput()->withErrors('その時間帯はすでに予約が入っています');
-        }
-        return parent::prepareResponse($request, $e);
-    }
-```

ControllerとViewの更新とHandler.phpの追記

2020/01/14 18:21

投稿

kamille-mio

スコア24

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -44,11 +44,15 @@
 このようなコードを書いて冒頭の画像の利用開始時刻と、利用終了時刻の選択内容をバインドしています。
-ですが、すでにおわかりかもしれませんが、私が未熟なので例えばこの場合利用開始に14:00、利用終了に18:00を選択して14:00~18:00の時間帯を予約したいというリクエストができてしまいます。
+ですが、例えばこの場合利用開始に14:00、利用終了に18:00を選択して14:00~18:00の時間帯を予約したいというリクエストができてしまいます。
 当然それでは前述の前提条件とバッティングしてしまうのでその場合、確認画面に遷移せずフォームにリダイレクトしてエラーメッセージを表示させたいのですがこの場合バリデーションでやるのか自分で独自例外を作成してtry-catchでその例外に投げるのかどちらにするのかがわからないのでお聞きしたいです。
+2020/01/10
-本来なら無効な値ということでバリデーションでどうにかするのかな？ という考えはあるのですが、上記の例の場合それぞれ利用開始・利用終了単独の値としては有効なので詰まってしまいました.......
+コメント欄でご指摘いただいた箇所を修正、かつ自分なりにエラーハンドリングをしてみたのでControllerにそれを反映。
@@ -98,7 +102,7 @@
 #### バリデーション
-とりあえず基本的なところだけです。
+とりあえず基本的なところだけ。rulesメソッドのみ抜粋。
@@ -122,53 +126,181 @@
 {
-    /**
-     * Determine if the user is authorized to make this request.
-     *
-     * @return bool
-     */
-    public function authorize() {
+    public function rules()
-    // 認証関係のバリデーションはここ。なければtrueを返す。
+    {
-        return true;
+        return [
+            'facility_name' => 'not_in:0',
+            'dateinfo' => 'required|date',
+            'start_time' => 'required|date_format:H:i',
+            'end_time' => 'required|date_format:H:i',
+        ];
     }
-    /**
-     * Get the validation rules that apply to the request.
-     *
-     * @return array
-     */
-    public function rules()
-    {
-        return [
-            'facility_name' => 'not_in:0',
-            'dateinfo' => 'required|date',
-            'start_time' => 'required|date_format:H:i',
-            'end_time' => 'required|date_format:H:i',
-        ];
+}
+```
+#### model
+```php
+<?php
+namespace App;
+use Illuminate\Database\Eloquent\Model;
+class Reserve extends Model {
+	public $incrementing = false;
+	protected $fillable = [
+		'user_id',
+		'facility_id',
+		'start_time',
+		'end_time',
+	];
+	public function user() {
+		return $this->belongsTo('App\User');
+	}
+	public function facility() {
+		return $this->belongsTo('App\Facility');
+	}
+    static function SearchReserveDates($facility_name, $dateinfo) {
+    	$result = \App\Reserve::whereHas('Facility', function($query) use($facility_name, $dateinfo) {
+		$query
+			->where('facility_name',$facility_name)
+			->whereDate('start_time',$dateinfo)
+			->whereDate('end_time',$dateinfo);
+						})->select('id', 'start_time', 'end_time')->get()->all();
+		return $result;
+	}
+}
+```
+#### Controller(自分でtry-catchを組んでみました、全体としては未完成なので抜粋してます)
+```php
+<?php
+namespace App\Http\Controllers;
+use App\Reserve;
+use App\Exceptions\ReserveDuplicationException;
+use App\Http\Requests\CreateReserveRequest;
+use Carbon\Carbon;
+use Illuminate\Http\Request;
+class ReserveController extends Controller {
+    // フォームから受け取った情報をもとにテーブルを検索、施設名から施設IDを引っ張り、日付と合わせて予約情報を取得しAjaxに返す。
+    public function searchReservation(Request $request) {
+        $data = $request->all();
+        if(isset($data['dateinfo']) && isset($data['facility_name'])) {
+        $dateinfo = $data['dateinfo'];
+        $facility_name = $data['facility_name'];
+        $reserveinfo = Reserve::SearchReserveDates($facility_name, $dateinfo);
+        \Debugbar::info();
+        return json_encode($reserveinfo, JSON_PRETTY_PRINT);
+        } else {
+       echo 'FAIL TO AJAX REQUEST';
     }
@@ -176,492 +308,382 @@
+    // 入力確認画面に最終的なフォームの値を渡す。Requestにはバリデーションの拡張クラスを渡す。
+    public function confirm(CreateReserveRequest $request) {
+        // 予約情報取得
+        $data = $request->all();
+        if(isset($data['dateinfo']) && isset($data['facility_name'])) {
+        $dateinfo = $data['dateinfo'];
+        $facility_name = $data['facility_name'];
+        $reserveinfo = Reserve::SearchReserveDates($facility_name, $dateinfo);
+    }
+        // 取得したレコードを配列の形にする。
+        // start_timeとend_timeそれぞれの値で配列を作る。
+        $arr_start_time = array_column($reserveinfo, 'start_time');
+        // var_dump($arr_start_time);
+        $arr_end_time = array_column($reserveinfo, 'end_time');
+        // var_dump($arr_end_time);
+         // 選択された時間帯が予約時間と重複していないか検証
+        if(isset($data['start_time']) && isset($data['end_time'])) {
+        // datetime型に整形
+        $start_datetime =$data['dateinfo'] .' '. $data['start_time'];
+        $end_datetime =$data['dateinfo'] .' '. $data['end_time'];
+        // Carbonに整形
+        $st = new Carbon($start_datetime);
+        $start = $st->format('Y-m-d H:i:s');
+        // var_dump($start);
+        $ed = new Carbon($end_datetime);
+        $end = $ed->format('Y-m-d H:i:s');
+        // var_dump($end);
+        // テーブルから予約情報取得して配列に格納
+        // それぞれ$arr_start_time[]と＄arr_end_times[]で呼び出せるようにする
+        // forかwhile文で$start_time[n]と$end_time[n]まで検証する
+        // $arr_start_time及び$arr_end_timeの配列の数を取得する。
+        $c1 = count($arr_start_time);
+        $c2 = count($arr_end_time);
+        // 時間帯比較の関数
+        function isTimeDuplication($start, $end, $start_time, $end_time) {
+            return ($start < $end_time && $start_time < $end);
+        }
+        // try-catchで例外が出たら処理中止する。
+        try {
+            for ($i=0; $i < $c1 && $c2 ; $i++) {
+            $start_time = $arr_start_time[$i];
+            $end_time = $arr_end_time[$i];
+            $result = isTimeDuplication($start, $end, $start_time, $end_time);
+                if($result === TRUE) {
+                    throw new ReserveDuplicationException;
+                }
+            }
+            // 入力確認ページのviewにinputsを渡す
+            return view('reserve-confirm', [
+                'data' => $data,
+            ]);
+        } catch(ReserveDuplicationException $e) {
+            throw $e;
+        }
+    }
+}
 ```
-#### model
+#### View(formの部分だけ抜粋)
 ```php
-<?php
-namespace App;
-use Illuminate\Database\Eloquent\Model;
-use Webpatser\Uuid\Uuid;
-use Katteba\UUID\UUIDShortener;
-class Reserve extends Model {
-	public $incrementing = false;
-	protected $fillable = [
-		'user_id',
-		'facility_id',
-		'start_time',
-		'end_time',
-	];
-	protected static function boot () {
-		parent::boot();
-		static::creating(function ($model) {
-			$model->{$model->getKeyName()} = Uuid::generate()->string;
-		});
-	}
-	public function user() {
-		return $this->belongsTo('App\User');
-	}
-	public function facility() {
-		return $this->belongsTo('App\Facility');
-	}
-    static function SearchReserveDates($facility_name, $dateinfo) {
-    	$result = \App\Reserve::whereHas('Facility', function($query) use($facility_name, $dateinfo) {
-		$query
-			->where('facility_name',$facility_name)
-			->whereDate('start_time',$dateinfo)
-			->whereDate('end_time',$dateinfo);
-						})->select('id', 'start_time', 'end_time')->get()->all();
-		return $result;
-	}
-}
+<form method="POST" id="test">
+							@csrf
+							<div class="form-group row">
+								<label for="facility_name" class="col-md-4 col-form-label text-md-right">{{ __('facility_name' )}}</label>
+								<div class="col-md-6">
+									<select class="form-control" id="facility_name" name="facility_name">
+										<option value="blank" selected="selected">選択してください</option>
+										<option value="test1">test1</option>
+										<option value="test2">test2</option>
+										<option value="test3">test4</option>
+									</select>
+								</div>
+							</div>
+							<div class="form-group row">
+								<label for="dates" class="col-md-4 col-form-label text-md-right">{{ __('dates' )}}</label>
+								<div class="col-md-6">
+									<input type="text" class="form-control dateinfo" placeholder="Select Date.." id="dateinfo" name="dateinfo" readonly="readonly">
+								</div>
+							</div>
+							<div class="form-group row">
+                            <div class="col-md-8 offset-md-4">
+                                <button type="submit" class="btn btn-primary" data-action='javascript:void(0);' id="date-select">
+                                    {{ __('submit') }}
+                                </button>
+							</div>
+							</div>
+							<div class="form-group row js-timeselectform" >
+								<label for="start_time" class="col-md-4 col-form-label text-md-right">{{ __('start_time' )}}</label>
+								<div class="col-md-6">
+									<input type="text" class="form-control time" placeholder="Select Time.." id="start_time" name="start_time" readonly="readonly">
+								</div>
+							</div>
+							<div class="form-group row js-timeselectform">
+								<label for="end_time" class="col-md-4 col-form-label text-md-right">{{ __('end_time' )}}</label>
+								<div class="col-md-6">
+									<input type="text" class="form-control time" placeholder="Select Time.." id="end_time" name="end_time" readonly="readonly">
+								</div>
+							</div>
+							<div class="form-group row js-timeselectform">
+                            	<div class="col-md-8 offset-md-4">
+                                	<button type="submit" class="btn btn-primary" data-action="{{ route('reserve.confirm') }}" id="reserve-settle">
+                                   		 {{ __('reserve-settle') }}
+                                	</button>
+								</div>
+							</div>
+						</form>
+					</div>
+				</div>
+				<!-- 予約時間重複によるエラーメッセージ表示。仮置してます。 -->
+				@if ($errors->any())
+				<ul id = "error" class="error">
+				@foreach ($errors->all() as $error)
+				<li>{{ $error }}</li>
+				@endforeach
+				</ul>
+				@endif
+(後略)
 ```
+## 質問に際してやったこと・作ったもの
+####console.log一覧
-#### Controller(作成途中につき、今回の処理に関係するところを抜粋)
+![イメージ説明](bf9d85eeb4899a2eb321994921a1f7d8.png)
+####定義しているテーブル
+![イメージ説明](3ea1e3d5e942984d320f517afee539f5.png)
+## 追記
+####Handler.php
 ```php
+ public function prepareResponse($request, Exception $e) {
-<?php
+        // 競合違反を条件分岐
-namespace App\Http\Controllers;
+        if($e instanceof ConflictHttpException) {
-use App\Reserve;
+            return $this->invaildHttpRequest($request, $e);
+        }
+        // 予約時間が競合していた場合エラーハンドリング
-use App\Exceptions\ReserveDuplicationException;
+        if($e instanceof ReserveDuplicationException) {
+            return redirect()->back()->withInput()->withErrors('その時間帯はすでに予約が入っています');
+        }
-use App\Http\Requests\CreateReserveRequest;
+        return parent::prepareResponse($request, $e);
-use Carbon\Carbon;
-use Illuminate\Http\Request;
-class ReserveController extends Controller {
-    // フォームから受け取った情報をもとにテーブルを検索、施設名から施設IDを引っ張り、日付と合わせて予約情報を取得しAjaxに返す。
-    public function searchReservation(Request $request) {
-        $data = $request->all();
-        if(isset($data['dateinfo']) && isset($data['facility_name'])) {
-        $dateinfo = $data['dateinfo'];
-        $facility_name = $data['facility_name'];
-        $reserveinfo = Reserve::SearchReserveDates($facility_name, $dateinfo);
-        \Debugbar::info();
-        return json_encode($reserveinfo, JSON_PRETTY_PRINT);
-        } else {
-       echo 'FAIL TO AJAX REQUEST';
     }
-}
-    // 入力確認画面に最終的なフォームの値を渡す。Requestにはバリデーションの拡張クラスを渡す。
-    public function confirm(CreateReserveRequest $request) {
-        // 予約情報取得
-        $data = $request->all();
-        if(isset($data['dateinfo']) && isset($data['facility_name'])) {
-        $dateinfo = $data['dateinfo'];
-        $facility_name = $data['facility_name'];
-        $reserveinfo = Reserve::SearchReserveDates($facility_name, $dateinfo);
-    }
-        // 取得したレコードを配列の形にする。
-        // start_timeとend_timeそれぞれの値で配列を作る。
-        $arr_start_time = array_column($reserveinfo, 'start_time');
-        // var_dump($arr_start_time);
-        $arr_end_time = array_column($reserveinfo, 'end_time');
-        // var_dump($arr_end_time);
-         // 選択された時間帯が予約時間と重複していないか検証
-        if(isset($data['start_time']) && isset($data['end_time'])) {
-        // datetime型に整形
-        $start_datetime =$data['dateinfo'] .' '. $data['start_time'];
-        $end_datetime =$data['dateinfo'] .' '. $data['end_time'];
-        // Carbonに整形
-        $st = new Carbon($start_datetime);
-        $start = $st->format('Y-m-d H:i:s');
-        // var_dump($start);
-        $ed = new Carbon($end_datetime);
-        $end = $ed->format('Y-m-d H:i:s');
-        // var_dump($end);
-        // テーブルから予約情報取得して配列に格納
-        // それぞれ$arr_start_time[]と＄arr_end_times[]で呼び出せるようにする
-        // forかwhile文で$start_time[n]と$end_time[n]まで検証する
-        // $arr_start_time及び$arr_end_timeの配列の数を取得する。
-        $c1 = count($arr_start_time);
-        $c2 = count($arr_end_time);
-        // 時間帯比較の関数
-        function isTimeDuplication($start, $end, $start_time, $end_time) {
-            return ($start < $end_time && $start_time < $end);
-        }
-        // try-catchで例外が出たら処理中止する。
-        try {
-            for ($i=0; $i < $c1 && $c2 ; $i++) {
-            $start_time = $arr_start_time[$i];
-            $end_time = $arr_end_time[$i];
-            $result = isTimeDuplication($start, $end, $start_time, $end_time);
-                if($result === TRUE) {
-                    throw new ReserveDuplicationException;
-                }
-            }
-            // 入力確認ページのviewにinputsを渡す
-            return view('reserve-confirm', [
-                'data' => $data,
-            ]);
-        } catch(ReserveDuplicationException $e) {
-            throw $e;
-        }
-    }
-}
 ```
-#### View(formの部分だけ抜粋)
-```php
-<form method="POST" id="test">
-							@csrf
-							<div class="form-group row">
-								<label for="facility_name" class="col-md-4 col-form-label text-md-right">{{ __('facility_name' )}}</label>
-								<div class="col-md-6">
-									<select class="form-control" id="facility_name" name="facility_name">
-										<option value="blank" selected="selected">選択してください</option>
-										<option value="test1">test1</option>
-										<option value="test2">test2</option>
-										<option value="test3">test4</option>
-									</select>
-								</div>
-							</div>
-							<div class="form-group row">
-								<label for="dates" class="col-md-4 col-form-label text-md-right">{{ __('dates' )}}</label>
-								<div class="col-md-6">
-									<input type="text" class="form-control dateinfo" placeholder="Select Date.." id="dateinfo" name="dateinfo" readonly="readonly">
-								</div>
-							</div>
-							<div class="form-group row">
-                            <div class="col-md-8 offset-md-4">
-                                <button type="submit" class="btn btn-primary" data-action='javascript:void(0);' id="date-select">
-                                    {{ __('submit') }}
-                                </button>
-							</div>
-							</div>
-							<div class="form-group row js-timeselectform" >
-								<label for="start_time" class="col-md-4 col-form-label text-md-right">{{ __('start_time' )}}</label>
-								<div class="col-md-6">
-									<input type="text" class="form-control time" placeholder="Select Time.." id="start_time" name="start_time" readonly="readonly">
-								</div>
-							</div>
-							<div class="form-group row js-timeselectform">
-								<label for="end_time" class="col-md-4 col-form-label text-md-right">{{ __('end_time' )}}</label>
-								<div class="col-md-6">
-									<input type="text" class="form-control time" placeholder="Select Time.." id="end_time" name="end_time" readonly="readonly">
-								</div>
-							</div>
-							<div class="form-group row js-timeselectform">
-                            	<div class="col-md-8 offset-md-4">
-                                	<button type="submit" class="btn btn-primary" data-action="{{ route('reserve.confirm') }}" id="reserve-settle">
-                                   		 {{ __('reserve-settle') }}
-                                	</button>
-								</div>
-							</div>
-						</form>
-```
-## 質問に際してやったこと・作ったもの
-####console.log一覧
-![イメージ説明](bf9d85eeb4899a2eb321994921a1f7d8.png)
-####定義しているテーブル
-![イメージ説明](3ea1e3d5e942984d320f517afee539f5.png)

指摘を受けた箇所を修正。

2020/01/09 18:29

投稿

kamille-mio

スコア24

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -296,6 +296,8 @@
 use App\Reserve;
+use App\Exceptions\ReserveDuplicationException;
 use App\Http\Requests\CreateReserveRequest;
 use Carbon\Carbon;
@@ -320,7 +322,7 @@
-        if(isset($_POST['dateinfo']) && isset($_POST['facility_name'])) {
+        if(isset($data['dateinfo']) && isset($data['facility_name'])) {
@@ -360,168 +362,156 @@
         // 予約情報取得
+        $data = $request->all();
+        if(isset($data['dateinfo']) && isset($data['facility_name'])) {
+        $dateinfo = $data['dateinfo'];
+        $facility_name = $data['facility_name'];
+        $reserveinfo = Reserve::SearchReserveDates($facility_name, $dateinfo);
+    }
+        // 取得したレコードを配列の形にする。
+        // start_timeとend_timeそれぞれの値で配列を作る。
+        $arr_start_time = array_column($reserveinfo, 'start_time');
+        // var_dump($arr_start_time);
+        $arr_end_time = array_column($reserveinfo, 'end_time');
+        // var_dump($arr_end_time);
+         // 選択された時間帯が予約時間と重複していないか検証
-        if(isset($_POST['dateinfo']) && isset($_POST['facility_name'])) {
+        if(isset($data['start_time']) && isset($data['end_time'])) {
+        // datetime型に整形
+        $start_datetime =$data['dateinfo'] .' '. $data['start_time'];
+        $end_datetime =$data['dateinfo'] .' '. $data['end_time'];
+        // Carbonに整形
+        $st = new Carbon($start_datetime);
+        $start = $st->format('Y-m-d H:i:s');
+        // var_dump($start);
+        $ed = new Carbon($end_datetime);
-        $dateinfo = $_POST['dateinfo'];
+        $end = $ed->format('Y-m-d H:i:s');
+        // var_dump($end);
+        // テーブルから予約情報取得して配列に格納
+        // それぞれ$arr_start_time[]と＄arr_end_times[]で呼び出せるようにする
+        // forかwhile文で$start_time[n]と$end_time[n]まで検証する
+        // $arr_start_time及び$arr_end_timeの配列の数を取得する。
+        $c1 = count($arr_start_time);
+        $c2 = count($arr_end_time);
+        // 時間帯比較の関数
+        function isTimeDuplication($start, $end, $start_time, $end_time) {
+            return ($start < $end_time && $start_time < $end);
+        }
+        // try-catchで例外が出たら処理中止する。
+        try {
+            for ($i=0; $i < $c1 && $c2 ; $i++) {
-        $facility_name = $_POST['facility_name'];
+            $start_time = $arr_start_time[$i];
+            $end_time = $arr_end_time[$i];
-        $reserveinfo = Reserve::SearchReserveDates($facility_name, $dateinfo);
+            $result = isTimeDuplication($start, $end, $start_time, $end_time);
+                if($result === TRUE) {
+                    throw new ReserveDuplicationException;
+                }
+            }
+            // 入力確認ページのviewにinputsを渡す
+            return view('reserve-confirm', [
+                'data' => $data,
+            ]);
+        } catch(ReserveDuplicationException $e) {
+            throw $e;
+        }
     }
-        // 取得したレコードを配列の形にする。
-        // start_timeとend_timeそれぞれの値で配列を作る。
-        $arr_start_time = array_column($reserveinfo, 'start_time');
-        var_dump($arr_start_time);
-        $arr_end_time = array_column($reserveinfo, 'end_time');
-        var_dump($arr_end_time);
-         // 選択された時間帯が予約時間と重複していないか検証
-        if(isset($_POST['start_time']) && isset($_POST['end_time'])) {
-        // datetime型に整形
-        $start_datetime =$_POST['dateinfo'] .' '. $_POST['start_time'];
-        $end_datetime =$_POST['dateinfo'] .' '. $_POST['end_time'];
-        // Carbonに整形
-        $st = new Carbon($start_datetime);
-        $start = $st->format('Y-m-d H:i:s');
-        var_dump($start);
-        $ed = new Carbon($end_datetime);
-        $end = $ed->format('Y-m-d H:i:s');
-        var_dump($end);
-        // テーブルから予約情報取得して配列に格納
-        // それぞれ$arr_start_time[]と＄arr_end_times[]で呼び出せるようにする
-        // forかwhile文で$start_time[n]と$end_time[n]まで検証する
-        // $arr_start_time及び$arr_end_timeの配列の数を取得する。
-        $c1 = count($arr_start_time);
-        $c2 = count($arr_end_time);
-        /*
-　　　　　以下の関数を用いてフォームで選択された時間と登録されている予約情報の時間とで
-　　　　　比較を行い、予約済みの時間帯と重複していた場合はエラーハンドリングを行いたい。
-　　　　*/
-        function isTimeDuplication($start, $end, $start_time, $end_time) {
-            return ($start < $end_time && $start_time < $end);
-        }
-        // 予約情報の数だけ時間帯比較を行いたいと考えて以下のようなコードを書いた。
-        for ($i=0; $i < $c1 && $c2 ; $i++) {
-            $start_time = $arr_start_time[$i];
-            $end_time = $arr_end_time[$i];
-            $result = isTimeDuplication($start, $end, $start_time, $end_time);
-            // 時間帯比較がTRUE、つまり重複した時間帯があった場合エラーメッセージ、違うなら比較をを続行する。
-            if(isTimeDuplication($start, $end, $start_time, $end_time) === TRUE) {
-                echo 'error';
-                var_dump($result);
-                break;
-            }
-        }
-        // ここまで書いてもしかしたら、try-catchしたほうがいいけどその場合例外処理どうするのだろうと詰まりました。
-        // 以下は比較でいずれも例外が認められなかった場合の処理
-        //フォームから受け取ったすべてのinputの値を取得
-        $inputs = $request->all();
-        // 確認ページのviewにinputsを渡す
-        return view('reserve-confirm', [
-            'inputs' => $inputs,
-        ]);
-    }
 }

レイアウト修正

2020/01/09 18:18

投稿

kamille-mio

スコア24

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -666,10 +666,12 @@
+####console.log一覧
-console.log一覧![イメージ説明](bf9d85eeb4899a2eb321994921a1f7d8.png)
+![イメージ説明](bf9d85eeb4899a2eb321994921a1f7d8.png)
-定義しているテーブル
+####定義しているテーブル
 ![イメージ説明](3ea1e3d5e942984d320f517afee539f5.png)

誤字修正

2020/01/06 19:32

投稿

kamille-mio

スコア24

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -58,7 +58,7 @@
 #### AjaxでPHPから帰ってくる予約情報
-ちなみにid3には2019-11-30とは別の日付の予約情報なので弾かれています。
+ちなみにid3は2019-11-30とは別の日付の予約情報なので弾かれています。
 ```