foreach循环中回调函数结果累积问题的分析与解决
在使用foreach循环迭代数组并调用回调函数时，可能会遇到意外的结果累积。本文将分析这种问题，并提供解决方案。
问题代码示例：
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 函数由于使用了 static 变量 $res，导致每次调用时，$res 都会累积之前的结果，最终导致 foreach 循环的结果错误。
问题根源：static 变量在函数调用之间保持其值，不会在函数执行完毕后被销毁。 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;
}
此方法通过递归调用 getchildareaid 并使用 array_merge 合并结果，避免了静态变量的使用，确保每次调用都独立计算。
方法二： 在每次 foreach 循环中重新实例化或重置变量
虽然不太推荐这种方法，因为不够优雅，但为了完整性，也列举如下：
foreach ($towns as $key => $val) {
$areaidarr = getchildareaid($val['id'], []); // 传递一个空数组作为初始值
$result[$val['name']] = $worker::where('area_id', 'in', $areaidarr)->count();
}
function getchildareaid($id, $res = []) { // 添加一个默认参数
$area = new \app\common\model\Area;
$result = collection($area->where(['pid' => $id])->order('id desc')->select())->toArray();
if ($result) {
foreach ($result as $key => $val) {
$res[] = $val['id'];
$res = getchildareaid($val['id'], $res); // 递归调用并传递累积结果
}
}
return $res;
}
这种方法通过在函数中添加一个默认参数 $res，并在每次调用时传递一个新的空数组或初始化值来避免累积。
选择哪种方法取决于个人偏好和代码风格。 方法一通常被认为更清晰、更易于维护。 关键在于理解 static 变量的作用，并避免在需要独立计算的场景中使用它。
