本文讲解在 yii2 框架中，如何根据用户所属的多个团队及其各自的 `onduty` 状态，正确循环生成多个独立按钮（每个按钮对应一个未值班的团队），避免因变量作用域错误导致仅渲染最后一个或漏渲染的问题。
在 Yii2 应用中，当用户可归属于多个团队，且需为每个“未处于值班状态（onduty = false）”的团队单独渲染一个操作按钮时，常见的逻辑错误是将数据库查询变量（如 $userAndTeam）移出内层循环，导致其始终只绑定到最后一次迭代的 $teamId，最终仅生成一个按钮，甚至按钮关联的是错误的团队。
正确的做法是：确保团队 ID 与对应的关系记录查询严格绑定在同一层级循环内。关键修复点在于将 UserAndTeams::find() 查询语句移入 foreach ($userTeams as $teams) 循环内部，使每次迭代都基于当前 $teams->idteam 获取该团队下用户的值班状态。
以下是优化后的完整代码示例（含安全增强与可读性改进）：
<?php
use yii\helpers\Html;
// 获取当前登录用户（注意：User::find()->where(...)->all() 返回数组，但通常单用户只需 one()）
$user = User::findOne(Yii::$app->user->identity->iduser);
if (!$user) {
return;
}
// 遍历用户所属的所有团队
foreach ($user->teamIdteams as $teams) {
$teamId = $teams->idteam;
// ✅ 关键修复：在此处即时查询该团队下的用户-团队关系记录
$userAndTeam = UserAndTeams::find()
->where(['userid' => Yii::$app->user->identity->iduser])
->andWhere(['teamid' => $teamId])
->one(); // 使用 one() 更合理：一个用户在一个团队中只应有一条关系记录
// 仅当该团队中用户未值班时，渲染激活按钮
if ($userAndTeam && $userAndTeam->onduty == false) {
echo Html::a(
'<span class="btn-label">' . Html::encode($teams->name) . '</span>',
['activate', 'id' => Yii::$app->user->identity->iduser, 'team' => $teamId],
[
'class' => 'btn btn-primary btn-sm',
'role' => 'button'
]
);
}
}
?>✅ 注意事项与最佳实践：
- 避免 all() + 多余循环：UserAndTeams 表中，一个用户对一个团队通常仅有唯一关联记录，使用 ->one() 替代 ->all() 并省略外层 foreach，提升性能与健壮性；
- 防止 XSS：使用 Html::encode($teams->name) 对团队名称做输出转义；
- 空值防御：添加 $userAndTeam && 判断，避免因数据不一致导致 PHP 警告；
- 语义清晰：条件判断推荐使用 $userAndTeam->onduty == false 或更明确的 !$userAndTeam->onduty（假设字段为布尔型），比 !== true 更直观；
- CSS 与可访问性：为按钮添加 role="button" 和合适尺寸类（如 btn-sm），兼顾响应式与无障碍支持。
通过以上重构，系统将为用户每一个“非值班状态”的所属团队准确生成一个独立按钮，URL 参数 team 值与团队名称严格对应，彻底解决“只显示一个错误按钮”的问题。
