foreach 循环与回调函数的静态变量累积问题
在使用 foreach 循环调用递归回调函数 getchildareaid 时，由于函数内部使用了静态变量 $res，导致每次循环的结果累积。这是因为静态变量在函数调用之间保持其值，而非每次调用都重新初始化。
以下代码展示了问题所在：
foreach ($towns as $key=>$val){
$areaidarr = getchildareaid($val['id']);
$result[$val['name']] = $worker::where('area_id','in',$areaidarr)->count();
}
function getChildAreaId($id){
static $area;
$area = $area ?? new \app\common\model\Area;
$result = collection($area->where(['pid'=>$id])->order('id desc')->select())->toArray();
static $res = []; // 问题所在：静态变量导致结果累积
if($result){
foreach ($result as $key=>$val) {
$res[] = $val['id'];
getChildAreaId($val['id']);
}
}
return $res;
}
解决方案：避免使用静态变量
为了解决这个问题，我们需要移除 getChildAreaId 函数中的静态变量 $res。可以使用以下方法：
方法一：修改 getChildAreaId 函数，使其返回一个新数组
function getChildAreaId($id){
$area = new \app\common\model\Area;
$result = collection($area->where(['pid'=>$id])->order('id desc')->select())->toArray();
$res = []; // 初始化一个新的空数组
if($result){
foreach ($result as $key=>$val) {
$res[] = $val['id'];
$res = array_merge($res, getChildAreaId($val['id'])); // 递归调用并合并结果
}
}
return $res;
}
此方法在每次函数调用时创建一个新的空数组 $res，并使用 array_merge 合并递归调用的结果，避免了静态变量的累积。
方法二：使用非递归方式获取子级区域ID
如果数据库结构允许，建议使用更有效率的非递归方式获取所有子级区域ID。例如，可以使用数据库查询中的 CTE (Common Table Expression) 或自连接来实现。 这将避免递归调用的开销，并直接解决静态变量的问题。 具体实现取决于你的数据库系统。 例如，在MySQL中，可以使用以下类似的查询（假设你的表名为area，主键为id，父ID字段为pid）：
WITH RECURSIVE ChildAreas AS (
SELECT id, pid FROM area WHERE pid = ? -- 替换 ? 为父级ID
UNION ALL
SELECT a.id, a.pid FROM area a
INNER JOIN ChildAreas ca ON a.pid = ca.id
)
SELECT id FROM ChildAreas;
这个查询会递归地查找所有子级区域的ID，并返回一个结果集，直接避免了PHP端的递归和静态变量问题。
选择哪种方法取决于你的数据库系统和性能需求。 非递归的数据库查询通常效率更高，尤其是在处理大量数据时。 如果数据库查询无法实现，则方法一提供了对原代码的有效修改。 记住在修改后测试你的代码以确保其正确性。
