From 92f48ecc0d219f1b329d8b473b63879fc46227b3 Mon Sep 17 00:00:00 2001
From: Yuka Higashide <higashide@s-force.co.jp>
Date: Fri, 30 Jan 2026 19:07:42 +0900
Subject: [PATCH] =?UTF-8?q?=E9=A7=90=E8=BC=AA=E5=A0=B4=E9=81=B8=E6=8A=9E?=
 =?UTF-8?q?=E7=94=BB=E9=9D=A2=E3=82=B3=E3=83=A1=E3=83=B3=E3=83=88=E4=BF=AE?=
 =?UTF-8?q?=E6=AD=A3?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/Http/Controllers/ParkDetailController.php | 16 +++++++-------
 .../RegularContractCreateController.php       | 21 +++++++------------
 2 files changed, 15 insertions(+), 22 deletions(-)

diff --git a/app/Http/Controllers/ParkDetailController.php b/app/Http/Controllers/ParkDetailController.php
index 197f14f..e706681 100644
--- a/app/Http/Controllers/ParkDetailController.php
+++ b/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
      */
diff --git a/app/Http/Controllers/RegularContractCreateController.php b/app/Http/Controllers/RegularContractCreateController.php
index 24ca40f..c1e21fa 100644
--- a/app/Http/Controllers/RegularContractCreateController.php
+++ b/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
-- 
2.47.3