在codeigniter中视图层执行数据库查询是严重架构违规，会导致n+1查询、xss风险、无法单元测试及维护困难；正确做法是控制器预组装树形数据并传入视图，视图仅负责用esc()安全渲染和递归调用局部视图。
在CodeIgniter项目中直接拼接HTML与PHP逻辑导致视图臃肿、难以复用、调试困难，甚至出现数据库查询混入模板的严重架构违规——这不是小问题，而是MVC分层崩塌的明确信号。
为什么不能在视图里写数据库查询
视图文件（如views/menu.php）中出现$this->db->query(...)或\Config\Database::connect()，意味着你正在绕过控制器和模型，把数据获取责任强行塞进展示层。
这会导致N+1查询：循环渲染主菜单时，每次迭代都执行一次子菜单查询，10个主项就触发10次独立数据库请求，响应时间陡增。
【CI4中视图层执行DB操作会绕过Query Builder的防注入机制，且无法被单元测试覆盖】
更关键的是，这种写法让视图彻底失去可移植性——换数据库驱动、改表结构、加缓存层，你得逐个翻查所有.php视图文件去修SQL。
正确传递嵌套数据：控制器预组装数组
第一步：在控制器中一次性查出全部菜单关系数据
第二步：用PHP原生数组构建树形结构，而非依赖视图层循环嵌套查询
第三步：将组装好的$menu_tree数组直接传入$this->view('menu', ['menu_tree' => $menu_tree])
这一步操作起来很简单，直接把组装好的数组丢进view方法第二个参数就行，不用改视图里的任何逻辑。
视图层精简写法：纯渲染，零逻辑
方法一：使用foreach递归渲染（适用于深度≤3的菜单）
<?php foreach ($menu_tree as $item): ?>
<li>= esc($item['name']) ?></li>
<?php if (!empty($item['children'])): ?>
<ul>
<?php $this->load->view('partials/submenu', ['items' => $item['children']]) ?></ul>
<?php endif; ?>
<?php endforeach; ?>
方法二：用CodeIgniter内置的esc()函数包裹所有输出变量，避免XSS漏洞——这是强制要求，不是可选项。
方法三：把重复渲染逻辑抽成局部视图（如views/partials/submenu.php），通过$this->load->view()引入，实现结构复用。
禁用视图自动加载数据库类
打开app/Config/View.php，确认$render_params未开启'database'等危险参数自动注入。
检查app/Config/Boot/production.php中是否残留$autoload['libraries'] = ['database'];——若存在，必须删除，否则视图可能意外获得DB实例。
【一旦视图能访问$this->db，就等于给MVC架构开了后门，后续所有代码审查都会失效】
