Delete row from database using PHP (using a HTML table) - php

Question

I'm making a "delete" function on my form. I'm not quite happy because it doesn't work! I have tried this code:
while($row = $result->fetch_assoc())
{
$ordernr = $row['ordernr'];
$klantnaam = $row['klantnaam'];
$productnaam = $row['productnaam'];
$productid = $row['productid'];
echo "<tr>";
echo "<td width='150px'>" . $ordernr . "</td>";
echo "<td width='150px'>" . $klantnaam . "</td>";
echo "<td width='300px'>" . $productnaam . "</td>";
echo "<td width='100px'>" . $productid . "</td>";
echo "<td align='center' width='50px'><input name='delete[$ordernr]' type='checkbox'></td>";
echo "</tr>";
}
echo "<tr>";
echo "<td><input type='submit' name='verwijderen' value='Verwijderen'/></td>";
echo "</tr>";
echo "</table>";
echo "</form>";
$delete = $_POST['delete'];
if (isset($_POST['verwijderen'])) {
foreach($delete as $ordernr => $delete)
{
$ordernr = mysqli_real_escape_string($ordernr);
$query = mysqli_query("DELETE FROM overzicht WHERE ordernr= $ordernr");
}
}
Do you guys know what I've done wrong?
Thanks in advance!

Answer 1

1)
echo "<td align='center' width='50px'><input name='delete[$ordernr]' type='checkbox'></td>";
transforms to
echo "<td align='center' width='50px'><input name='delete' type='checkbox' value='$ordernr'></td>";
2)
foreach($delete as $ordernr => $delete)
do it:
foreach($delete as $ordernr => $del)
you are overwriting the $_POST

Answer 2

mysqli_query syntaxt is
mysqli_query($con,"your query");

Answer 3

I'd go and say that your input checkbox needs a value. Like
<input type="checkbox" name="delete[1]" value="1" />
On the other hand, then it would be easier to have
<input type="checkbox" name="delete[]" value="$ordernr" />
And then you loop with
foreach ($_POST["delete"] as $id) delete(id)

Answer 4

where con variable in your query...
$query = mysqli_query($con,"DELETE FROM overzicht WHERE ordernr= $ordernr");

Answer 5

You're iterating while reusing your variable here:
foreach($delete as $ordernr => $delete)
Where the second $delete will overwrite the value of the first, and throw a spanner in the works.
I suggest to rename the prior instances of $delete to $deleteList and do:
foreach($deleteList as $ordernr => $delete)
Keep in mind that much of your code could be done in different ways that may be more suitable. You will find those as you follow your path in learning PHP.