PHP循环中数组元素赋值后未生效问题求助

你遇到的问题其实是PHP数组元素赋值的常见坑——当你用$lastPeriod = end($periods);时，$lastPeriod只是原数组最后一个元素的副本，不是原元素的引用。所以你修改$lastPeriod['closeTime']时，只是修改了这个副本的值，原数组$periods里的元素根本没被触动，这就是为什么即时dd($lastPeriod)显示更新了，但最终结果里原数组没变化的原因。

和你猜测的end函数异常或者array_splice冲突没关系，核心是变量赋值的方式错了。

要修改原数组$periods的最后一个元素，你需要直接通过数组的键来访问它，而不是赋值给一个新变量。这里有两种可靠的方式：

方式1：用array_key_last()（PHP 7.3及以上推荐） #

array_key_last()可以直接获取数组最后一个元素的键，不需要移动数组内部指针，用法更安全：

for ( $i = 0; $i < sizeof($dailyHours); $i++ ) {
    $periods = &$dailyHours[$i]['periods']; // 用引用避免重复复制数组，提升性能
    $periodsCount = sizeof($periods);
    for ($j = 0; $j < $periodsCount; $j++) {
        if (empty($periods)) break; // 防止数组为空时报错
        $lastKey = array_key_last($periods);
        
        if ($periods[$lastKey]['closeTime'] === '00:00' 
            && isset($dailyHours[$i + 1]) // 先判断下一天是否存在，避免索引越界
            && $dailyHours[$i + 1]['periods'][0]['openTime'] === '00:00' 
            && $dailyHours[$i + 1]['periods'][0]['closeTime'] !== '00:00') {
            
            $nextDayFirstClose = $dailyHours[$i + 1]['periods'][0]['closeTime'];
            if (Carbon::parse($nextDayFirstClose)->isBefore(Carbon::createFromTimeString('11:59'))) {
                // 直接修改原数组的元素
                $periods[$lastKey]['closeTime'] = $nextDayFirstClose;
                
                array_splice($dailyHours[$i + 1]['periods'], 0, 1);
                if (sizeof($dailyHours[$i + 1]['periods']) < 1) {
                    $dailyHours[$i + 1]['isOpen'] = 0;
                }
            }
        }
    }
}

方式2：用end()结合key()（兼容PHP旧版本） #

如果你的PHP版本低于7.3，可以用end()移动指针后，用key()获取最后一个元素的键：

for ( $i = 0; $i < sizeof($dailyHours); $i++ ) {
    $periods = &$dailyHours[$i]['periods'];
    $periodsCount = sizeof($periods);
    for ($j = 0; $j < $periodsCount; $j++) {
        if (empty($periods)) break;
        end($periods); // 移动数组内部指针到最后一个元素
        $lastKey = key($periods); // 获取当前指针位置的键
        
        if ($periods[$lastKey]['closeTime'] === '00:00' 
            && isset($dailyHours[$i + 1])
            && $dailyHours[$i + 1]['periods'][0]['openTime'] === '00:00' 
            && $dailyHours[$i + 1]['periods'][0]['closeTime'] !== '00:00') {
            
            $nextDayFirstClose = $dailyHours[$i + 1]['periods'][0]['closeTime'];
            if (Carbon::parse($nextDayFirstClose)->isBefore(Carbon::createFromTimeString('11:59'))) {
                $periods[$lastKey]['closeTime'] = $nextDayFirstClose;
                
                array_splice($dailyHours[$i + 1]['periods'], 0, 1);
                if (sizeof($dailyHours[$i + 1]['periods']) < 1) {
                    $dailyHours[$i + 1]['isOpen'] = 0;
                }
            }
        }
    }
}

1. 新增isset($dailyHours[$i + 1])判断，防止当$i是最后一个元素时，因$dailyHours[$i + 1]不存在导致索引越界错误。
2. 将$periods赋值为引用&$dailyHours[$i]['periods']，避免每次循环都复制整个数组，提升运行效率。
3. 提前把sizeof($periods)赋值给变量$periodsCount，避免循环条件里重复计算数组长度。

内容的提问来源于stack exchange，提问作者devinthemaking