Merge pull request '駐輪場選択画面コメント修正' (#64) from main_higashide into main

Reviewed-on: #64

app/Http/Controllers/ParkDetailController.php

@@ -38,8 +38,6 @@ class ParkDetailController extends Controller
             return response()->json(['error' => 'Park not found'], 404);
         }
 
-        $park = DB::table('park')->where('park_id', $park_id)->first();
-
         // ゾーン情報取得
         $zones = $this->getZones($park_id);
 
@@ -80,7 +78,8 @@ class ParkDetailController extends Controller
     /**
      * 駐輪場情報取得（マルチテナントフィルタ適用）
      * 
-     * なぜ: 運営元に紐づく駐輪場のみ取得するため
+     * 運営元に紐づく駐輪場のみを取得することで、他の運営元のデータが
+     * 誤って表示されることを防ぐ。
      * 
      * @param int $parkId
      * @param int $managementId
@@ -100,7 +99,7 @@ class ParkDetailController extends Controller
     }
 
     /**
-     * zone情報取得
+     * ゾーン情報取得
      * 
      * @param int $parkId
      * @return \Illuminate\Support\Collection
@@ -125,7 +124,7 @@ class ParkDetailController extends Controller
     }
 
     /**
-     * reserve情報取得
+     * 予約情報取得
      * 
      * @param int $parkId
      * @return \Illuminate\Support\Collection
@@ -152,7 +151,7 @@ class ParkDetailController extends Controller
     /**
      * 車種別の基準台数集計
      * 
-     * なぜ: 駐輪場の車種別定員を把握するため
+     * 駐輪場の車種別定員（zone_standard の合計）を算出する。
      * 
      * @param \Illuminate\Support\Collection $zones
      * @return array
@@ -175,7 +174,8 @@ class ParkDetailController extends Controller
     /**
      * 車種別の空き台数計算
      * 
-     * なぜ: reserve（予約）を考慮した実際の空き台数を算出するため
+     * 予約を考慮した実際の空き台数を算出する。
+     * 許容台数から現在台数を引き、さらに有効な予約件数分を減算することで、正確な空き状況を把握する。
      * 
      * @param \Illuminate\Support\Collection $zones
      * @param \Illuminate\Support\Collection $reserves
@@ -208,8 +208,6 @@ class ParkDetailController extends Controller
     /**
      * 駐輪場の更新可能期間取得
      * 
-     * なぜ: 更新可能期間がcityテーブルからparkテーブルに変更されたため
-     * 
      * @param int $parkId
      * @return object|null
      */

app/Http/Controllers/RegularContractCreateController.php

@@ -124,7 +124,7 @@ class RegularContractCreateController extends Controller
     /**
      * 駅名一覧取得（運営元フィルタ適用）
      * 
-     * なぜ: マルチテナント対応のため、運営元に紐づく駐輪場の駅のみ表示
+     * マルチテナント対応のため、運営元に紐づく駐輪場の駅のみ表示する。
      * 
      * @param int $managementId
      * @return \Illuminate\Support\Collection
@@ -147,7 +147,7 @@ class RegularContractCreateController extends Controller
     /**
      * 駐輪場名一覧取得（運営元フィルタ適用）
      * 
-     * なぜ: マルチテナント対応のため、運営元に紐づく駐輪場のみ表示
+     * マルチテナント対応のため、運営元に紐づく駐輪場のみ表示する。
      * 
      * @param int $managementId
      * @return \Illuminate\Support\Collection
@@ -169,7 +169,7 @@ class RegularContractCreateController extends Controller
     /**
      * 駐輪場テーブルデータ取得（検索・ページング適用、運営元フィルタ適用）
      * 
-     * なぜ: マルチテナント対応のため、運営元に紐づく駐輪場のみ表示
+     * マルチテナント対応のため、運営元に紐づく駐輪場のみ表示する。
      * 
      * @param int $managementId
      * @param int|null $cityId
@@ -232,9 +232,9 @@ class RegularContractCreateController extends Controller
     }
 
     /**
-     * zone データを park_id でグループ化して取得（運営元フィルタ適用）
+     * ゾーンデータを park_id でグループ化して取得（運営元フィルタ適用）
      * 
-     * なぜ: マルチテナント対応のため、運営元に紐づく駐輪場のzoneのみ取得
+     * マルチテナント対応のため、運営元に紐づく駐輪場のゾーンのみ取得する。
      * 
      * @param int $managementId
      * @return \Illuminate\Support\Collection
@@ -264,9 +264,9 @@ class RegularContractCreateController extends Controller
     }
 
     /**
-     * reserve データを park_id でグループ化して取得（運営元フィルタ適用）
+     * 予約データを park_id でグループ化して取得（運営元フィルタ適用）
      * 
-     * なぜ: マルチテナント対応のため、運営元に紐づく駐輪場のreserveのみ取得
+     * マルチテナント対応のため、運営元に紐づく駐輪場の予約のみ取得する。
      * 
      * @param int $managementId
      * @return \Illuminate\Support\Collection
@@ -290,7 +290,6 @@ class RegularContractCreateController extends Controller
     /**
      * 駐輪場ごとの更新可能期間を取得（運営元フィルタ適用）
      * 
-     * なぜ: 更新可能期間がcityテーブルからparkテーブルに変更されたため
      * 
      * @param int $managementId
      * @return \Illuminate\Support\Collection
@@ -318,7 +317,7 @@ class RegularContractCreateController extends Controller
     /**
      * 運営元が取り扱う車種一覧を取得
      * 
-     * なぜ: マルチテナント対応のため、運営元ごとに表示する車種を可変にする
+     * マルチテナント対応のため、運営元ごとに表示する車種を可変にする。
      * 
      * @param int $managementId
      * @return array
@@ -345,8 +344,6 @@ class RegularContractCreateController extends Controller
     /**
      * 駐輪場ごと・車種ごとの状態を計算
      * 
-     * なぜ: Bladeでのビジネスロジック実行を避け、Fat Controller回避のため事前計算
-     * 
      * @param \Illuminate\Support\Collection $parks
      * @param \Illuminate\Support\Collection $zones
      * @param \Illuminate\Support\Collection $reserve
@@ -403,8 +400,6 @@ class RegularContractCreateController extends Controller
     /**
      * 猶予期間内かどうかを判定
      * 
-     * なぜ: 複雑な日時判定ロジックを分離し、可読性向上
-     * 
      * @param object|null $grace
      * @param \Carbon\Carbon $now
      * @return bool