PHP+MySQL多表关联下拉选多值提交插入异常排查

你的代码无法运行是多个明确错误叠加导致的：

• 关联查询字段名不匹配：viaturas表存储燃料类型的外键字段是idtipocomb，combustivel表的主键是idcombustivel，你写的JOIN条件viaturas.idcomb = combustivel.idcomb中两个字段在对应表中都不存在，查询直接返回空或报错，下拉框无法加载正确选项。
• 表单缺少必要配置：你没有给<form>标签加method="post"属性，默认用GET方式提交，后端$_POST根本接收不到数据；同时未配置提交按钮的name="submit"属性，后端写的isset($_POST['submit'])判断永远不成立；另外<select>标签未写闭合标签，HTML结构非法。
• PDO操作顺序完全颠倒：insert.php中你先调用$statement->bindValue()，后面才执行prepare()初始化$statement对象，直接触发调用空对象方法的致命错误；同时你既手动调用bindValue绑定参数，又给execute传参数字段，属于重复操作；且INSERT是写入操作，不需要调用fetchAll()拉取结果集，这个方法仅用于SELECT查询场景。
• 潜在逻辑风险：用|拼接两个字段值放在option的value中，如果字段值本身包含|字符，explode拆分时会出现数据错乱。更稳妥的方案是传递车辆主键ID，提交时再关联查询对应字段值，还能避免前端篡改传参。

form.php #

<?php
$con = mysqli_connect("localhost","root","","xxxx");
// 修正关联字段和表结构对应
$sql = "SELECT v.idviatura, v.matricula, c.tipocomb 
        FROM viaturas v
        INNER JOIN combustivel c ON v.idtipocomb = c.idcombustivel";  
$result = mysqli_query($con, $sql);  
?>
<!-- 必须指定post提交方式，action指向处理脚本 -->
<form method="post" action="insert.php">
    <select name="car_id">
    <?php 
    while ($row = mysqli_fetch_assoc($result)): 
    ?>
        <!-- 传车辆主键ID，避免拼接值的风险 -->
        <option value="<?php echo $row['idviatura'];?>"><?php echo $row['matricula'];?></option>  
    <?php endwhile; ?>
    </select>
    <!-- 提交按钮必须加name="submit"和后端判断对应 -->
    <button type="submit" name="submit">提交</button>
</form>

insert.php #

<?php
if(isset($_POST['submit']))
{
    include('database_connection.php');
    $carId = intval($_POST['car_id']);
    // 查库获取对应车辆信息，校验参数合法性同时拿到需要的字段
    $stmt = $connect->prepare("SELECT v.matricula, c.tipocomb FROM viaturas v INNER JOIN combustivel c ON v.idtipocomb = c.idcombustivel WHERE v.idviatura = ?");
    $stmt->execute([$carId]);
    $carData = $stmt->fetch(PDO::FETCH_ASSOC);
    if(!$carData){
        die("所选车辆信息无效");
    }
    // 注意操作顺序：先prepare初始化语句对象，再传参执行
    $query = "INSERT INTO tabledata (matricula, tipocomb) VALUES(:matricula, :tipocomb)";
    $statement = $connect->prepare($query);
    $executeRes = $statement->execute([
        ':matricula' => $carData['matricula'],
        ':tipocomb' => $carData['tipocomb'],
    ]);
    // 写入操作直接判断execute返回值即可，不需要调用fetchAll
    if($executeRes)
    {
        echo 'done';
    }
}
?>

如果你坚持要用拼接值的方式传参，只需要把option的value改回matricula|tipocomb的格式，在insert.php中explode后先校验拆分出的字段数量是否为2再做后续处理即可，但传递主键ID的方案安全性、稳定性都更高。

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