Compare commits

..

No commits in common. "fd5b28497da1dc1dac9bdf5f51851314764ce938" and "0f683e1555cb1632a43e38179b32f01d711707bd" have entirely different histories.

2 changed files with 22 additions and 15 deletions

View File

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

View File

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