本文详解在 while 循环内使用 foreach 时因数组未重置引发的重复渲染问题，通过清空累积数组、优化查询结构和避免逻辑嵌套，实现正确层级化的复选框列表输出。
本文详解在 while 循环内使用 foreach 时因数组未重置引发的重复渲染问题，通过清空累积数组、优化查询结构和避免逻辑嵌套，实现正确层级化的复选框列表输出。
在构建多级嵌套复选框列表（如“集团下属公司 → 模块 → 功能权限”）时，常见的错误是将数据收集逻辑与输出逻辑耦合在同一个循环中，尤其当使用类似 $namearray[] = $name 这样的追加操作而未及时重置数组时，会导致后续迭代不断累积历史值，最终在页面上呈现重复项——正如问题中截图所示：第一层“Sister Concern”名称反复出现，且子模块逐轮倍增。
根本原因在于以下两处关键错误：
$namearray和$modearray在while外部声明但未重置
每次while迭代执行$namearray[] = $name，数组持续增长；随后foreach($namearray as $data)遍历的是全部历史数据，而非当前单条记录。同理，$modearray在内层while($rowmodule...)中也不断追加，导致模块名重复渲染。SQL 查询位置不当，造成 N+1 查询与逻辑错位
SELECT * FROM Module和SELECT * FROM Module_hooks被置于while($row...)内部，不仅引发性能问题（每家公司查一次所有模块），更因$modearray全局累积，使foreach($modearray as $val)实际遍历的是此前所有公司累计的模块名，彻底破坏层级对应关系。
✅ 正确做法如下：
立即学习“PHP免费学习笔记（深入）”；
① 重置临时数组，确保作用域隔离
将 $namearray 和 $modearray 声明移至 while 循环内部，并在每次迭代开始时清空（或直接初始化为空数组）：
while ($row = mysqli_fetch_array($result_concern)) {
$sisid = $row['Id'];
$name = $row['Name'];
// ✅ 每次迭代独立维护，不再累积
$namearray = [$name]; // 或直接用 $name，无需数组
echo '<li style="list-style-type:none;">';
echo '<input type="checkbox" name="sister_concern" class="chkMainConcern" id="chkMainConcern'.$sisid.'" value="'.$sisid.'"> ' . htmlspecialchars($name);
echo '</li>';
// ✅ 获取当前 concern 关联的模块（推荐关联查询或预加载，此处先修正逻辑）
$sql_module = "SELECT * FROM Module WHERE Status='Active'";
$result_module = mysqli_query($link, $sql_module);
echo '<ul class="modulename'.$sisid.'" style="display:none;">';
while ($rowmodule = mysqli_fetch_array($result_module)) {
$modid = $rowmodule['Id'];
$modname = $rowmodule['Name'];
// ✅ 当前模块独立处理，不依赖 $modearray
echo '<li style="list-style-type:none;">';
echo '<input type="checkbox" name="module['.$sisid.']" value="'.$modid.'"> ' . htmlspecialchars($modname);
echo '</li>';
// ✅ 查询该模块的 hooks（注意 SQL 注入防护！应使用预处理）
$sql_modhooks = "SELECT * FROM Module_hooks WHERE Status='Active' AND ModuleID = ?";
// 推荐改用 mysqli_prepare + bind_param，此处为简化演示：
$result_modhooks = mysqli_query($link, "SELECT * FROM Module_hooks WHERE Status='Active' AND ModuleID='$modid'");
echo '<ul class="modulehooks'.$modid.'" style="display:none;">';
while ($row_modhooks = mysqli_fetch_array($result_modhooks)) {
echo '<li style="list-style-type:none;">';
echo '<input type="checkbox" name="hook['.$modid.']" value="'.$row_modhooks['Id'].'"> ' . htmlspecialchars($row_modhooks['display_txt']);
echo '</li>';
}
echo '</ul>';
}
echo '</ul>';
}② 进阶优化建议（强烈推荐）
- 避免 N+1 查询：提前一次性获取所有 Active 模块及 hooks，用 PHP 关联映射，减少数据库往返；
-
防止 XSS：对输出到 HTML 的
$name、$modname等变量统一使用htmlspecialchars()； -
防御 SQL 注入：
ModuleID='$modid'必须改用预处理语句，杜绝字符串拼接； -
语义化命名与结构：复选框
name属性应体现层级关系（如name="hook[{$modid}][{$hook_id}]"），便于后端接收解析。
⚠️ 注意事项：
- 切勿在循环中无条件
unset($namearray)—— 若数组未定义会触发 Notice；应在循环开始时明确初始化（如$namearray = []）； -
mysqli_fetch_array()已被弃用，建议升级至mysqli_fetch_assoc()提升可读性； - 嵌套过深（三层以上）易引发维护困难，可考虑前端 JavaScript 动态展开，服务端仅提供扁平化 JSON 数据。
总结：重复渲染的本质是状态污染——变量生命周期超出单次迭代范围。牢记“每次循环即新上下文”，显式初始化、避免全局累积、分离数据获取与视图渲染，即可彻底解决此类问题。
