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 ppcommonmodelArea; $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 ppcommonmodelArea; $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端的递归和静态变量问题。
选择哪种方法取决于你的数据库系统和性能需求。 非递归的数据库查询通常效率更高,尤其是在处理大量数据时。 如果数据库查询无法实现,则方法一提供了对原代码的有效修改。 记住在修改后测试你的代码以确保其正确性。
以上就是在 Foreach 循环中调用回调函数时,如何避免静态变量导致的结果累积问题?的详细内容,更多请关注资源网其它相关文章!
发表评论:
◎欢迎参与讨论,请在这里发表您的看法、交流您的观点。