
本文介绍如何用清晰、可维护、无年份边界错误的方式重构多季节价格计算逻辑,通过日期归一化、季节映射表和职责分离设计,彻底替代硬编码字符串比较的脆弱实现。
在租车预订系统中,按日期区间动态应用不同季节价格(如低/中/高/峰值季)是核心业务逻辑。原始代码存在多个严重缺陷:使用 d-m-Y 字符串比较导致跨年失效(如 11月→次年3月)、未处理闰年与月份天数差异、条件嵌套混乱且不可测试,更致命的是将“16/7”这类非标准格式直接用于字符串比对——PHP 会将其解析为浮点数 2.2857...,造成逻辑完全错乱。
✅ 正确设计原则
- 日期标准化:统一使用 Carbon 实例进行比较,避免字符串解析歧义;
- 季节定义解耦:将季节规则抽象为配置数组,支持动态扩展与单元测试;
- 单日定价职责分离:season() 方法只负责判断单日所属季节,calculate() 聚焦价格累加,符合单一职责原则;
- 跨年场景健壮处理:不依赖 $startYear / $endYear 拼接,而是对每个日期独立判断其在当年日历中的位置(因季节定义本身是年度循环模式,无需跨年逻辑)。
?️ 重构后核心实现(Laravel + Carbon)
use Carbon\Carbon;
private function season(Carbon $date): string
{
// 定义各季节起始日与持续天数(基于当年日历)
$seasons = [
'peak' => [[7, 16, 30]], // 7月16日 → 8月14日(含)共30天
'high' => [[7, 1, 14], [8, 16, 45]], // 7月1–14日;8月16日–9月29日(45天)
'medium' => [[4, 1, 90], [10, 1, 30]], // 4月1–30日(90天→6月29日);10月1–30日
];
$year = $date->year; // 使用日期实际年份,确保闰年等正确
foreach ($seasons as $name => $periods) {
foreach ($periods as [$month, $day, $duration]) {
$start = Carbon::createFromDate($year, $month, $day);
$end = $start->copy()->addDays($duration - 1); // 含首尾日,故减1
if ($date->betweenIncluded($start, $end)) {
return $name;
}
}
}
return 'low'; // 默认低季节(11月1日–次年3月31日)
}
private function calculatePriceForDate(
string $season,
$group,
array &$totalGroupPrices,
array &$totalGroupPricesWithInsurance
): void {
$priceKey = "{$season}SeasonPrice";
$priceWithInsKey = "{$season}SeasonPriceWithInsurance";
$totalGroupPrices[$group->id] = ($totalGroupPrices[$group->id] ?? 0) + $group->$priceKey;
$totalGroupPricesWithInsurance[$group->id] = ($totalGroupPricesWithInsurance[$group->id] ?? 0) + $group->$priceWithInsKey;
}
// 主调用逻辑(精简版)
public function calculateTotalPrices(\DateTimeInterface $startDate, \DateTimeInterface $endDate, $groupPrices)
{
$begin = Carbon::parse($startDate);
$end = Carbon::parse($endDate)->endOfDay(); // 确保包含结束日
$daterange = CarbonPeriod::create($begin, $end);
$totalGroupPrices = [];
$totalGroupPricesWithInsurance = [];
foreach ($groupPrices as $group) {
foreach ($daterange as $date) {
$season = $this->season($date);
$this->calculatePriceForDate($season, $group, $totalGroupPrices, $totalGroupPricesWithInsurance);
}
}
return [
'prices' => $totalGroupPrices,
'prices_with_insurance' => $totalGroupPricesWithInsurance,
];
}⚠️ 关键注意事项
- 不要拼接年份字符串:原始代码中 "1/11/" . $startYear 在跨年预订(如 2024-12-01 至 2025-02-15)时会导致 1/11/2024 与 31/3/2025 的字符串比较失效("31/3/2025"
- 使用 CarbonPeriod 替代 DatePeriod:更语义化,自动处理时区与边界;
- 季节范围需人工校验:例如 High Season 中 "16th of August - 30th of September" 是 45 天(8月16日→9月30日),代码中应设为 [8,16,45] 并用 addDays(44) 得到正确终点;
- 数据库字段命名建议:统一为 low_season_price, medium_season_price 等 snake_case,避免动态属性访问风险;
- 性能优化(可选):若日期跨度极大(>365天),可先按年分组计算,再聚合,避免逐日循环。
该方案将季节逻辑从 50+ 行易错条件语句压缩为 20 行可读、可测、可配置的代码,大幅提升可维护性与业务适应性——后续新增“节日加价”或调整季节时间,仅需修改 $seasons 数组即可。










