Simple Admin Panel更新异常：点击首行更新按钮却修改第三行数据

嘿，这个问题我太熟悉啦！你遇到的核心问题是所有行都共用了同一个表单，且表单元素的name属性重复，导致提交时浏览器只会传递最后一组同名元素的值，所以不管点哪一行的按钮，服务器拿到的都是最后一行的ID和Content，自然就更新错行了。

问题根源拆解 #

你把<form>标签放在了while循环外面，循环里的每一行都生成了同名的txtid、txtarea和button元素。HTML里同名的表单元素，提交时只会保留最后一个的值，这就是为什么点击第一行按钮却执行第三行更新的原因。

另外还有两个重要的坑需要注意：

• mysql_*系列函数已经被PHP官方废弃（PHP 5.5起废弃，PHP7起彻底移除），继续使用会导致兼容性和安全问题，建议替换成mysqli_*或PDO。
• 直接把用户输入拼进SQL语句会导致SQL注入攻击，这是非常危险的操作，必须对用户输入进行转义或使用预处理语句。

修复后的代码示例 #

我帮你把代码改成了每个行独立表单，同时替换成了mysqli扩展，还做了SQL注入防护：

<?php
include 'config.php';
// 假设你的config.php里已经用mysqli_connect连接了数据库，$conn是连接对象

echo '<link rel="stylesheet" href="style.css" type="text/css">';

// 使用mysqli_query替代mysql_query
$result = mysqli_query($conn, "SELECT * FROM English");

// 循环生成每一行的独立表单
while($row = mysqli_fetch_array($result)) {
    // 把<form>放到循环内，每个行一个独立表单
    echo "<form action='' method='post'>";
    echo "<table>";
    echo "<tr>";
    echo "<td><textarea rows='1' cols='1' name='txtid' readonly style='overflow:auto;resize:none'>" . htmlspecialchars($row['ID']) . "</textarea></td>";
    echo "<td><textarea rows='4' cols='50' name='txtarea'>" . htmlspecialchars($row['Content']) . "</textarea></td>";
    echo "<td><input type='submit' name='button' value='Update!'/></td>";
    echo "</tr>";
    echo "</table>";
    echo "</form>"; // 每个表单单独闭合
}

if(isset($_POST['button'])){
    // 转义用户输入，防止SQL注入
    $textarea = mysqli_real_escape_string($conn, $_POST['txtarea']);
    $id = mysqli_real_escape_string($conn, $_POST['txtid']);
    
    $sql = "UPDATE English SET Content='".$textarea."' WHERE ID='".$id."'";
    // 执行更新并判断是否成功
    if(mysqli_query($conn, $sql)){
        echo "更新成功！";
    } else {
        echo "更新失败：" . mysqli_error($conn);
    }
}

mysqli_close($conn);
?>

关键修改点说明 #

• 独立表单：把<form>的开启和闭合都放到while循环内部，确保每一行的输入和按钮都属于单独的表单，提交时只会传递当前行的数据。
• 替换为mysqli扩展：替换所有mysql_*函数为mysqli_*函数，保证代码在新版本PHP中能正常运行。
• HTML转义：用htmlspecialchars()处理输出到页面的内容，防止XSS攻击。
• SQL注入防护：用mysqli_real_escape_string()转义用户输入的内容，避免恶意SQL注入（更推荐使用mysqli预处理语句，安全性更高）。

内容的提问来源于stack exchange，提问作者Mahmoud M. Abdelmalek