質問編集履歴

コントローラーを調整

2020/01/14 18:21

投稿

kamille-mio

スコア24

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -117,19 +117,10 @@
 }
 ```
-#### Controller(自分でtry-catchを組んでみました、全体としては未完成なので抜粋してます)
+#### Controller(自力で予約確定まで組んでみたものです)
 ```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に返す。
@@ -205,7 +196,7 @@
         function isTimeDuplication($start, $end, $start_time, $end_time) {
             return ($start < $end_time && $start_time < $end);
         }
-        // try-catchで例外が出たら処理中止する。
+        // try-catchで例外が出たら入力フォームまでロールバックする。
         try {
@@ -221,7 +212,7 @@
                 }
             }
-            // 入力確認ページのviewにinputsを渡す
+            // 入力確認ページのviewにdataを渡す
             return view('reserve-confirm', [
                 'data' => $data,
             ]);
@@ -234,87 +225,99 @@
     }
 }
-```
+        // 予約完了メールの発送
+        public function send(Request $request) {
-#### View(formの部分だけ抜粋)
+            //フォームから受け取ったactionの値を取得
+            $action = $request->input('action');
-```php
+            $inputs = $request->except('action');
+            //actionの値で分岐
-<form method="POST" id="test">
+            if($action !== 'submit'){
-							@csrf
+                return redirect()
-							<div class="form-group row">
+                    ->route('reserve.index')
-								<label for="facility_name" class="col-md-4 col-form-label text-md-right">{{ __('facility_name' )}}</label>
+                    ->withInput($inputs);
+            } else {
+                $user = Auth::user();
-								<div class="col-md-6">
+                $email = $user->email;
-									<select class="form-control" id="facility_name" name="facility_name">
-										<option value="blank" selected="selected">選択してください</option>
-										<option value="test1">test1</option>
+                \Mail::to($email)->send(new ContactSendmail($inputs));
-										<option value="test2">test2</option>
-										<option value="test3">test4</option>
-									</select>
-								</div>
-							</div>
+            // トークンを再発行して再送信防止
-							<div class="form-group row">
+                $request->session()->regenerateToken();
-								<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">
+    public function store(Request $request) {
-								<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>
+        $request->validate([
+             'facility_name' => 'required',
+             'dateinfo' => 'required',
+             'start_time' => 'required',
+             'end_time' => 'required',
+        ]);
-							<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)
+        $this->confirm($request);
-				<li>{{ $error }}</li>
+        // 以下、ダブルブッキングなしの場合の更新処理
-				@endforeach
-				</ul>
+        //フォームから受け取ったactionを除いたinputの値を取得
-				@endif
+        $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,
-(後略)
+            ]);
+    }
 ```
 ## 質問に際してやったこと・作ったもの
 ####console.log一覧

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

2020/01/14 18:21

投稿

kamille-mio

スコア24

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -21,10 +21,12 @@
 ![イメージ説明](f9df6d18f924b75bf8dce0a639c986a7.png)
 このようなコードを書いて冒頭の画像の利用開始時刻と、利用終了時刻の選択内容をバインドしています。
-ですが、すでにおわかりかもしれませんが、私が未熟なので例えばこの場合利用開始に14:00、利用終了に18:00を選択して14:00~18:00の時間帯を予約したいというリクエストができてしまいます。
+ですが、例えばこの場合利用開始に14:00、利用終了に18:00を選択して14:00~18:00の時間帯を予約したいというリクエストができてしまいます。
 当然それでは前述の前提条件とバッティングしてしまうのでその場合、確認画面に遷移せずフォームにリダイレクトしてエラーメッセージを表示させたいのですがこの場合バリデーションでやるのか自分で独自例外を作成してtry-catchでその例外に投げるのかどちらにするのかがわからないのでお聞きしたいです。
-本来なら無効な値ということでバリデーションでどうにかするのかな？ という考えはあるのですが、上記の例の場合それぞれ利用開始・利用終了単独の値としては有効なので詰まってしまいました.......
+2020/01/10
+コメント欄でご指摘いただいた箇所を修正、かつ自分なりにエラーハンドリングをしてみたのでControllerにそれを反映。
 以下コード他です。
 #### AjaxでPHPから帰ってくる予約情報
@@ -48,7 +50,7 @@
 ```
 #### バリデーション
-とりあえず基本的なところだけです。
+とりあえず基本的なところだけ。rulesメソッドのみ抜粋。
 ```php
 <?php
@@ -60,22 +62,7 @@
 class CreateReserveRequest extends FormRequest
 {
-    /**
-     * Determine if the user is authorized to make this request.
-     *
-     * @return bool
-     */
-    public function authorize() {
-    // 認証関係のバリデーションはここ。なければtrueを返す。
-        return true;
-    }
-    /**
-     * Get the validation rules that apply to the request.
-     *
-     * @return array
-     */
     public function rules()
     {
         return [
@@ -97,8 +84,6 @@
 namespace App;
 use Illuminate\Database\Eloquent\Model;
-use Webpatser\Uuid\Uuid;
-use Katteba\UUID\UUIDShortener;
 class Reserve extends Model {
 	public $incrementing = false;
@@ -111,14 +96,6 @@
 	];
-	protected static function boot () {
-		parent::boot();
-		static::creating(function ($model) {
-			$model->{$model->getKeyName()} = Uuid::generate()->string;
-		});
-	}
 	public function user() {
 		return $this->belongsTo('App\User');
 	}
@@ -140,7 +117,7 @@
 }
 ```
