PHP MySQL多选下拉菜单数据无法保存至数据库问题求助

看起来你的代码里有几个关键问题导致选中的房间数据没能正确插入数据库，咱们一步步来排查和修复：

问题1：POST参数检查逻辑错误 #

你当前用isset($_POST['room[]'])判断是否有提交的房间数据，但PHP处理表单里的name="room[]"时，会自动解析为$_POST['room']数组，而不是room[]作为键名。这就导致你的插入逻辑直接被跳过了，根本没执行。

修复：
把判断条件改成if(isset($_POST['room']) && is_array($_POST['room']) && !empty($_POST['room']))，既检查参数存在，也确保是有效数组。

问题2：未正确提取房间ID #

mysqli_query($conn, "SELECT master_room_id FROM master_rooms WHERE master_room_name = '$r'")返回的是MySQL结果集对象，不是直接的ID值。你直接把这个对象插入数据库，只会存进去类似mysqli_result Object的无效字符串，肯定不对。

修复：
执行查询后，要从结果集中取出实际的ID值：

$result = mysqli_query($conn, "SELECT master_room_id FROM master_rooms WHERE master_room_name = '$r'");
if($row = mysqli_fetch_assoc($result)) {
    $room_id = $row['master_room_id'];
    // 再执行插入操作
}

问题3：SQL语句的空格语法错误 #

你的插入SQL里VALUES(' ".$u_id." ', ' " .$room_id. " ') 有多余的空格，会让参数变成带空格的字符串（比如' 123 '），可能和数据库字段类型不匹配，导致插入失败。

修复：
去掉多余空格，简化写法：

$sql = "INSERT INTO user_room_list(user_info_id, master_room_id) VALUES('$u_id', '$room_id')";

问题4：HTML选项值未加引号 #

如果房间名称包含空格（比如"Living Room"），你的<option value=". $row[$i] . ">"会生成<option value=Living Room>，这会导致value只识别到Living，后面的Room被当成无效属性，提交的房间名称就错了。

修复：
给value属性加上单引号：

echo "<option value='". $row[$i] . "'>" . $row[$i] ."</option>";

修正后的完整PHP代码（insert_rooms.inc.php） #

<?php 
session_start(); 
require_once '../dbconfig.php'; 

// 保留日志逻辑
$date = new DateTime(); 
$date = $date->format("y:m:d h:i:s"); 
$id = $_SESSION['u_nickname']; 
$u_id = $_SESSION['u_id']; 
$file = "../ips.txt"; 
$text = file_get_contents($file); 
$text .= $date ." " . $id . " visited insert_rooms.inc.php"."\n"; 
file_put_contents($file, $text); 

$conn = OpenCon(); 
if ($conn) { 
    // 修正POST参数检查
    if(isset($_POST['room']) && is_array($_POST['room']) && !empty($_POST['room'])) { 
        foreach($_POST['room'] as $r) { 
            $r = mysqli_real_escape_string($conn, $r); // 转义每个房间名称，避免SQL注入
            // 获取对应的房间ID
            $result = mysqli_query($conn, "SELECT master_room_id FROM master_rooms WHERE master_room_name = '$r'");
            if($row = mysqli_fetch_assoc($result)) {
                $room_id = $row['master_room_id'];
                // 修正后的插入SQL
                $sql = "INSERT INTO user_room_list(user_info_id, master_room_id) VALUES('$u_id', '$room_id')"; 
                // 执行并检查错误（调试阶段可以保留错误输出）
                if(!mysqli_query($conn, $sql)) {
                    error_log("插入失败: " . mysqli_error($conn));
                }
            } else {
                error_log("未找到对应房间: " . $r);
            }
        } 
    } 
    header("Location: ../room_choice.php"); 
    exit(); 
    CloseCon($conn); 
} 
?>

修正后的HTML代码片段 #

<div class="dropdown"> 
    <form action="includes/insert_rooms.inc.php" method="POST"> 
        "Hold down the Ctrl (windows) / Command (Mac) button to select multiple options." <br/> <br/> 
        <select name="room[]" multiple> 
            <?php 
            $conn = OpenCon(); 
            if ($conn) { 
                $query = "SELECT master_room_name FROM master_rooms"; 
                $result = mysqli_query($conn, $query) or die('Error querying database.'); 
                while ($row = mysqli_fetch_array($result, MYSQLI_NUM)) { 
                    for ($i = 0; $i < count($row); $i++) { 
                        // 给value属性添加单引号
                        echo "<option value='". $row[$i] . "'>" . $row[$i] ."</option>"; 
                    } 
                } 
                CloseCon($conn); 
            } 
            ?> 
        </select> 
        <input type="submit" value="Add" name="send"> 
    </form> 
</div>

额外建议 #

1. 调试阶段可以暂时注释掉header跳转语句，添加echo mysqli_error($conn)来查看具体的SQL错误信息，更快定位问题。
2. 尽量使用预处理语句（Prepared Statements）替代mysqli_real_escape_string，能更彻底地避免SQL注入风险。

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