
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 appcommonmodelArea; $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 appcommonmodelArea; $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 循环中调用回调函数时,如何避免静态变量导致的结果累积问题?的详细内容,更多请关注创想鸟其它相关文章!
版权声明:本文内容由互联网用户自发贡献,该文观点仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。
如发现本站有涉嫌抄袭侵权/违法违规的内容, 请发送邮件至 chuangxiangniao@163.com 举报,一经查实,本站将立刻删除。
发布者:程序猿,转转请注明出处:https://www.chuangxiangniao.com/p/1255746.html
微信扫一扫
支付宝扫一扫