-#### Controller(作成途中につき、今回の処理に関係するところを抜粋)
+#### Controller(自分でtry-catchを組んでみました、全体としては未完成なので抜粋してます)
 ```php
 <?php
@@ -322,13 +299,47 @@
 								</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一覧
 ![イメージ説明](bf9d85eeb4899a2eb321994921a1f7d8.png)
 ####定義しているテーブル
-![イメージ説明](3ea1e3d5e942984d320f517afee539f5.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);
+    }
+```

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

2020/01/09 18:29

投稿

kamille-mio

スコア24

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -147,6 +147,7 @@
 namespace App\Http\Controllers;
 use App\Reserve;
+use App\Exceptions\ReserveDuplicationException;
 use App\Http\Requests\CreateReserveRequest;
 use Carbon\Carbon;
 use Illuminate\Http\Request;
@@ -159,7 +160,7 @@
         $data = $request->all();
-        if(isset($_POST['dateinfo']) && isset($_POST['facility_name'])) {
+        if(isset($data['dateinfo']) && isset($data['facility_name'])) {
         $dateinfo = $data['dateinfo'];
         $facility_name = $data['facility_name'];
@@ -179,39 +180,41 @@
     public function confirm(CreateReserveRequest $request) {
         // 予約情報取得
-        if(isset($_POST['dateinfo']) && isset($_POST['facility_name'])) {
-        $dateinfo = $_POST['dateinfo'];
+        $data = $request->all();
-        $facility_name = $_POST['facility_name'];
+        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);
+        // var_dump($arr_start_time);
         $arr_end_time = array_column($reserveinfo, 'end_time');
-        var_dump($arr_end_time);
+        // var_dump($arr_end_time);
          // 選択された時間帯が予約時間と重複していないか検証
-        if(isset($_POST['start_time']) && isset($_POST['end_time'])) {
+        if(isset($data['start_time']) && isset($data['end_time'])) {
         // datetime型に整形
-        $start_datetime =$_POST['dateinfo'] .' '. $_POST['start_time'];
+        $start_datetime =$data['dateinfo'] .' '. $data['start_time'];
-        $end_datetime =$_POST['dateinfo'] .' '. $_POST['end_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);
+        // var_dump($start);
         $ed = new Carbon($end_datetime);
         $end = $ed->format('Y-m-d H:i:s');
-        var_dump($end);
+        // var_dump($end);
         // テーブルから予約情報取得して配列に格納
         // それぞれ$arr_start_time[]と＄arr_end_times[]で呼び出せるようにする
         // forかwhile文で$start_time[n]と$end_time[n]まで検証する
@@ -220,46 +223,38 @@
         $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++) {
+            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);
+                if($result === TRUE) {
-                break;
+                    throw new ReserveDuplicationException;
+                }
             }
+            // 入力確認ページのviewにinputsを渡す
+            return view('reserve-confirm', [
+                'data' => $data,
+            ]);
+        } catch(ReserveDuplicationException $e) {
+            throw $e;
         }
-        // ここまで書いてもしかしたら、try-catchしたほうがいいけどその場合例外処理どうするのだろうと詰まりました。
-        // 以下は比較でいずれも例外が認められなかった場合の処理
-        //フォームから受け取ったすべてのinputの値を取得
-        $inputs = $request->all();
-        // 確認ページのviewにinputsを渡す
-        return view('reserve-confirm', [
-            'inputs' => $inputs,
-        ]);
     }
 }
 ```

レイアウト修正

2020/01/09 18:18

投稿

kamille-mio

スコア24

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -332,7 +332,8 @@
 ## 質問に際してやったこと・作ったもの
+####console.log一覧
-console.log一覧![イメージ説明](bf9d85eeb4899a2eb321994921a1f7d8.png)
+![イメージ説明](bf9d85eeb4899a2eb321994921a1f7d8.png)
-定義しているテーブル
+####定義しているテーブル
 ![イメージ説明](3ea1e3d5e942984d320f517afee539f5.png)

誤字修正

2020/01/06 19:32

投稿

kamille-mio

スコア24

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -28,7 +28,7 @@
 以下コード他です。
 #### AjaxでPHPから帰ってくる予約情報
-ちなみにid3には2019-11-30とは別の日付の予約情報なので弾かれています。
+ちなみにid3は2019-11-30とは別の日付の予約情報なので弾かれています。
 ```
  {
         "id": "1",