Question

我想用复选框删除mySQL行。这是一个代码应该根据互联网上的某人工作，但有些原因它不适合我。当我单击删除它只刷新但行不会消失。这与表格中的ID有关吗？

<body>
<?php
$host="localhost"; // Host name
$username="root"; // Mysql username
$password="*****"; // Mysql password
$db_name="test"; // Database name
$tbl_name="deviation"; // Table name

// Connect to server and select databse.
mysql_connect("$host", "$username", "$password")or die("cannot connect");
mysql_select_db("$db_name")or die("cannot select DB");

$sql="SELECT * FROM $tbl_name";
$result=mysql_query($sql);

$count=mysql_num_rows($result);

?>
<table width="400" border="0" cellspacing="1" cellpadding="0">
<tr>
<td><form name="form1" method="post" action="">
<table width="400" border="0" cellpadding="3" cellspacing="1" bgcolor="#CCCCCC">
<tr>
<td bgcolor="#FFFFFF">&nbsp;</td>
<td colspan="4" bgcolor="#FFFFFF"><strong>Delete multiple rows in mysql</strong> </td>
</tr>
<tr>
<td align="center" bgcolor="#FFFFFF">Åtgärda</td>
<td align="center" bgcolor="#FFFFFF"><strong>Chassinummer</strong></td>
<td align="center" bgcolor="#FFFFFF"><strong>Problem detail</strong></td>
<td align="center" bgcolor="#FFFFFF"><strong>Fault code</strong></td>
<td align="center" bgcolor="#FFFFFF"><strong>Position</strong></td>
<td align="center" bgcolor="#FFFFFF"><strong>Help object</strong></td>
<td align="center" bgcolor="#FFFFFF"><strong>Operation step</strong></td>
</tr>
<?php
while($rows=mysql_fetch_array($result)){
?>
<tr>
<td align="center" bgcolor="#FFFFFF"><input name="checkbox[]" type="checkbox"     id="checkbox[]" value="<?php echo $rows['id']; ?>"></td>
<td bgcolor="#FFFFFF"><?php echo $rows['chassi']; ?></td>
<td bgcolor="#FFFFFF"><?php echo $rows['problem_detail']; ?></td>
<td bgcolor="#FFFFFF"><?php echo $rows['fault_code']; ?></td>
<td bgcolor="#FFFFFF"><?php echo $rows['fault_code']; ?></td>
<td bgcolor="#FFFFFF"><?php  echo $rows['position']; ?></td>
<td bgcolor="#FFFFFF"><?php echo $rows['help_object']; ?></td>
<td bgcolor="#FFFFFF"><?php echo $rows['operation_step']; ?></td>
</tr>
<?php
}
?>

<tr>
<td colspan="5" align="center" bgcolor="#FFFFFF"><input name="delete"     type="submit"     id="delete" value="Delete"></td>
</tr>
<?php

**$checkbox = $_POST['checkbox'];**
**$delete = $_POST['delete'];**

// Check if delete button active, start this
if($delete){
for($i=0;$i<$count;$i++){
$del_id = $checkbox[$i];

**$sql = "DELETE FROM $tbl_name WHERE id='$del_id'";**
$result = mysql_query($sql);
}

// if successful redirect to delete_multiple.php
if($result){
echo "<meta http-equiv=\"refresh\" content=\"0;URL=deletetable.php\">";
}
}
mysql_close();
?>
</table>
</form>
</td>
</tr>
</table>
</body>

Answer 1

我想引导你注意某些事情：

1）PHP中有许多行用无意义的双星号括起来。我很震惊PHP实际上经历过那些。

2）您在脚本顶部的全局范围内定义了$result（从mysql_query()获得了资源指针）。这意味着在底部，您检查if($result)，检查将始终到true（除非存在语法错误）。这也意味着无论删除是否实际发生，当您单击“提交”时页面将始终刷新。

正如我在评论中所提到的那样，（坦率地说）该代码是一个废话。它容易受到SQL注入攻击，代码实际上并没有做它应该做的事情，它使用了不推荐使用的属性和函数......代码无法做什么你想要的不是因为你的表中的ID，而是因为你从互联网上复制了破损，无效的代码。

Answer 2

不要遍历整个表格替换此

for($i=0;$i<$count;$i++){
$del_id = $checkbox[$i];

**$sql = "DELETE FROM $tbl_name WHERE id='$del_id'";**
$result = mysql_query($sql);
}

与

if(isset($_POST['checkbox']) && is_array($_POST['checkbox')){
    foreach($_POST['checkbox'] as $id){
        $id = (int)$id;
        $sql = "DELETE FROM $tbl_name WHERE id='$id'";
        $result = mysql_query($sql);
    }
}

对于php中的循环获取数组

2 个答案: