MySQL zapytanie usuwające wiersz o konkretnym id. Opcje

[Peperoni]

Witam,
piszę stronkę na zaliczenie i mam w panelu administracyjnym tabelę z menu do której ładnie się dodaje i usuwa kiedy podam konkretne ID. Zrobiłem obok wiersza w tabeli html przycisk usuń i chciałbym żeby usuwał dokładnie ten wiersz do którego jest przypisany.

Tutaj kod tworzący tabelę i wypełniający ją danymi z bazy:

[PHP] pobierz, plaintext 
require_once ('../db_connect.php');
 
		if ($result = $polaczenie->query("SELECT * FROM jadlospis ORDER BY ID")){
			if($result->num_rows > 0) {
			echo  '<div class="card mb-3">';
            echo  '<div class="card-header">';
 
            echo  'Jadłospis</div>';
            echo '<div class="card-body">';
             echo '<div class="table-responsive">';
 
				echo '<table class="table table-bordered" id="dataTable" width="100%" cellspacing="0">';
				echo '<thead>';
				echo "<tr>
				<th>ID </th>
				<th>Dzień </th>
				<th>Śniadanie </th>
				<th>Obiad</th>
				<th>Podwieczorek</th>
				<th>Usuń/Popraw</th>
				</tr>";
 
				echo '</thead>';
 
				while($row = $result->fetch_object()){
					echo '<tbody>';
					echo "<tr>";
 
					echo "<td>" . $row ->id . "</td>";
					echo "<td>" . $row ->dzien . "</td>";
					echo "<td>" . $row ->sniadanie . "</td>";
					echo "<td>" . $row ->obiad . "</td>";
					echo "<td>" . $row ->podwieczorek . "</td>";
 
					echo "<td> <input type=submit class=btn btn-primary btn-block name=idusun value=usuń> </td>";
 
 
 
					echo "</tr>";
					echo '<tbody>';
 
				}
 
				echo "</table>";
[PHP] pobierz, plaintext 

Zapytanie SQL pod Submita wygląda następujaco:

[PHP] pobierz, plaintext 
  if (isset($_POST['idusun']))
    {
 
		//Udana walidacja 
		$wszystko_OK=true;
 
		$userid = $_SESSION['user'];
 
 
 
		[sql]$delete = "DELETE FROM `jadlospis` WHERE `ID` = id";[/sql]
 
		//Połączenie z bazą
 
		require_once "../db_connect.php";
        mysqli_report(MYSQLI_REPORT_STRICT);
 
		try
        {
		 $polaczenie = new mysqli($host, $db_user, $db_password, $db_name);
		 if ($polaczenie->connect_errno!=0)
            {
                throw new Exception(mysqli_connect_errno());
            }
		else
            {
 
				if ($polaczenie->query($delete))
                    {
                        $_SESSION['udanezamowienie']=true;
                        header('Location: form-basic.php');
                    }
                    else
                    {
                        throw new Exception($polaczenie->error);
                    }
 
                }
 
                }       
 
        catch(Exception $e)
        {
            echo '<span style="color:red;">Coś się popsuło!</span>';
            echo '<br />Informacja developerska: '.$e;
        }
 
	}
[PHP] pobierz, plaintext 

Niestety usuwa ono wszystkie wiersze, a chciałbym tylko jeden.
Proszę bardzo o pomoc.

[viking]

Zmień to sobie najlepiej na formularz dla każdego wiersza który np w input hidden będzie zawierał id do usunięcia.

[Peperoni]

Zrobiłem tak jak poradziłeś:

[PHP] pobierz, plaintext 
echo "<input type=hidden name=idusun value=".$row->id.">";
 
$usun = $_POST['idusun'];
$delete = "DELETE FROM `jadlospis` WHERE `ID` = '$usun'";
[PHP] pobierz, plaintext 

I teraz usuwa mi po kolei od najwyższego ID niezależnie od tego, w którym wierszu wcisnę usuń. W źródle strony sprawdzałem, php pokazuje po kolei wartości ID każdego wiersza.

Ten post edytował Peperoni 30.08.2019, 05:45:24

[viking]

Pokaż cały wygenerowany html tego fragmentu.

[Tomplus]

Nie musi nic pokazywać. Zamknął w <form> wszystkie input-hidden więc wiadomo, że formularz po wysłaniu będzie odczytywał ostatnią taką zmienną.

@peperoni
Zamiast input-submit daj button-submit. Jest to lepsze rozwiązanie, i pominiesz problem który masz wyżej:

[PHP] pobierz, plaintext 
echo "<td> <button type='submit' class='btn btn-primary btn-block' name='idusun' value='".$row->id."'>usuń</button> </td>";
[PHP] pobierz, plaintext 

Teraz nie potrzebujesz input-hidden, a wartość _POST[idusun] zostanie poprawnie przesłana jednym kliknięciem.

Dobrym zwyczajem jest zamykanie w cudzysłowie podwójne lub pojedyńcze wartości atrybutów tagów HTML.

[Peperoni]

Działa jak powinno. Dzięki wielkie.

[viking]

Ja to wiem ale przy okazji chciałem nauczyć żeby czasami sprawdzać wygenerowany kod czy zgadza się z tym co być powinno. I też pokazać że to co zrobił nijak ma się do mojej odpowiedzi. W tym kodzie nie było też żadnego formularza co jest niepoprawne.
Jeszcze kolejna rzecz. Rzutuj to id na int chyba że chcesz atak sql injection.