PHP单页while循环生成多表单 POST提交参数获取问题

你现在拿不到参数是原代码有3个基础写法错误，和是否用用户ID区分name属性无关：

• <select>标签本身没有加name属性，你把name写在内部的<option>标签上是无效的：表单提交只会识别带name属性的表单控件，下拉框的参数名必须定义在<select>上，选中对应选项时会自动把选中项的value作为参数值提交。
• 你把用户ID存在了显示昵称的普通按钮value里，这个按钮不是提交类型，点击时不会触发表单提交，它的value也不会被带入POST请求。
• 你在循环用户列表的内部重复查询角色表，会产生N+1查询问题，用户量上来后会严重拖慢页面性能。

注意：你现在是每个用户对应一个独立的<form>标签，提交时只会上传当前点击按钮所在表单内的参数，根本不需要给控件name加用户ID后缀做区分，完全没必要。

前端表单部分 #

先把角色列表提前查询一次复用，不要在循环里查库，修正表单控件的属性：

<?php
// 提前查询全量角色列表，循环内直接复用，避免重复查库
$getListRole = $bdd->prepare('SELECT * FROM role');
$getListRole->execute();
$roleList = $getListRole->fetchAll();

while($user = $getUser->fetch()){
?>
<form method="post" action="">
  <p>Pseudo: <?php echo htmlspecialchars($user["pseudo"]); ?></p>
  <!-- 用隐藏域存储当前行用户ID，提交时会自动携带 -->
  <input type="hidden" name="user_id" value="<?php echo $user["id_user"]; ?>">
  
  <!-- select标签必须加name属性，增加必填校验防止空提交 -->
  <select class="form-select" name="role_id" aria-label="Default" required>
    <option selected disabled value="">Choose</option>
    <?php foreach($roleList as $role){ ?>
      <option value="<?php echo $role["id_role"]; ?>">
        <?php echo htmlspecialchars($role["nom"]); ?>
      </option>
    <?php } ?>
  </select>

  <button type="submit" name="addRole" value="1">Add role</button>
</form>
<?php } ?>

POST请求处理部分 #

提交后直接读取当前表单内的参数即可，不需要做额外的遍历匹配：

<?php
if(isset($_POST['addRole'])){
  // 读取参数时转成整数，避免SQL注入风险
  $userId = (int)$_POST['user_id'];
  $roleId = (int)$_POST['role_id'];

  // 此处可增加业务校验：比如判断用户是否存在、角色是否存在、是否已经绑定过该角色
  // 校验通过后执行插入逻辑
  // $insertBind = $bdd->prepare("INSERT INTO user_role (id_user, id_role) VALUES (?, ?)");
  // $insertBind->execute([$userId, $roleId]);

  // 处理完成后跳转，防止刷新页面重复提交表单
  header("Location: ".$_SERVER['PHP_SELF']);
  exit;
}
?>

• 输出数据库存储的内容到HTML时，必须用htmlspecialchars()做转义，避免XSS注入风险。
• 如果你后续改成整页一个大表单、批量提交多用户的场景，再用数组格式命名name即可，比如<input name="user[<?= $user['id_user'] ?>][role_id]">，处理时遍历$_POST['user']就能拿到所有用户的提交值，当前单表单单用户提交的场景不需要这么写。
• 下拉框默认选项加disabled属性和空value，配合required属性，可以避免用户未选择角色就提交空值。

内容的提问来源于stack exchange，提问作者Akael