PHP递归函数listRedCars无法返回数组问题求助

我来帮你排查这个递归函数的问题，以及给出修复方案：

核心问题：递归调用未传递返回值 #

你的listRedCars()函数出现无输出的根本原因是：递归调用时没有将下层递归的返回值传递回上层。

当$index < sizeof($this->cars)时，你调用了$this->listRedCars($index + 1)，但没有把这个调用的结果return出去。只有最内层的递归（当$index达到数组长度时）会返回$this->red_cars，但外层的递归调用并没有把这个结果向上传递，最终顶层调用print_r($my_cars->listRedCars())拿到的是null，所以没有输出。

另外还有个小坑：PHP数组索引从0开始，你默认$index = 1会直接跳过第一个元素（索引0的汽车），这也会导致结果不准确。

快速修复方案 #

修改listRedCars()函数，补充递归调用的返回语句，并修正初始索引：

public function listRedCars(int $index = 0) {
    if ($index < sizeof($this->cars)) {
        if ($this->cars[$index]->color == "Red") {
            $this->red_cars[] = $this->cars[$index];
        }
        // 关键：将递归调用的结果返回给上层
        return $this->listRedCars($index + 1);
    } else {
        return $this->red_cars;
    }
}

这样修改后，顶层调用就能正确接收到递归最终返回的红色汽车数组了。

更优的递归写法（避免状态污染） #

当前写法依赖类属性$red_cars存储中间结果，如果你多次调用listRedCars()，这个属性会累积之前的结果，导致数据错误。推荐把结果数组作为递归参数传递，让每次调用都是独立无状态的：

class Cars extends Controller {
    public $cars = array();
    // 移除$red_cars类属性

    // ... 其他方法保持不变 ...

    public function listRedCars(int $index = 0, array $redCars = []) {
        if ($index < sizeof($this->cars)) {
            if ($this->cars[$index]->color == "Red") {
                // 将符合条件的汽车添加到临时数组中
                $redCars[] = $this->cars[$index];
            }
            // 传递更新后的数组进行下一层递归
            return $this->listRedCars($index + 1, $redCars);
        } else {
            return $redCars;
        }
    }
}

这种写法不需要依赖类属性存储中间值，每次调用函数都会重新计算，避免了多次调用时的状态残留问题，更符合递归函数的设计原则。

为什么函数内的print_r($this->red_cars)能正常显示？
因为当递归执行到最后一层时，$this->red_cars已经被所有符合条件的元素填充完成，所以内部打印能看到结果，但由于上层递归没有返回这个值，外部调用无法获取到它。

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