Lösch Button für mysql Datensatz in Tabelle

qwertz · 6 Juli 2020

Hallo,

An der Webseite an der ich gerade Baue, gibt es eine Tabelle, die Datensätze anzeigen soll. Nachdem der Datensatz von uns verarbeitet wurde, soll man ihn über einen Button Löschen können. Der Button existiert und funktioniert, er kann einzelne Datensätze löschen. Nur das Problem ist, er löscht die letzten Datensätze und nicht die Datensätze, die man über den Button auswählt. Ich hatte auch meine Probleme überhaubt den Code so zu schreiben, dabei hat mir ein anderes Thema geholfen: https://www.html.de/threads/loeschen-button-in-tabelle.56916/

Ich hoffe ihr könnt mir helfen, bzw. habt mich überhaubt verstanden. Ich Bedanke mich schon mal im vorhinein. Hier sind noch die Codes:

Der Code hier der ist die Tabelle

Code:

  <table class="table table-striped">
    <form action='delete.php' method="POST" >
    <thead class="thead-dark">
      <tr>
        <th>  </th>
        <th> Order ID </th>
        <th> Food ID </th>
        <th> Order Date </th>
        <th> Food Name </th>
        <th> Quantity </th>
        <th> Ordernumber </th>
        <th> actions </th>

      </tr>
    </thead>

    <?PHP
      //OUTPUT DATA OF EACH ROW
      while($row = mysqli_fetch_assoc($result)){
    ?>

  <tbody>
    <tr>
      <td> <span class="glyphicon glyphicon-menu-right"></span> </td>
      <td><?php echo $row["order_ID"]; ?></td>
      <td><?php echo $row["F_ID"]; ?></td>
      <td><?php echo $row["order_date"]; ?></td>
      <td><?php echo $row["foodname"]; ?></td>
      <td><?php echo $row["quantity"]; ?></td>
      <td><?php echo $row["b_numm"]; ?></td>
      <td>
      <input type='hidden' name='id' value="<?php echo $row['order_ID']; ?>">
      <input type="SUBMIT" name="delete" value="Loeschen">
      </td>

    </tr>
  </tbody>

<?php } ?>
</form>
  </table>
    <br>

das hier ist der Code der die sachen löscht

PHP:

<?php
error_reporting(-1);
ini_set('display_errors', true);

$gut = $_POST['id'];

  $con = mysqli_connect('localhost','root','', 'foodorder');

    mysqli_select_db($con,'orders');

  $sql = "DELETE FROM `orders` WHERE `order_ID` = ";

  if(mysqli_query($con,$sql))
        header("refresh:1; url=view_order_kitchen.php");

    else{
        echo "Fehler";
}
  ?>

(Die erste code ist von der Datei view_order_kitchen.php und der zweite code von delete.php)

MfG
BWDS

qwertz · 6 Juli 2020

Das bedeutet sie würden es über get regeln. Ich habe es vorher auch über get ausprobiert nur zeigte er mir immer ein fehler der string conversation an deshalb bin ich auf post gewechselt. Wenn man mir bei get auch noch weiterhelfen würde, wär ich dankbar

MfG
BWDS

qwertz · 6 Juli 2020

Wie würde man das jetzt via Get machen (Code für den Lösungsansatz währe nett)

Gelöschtes Mitglied 36401 · 6 Juli 2020

Dafür brauchst du kein Formular.
<form action='delete.php' method="POST" > und </form> raus

PHP:

    <tr>
      <td> <span class="glyphicon glyphicon-menu-right"></span> </td>
      <td><?php echo $row["order_ID"]; ?></td>
      <td><?php echo $row["F_ID"]; ?></td>
      <td><?php echo $row["order_date"]; ?></td>
      <td><?php echo $row["foodname"]; ?></td>
      <td><?php echo $row["quantity"]; ?></td>
      <td><?php echo $row["b_numm"]; ?></td>
      <td><a href="delete.php?order_id="<?php echo $row['order_ID']; ?>" class="button">Löschen</a></td>
    </tr>

In delete.php

PHP:

<?php
  if (isset($_GET['order_id'))
  {
     // Hier hast du jetzt die ID von dem Datensatz, der gelöscht werden soll
  }
?>

Möglicher CSS-Code für den Link

CSS:

<style>
      .button {
      background-color: #FF4500;
      border: none;
      color: white;
      padding: 5px;
      text-align: center;
      text-decoration: none;
      font-size: 14px;
      cursor: pointer;
      }
</style>

qwertz · 6 Juli 2020

Jetzt haben wir noch ein problem

er übermittelt die order id nicht.

MfG

BWDS

Gelöschtes Mitglied 36401 · 6 Juli 2020

Und du glaubst jetzt wirklich, wir könnten das Problem ohne deinen angepassten Quellcode erkennen? Wenn ja, dann muss ich dich enttäuschen, meine Glaskugel ist wegen Corona in Kurzarbeit.

qwertz · 6 Juli 2020

Angepasster Quellcode:

view_order_kitchen.php

Code:

  <table class="table table-striped">
    <thead class="thead-dark">
      <tr>
        <th>  </th>
        <th> Order ID </th>
        <th> Food ID </th>
        <th> Order Date </th>
        <th> Food Name </th>
        <th> Quantity </th>
        <th> Ordernumber </th>
        <th> actions </th>

      </tr>
    </thead>

    <?PHP
      //OUTPUT DATA OF EACH ROW
      while($row = mysqli_fetch_assoc($result)){
    ?>

  <tbody>
    <tr>
      <td> <span class="glyphicon glyphicon-menu-right"></span> </td>
      <td><?php echo $row["order_ID"]; ?></td>
      <td><?php echo $row["F_ID"]; ?></td>
      <td><?php echo $row["order_date"]; ?></td>
      <td><?php echo $row["foodname"]; ?></td>
      <td><?php echo $row["quantity"]; ?></td>
      <td><?php echo $row["b_numm"]; ?></td>
      <td><a href="delete.php?order_id="<?php echo $row['order_ID']; ?> class="button">Löschen</a></td>

    </tr>
  </tbody>

<?php } ?>
  </table>

delete.php

Code:

<?php
error_reporting(-1);
 ini_set('display_errors', true);


  $con = mysqli_connect('localhost','root','', 'foodorder');

    mysqli_select_db($con,'orders');

    if (isset($_GET['order_id']))
{
    $sql = "DELETE FROM `orders` WHERE `order_ID` = $_GET";

}


  if(mysqli_query($con,$sql))
        header("refresh:1; url=view_order_kitchen.php");

    else{
        echo "Fehler";
}

  ?>

Hier der Fehler CODE

Gelöschtes Mitglied 36401 · 6 Juli 2020

if (isset($_GET['order_id']))

$sql = "DELETE FROM `orders` WHERE `order_ID` = $_GET";

Da fällt dir nix auf?

qwertz · 6 Juli 2020

Nein leider nicht. Trotzdem wird von view_order_kitchen.php keine id weitergegeben.

Gelöschtes Mitglied 36401 · 6 Juli 2020

BWDS schrieb:
Nein leider nicht.

Traurig, dann solltest du dich mehr mit den Grundlagen beschäftigen.

BWDS schrieb:
Trotzdem wird von view_order_kitchen.php keine id weitergegeben.

Woher weißt du das? Ich sehe da keinerlei Debugausgaben.

Richtig debuggen

Man bemerkt, dass ein Skript nicht das tut, was es soll.
Man schreibt an den Anfang des Scriptes die Zeile: error_reporting(-1);
Man verwendet ini_set('display_errors', true); damit die Fehler auch angezeigt werden.
Man versucht, die Stelle die daran Schuld sein kann, schonmal einzugrenzen. Falls dies nicht geht, wird zunächst das komplette Skript als fehlerhaft angesehen.
An markanten Stellen im Skript lässt man sich wichtige Variableninhalte ausgeben und ggf. auch in bedingten Anweisungen eine kurze Ausgabe machen, um zu überprüfen, welche Bedingung ausgeführt wurde. Wichtig bei MySQL Fehlern (...not a valid MySQL result resource...): mysqli_error() verwenden oder Abfrage ausgeben und zb mit phpmyadmin testen.
Schritt 5 wird so lange wiederholt, bis Unstimmigkeiten im Skript auffallen
Damit hat man das Problem (Unstimmigkeit) gefunden und kann versuchen diese zu beheben. Hierzu dienen dann die PHP-Dokumentation und andere Quellen als Ratgeber.
Lässt sich das konkrete Problem trotzdem nicht beheben, kann man in Foren um Rat fragen.
Das Programm läuft und man kann die Debug-Ausgaben wieder entfernen.

qwertz · 6 Juli 2020

m.scatello schrieb:
Woher weißt du das? Ich sehe da keinerlei Debugausgaben.

indem ich einfach den code aus der delete.php ersetze durch den code:

Code:

<?php
echo '<pre>';
print_r($_GET);
echo '</pre>';
?>

dann kommt das

daran sieht man, dass keine id übergeben wurde

hat man zb die id 467 würde das heraus kommen:

m.scatello schrieb:
Da fällt dir nix auf?

was sollte mir da auffallen? dass man nach dem gleich zeichen das schreibt?

if (isset($_GET['order_id']))

$sql = "DELETE FROM `orders` WHERE `order_ID` = $_GET['order_ID']";

kläre mich bitte mal auf dies wäre eusers nett

MfG
BWDS

Gelöschtes Mitglied 36401 · 6 Juli 2020

Dann kann doch nur hier das Problem liegen:

PHP:

<a href="delete.php?order_id="<?php echo $row['order_ID']; ?> class="button">Löschen</a></td>

Du müsstest in der URL bei delete.php die ID sehen. Wenn das nicht so ist, dann musst du mal nachgesehen, wie der HTML-Code aussieht, den der Browser bei der Zeile bekommt.

qwertz · 6 Juli 2020

ich habe es jetzt gerade gerückt, und nun wird die ID übertragen.
quellcode aktualisierung:
view_order_kitchen.php

Code:

  <table class="table table-striped">
    <thead class="thead-dark">
      <tr>
        <th>  </th>
        <th> Order ID </th>
        <th> Food ID </th>
        <th> Order Date </th>
        <th> Food Name </th>
        <th> Quantity </th>
        <th> Ordernumber </th>
        <th> actions </th>

      </tr>
    </thead>

    <?PHP
      //OUTPUT DATA OF EACH ROW
      while($row = mysqli_fetch_assoc($result)){
    ?>

  <tbody>
    <tr>
      <td> <span class="glyphicon glyphicon-menu-right"></span> </td>
      <td><?php echo $row["order_ID"]; ?></td>
      <td><?php echo $row["F_ID"]; ?></td>
      <td><?php echo $row["order_date"]; ?></td>
      <td><?php echo $row["foodname"]; ?></td>
      <td><?php echo $row["quantity"]; ?></td>
      <td><?php echo $row["b_numm"]; ?></td>
      <?php
    echo  "<td><a href=delete.php?id=".$row['order_ID'].">ENTFERNEN</a></td>"; 
      ?>


    </tr>
  </tbody>

<?php } ?>

  </table>
    <br>

delete.php

Code:

<?php
error_reporting(-1);
 ini_set('display_errors', true);


  $con = mysqli_connect('localhost','root','', 'foodorder');

    mysqli_select_db($con,'orders');

    if (isset($_GET['order_id']))
{
    $sql = "DELETE FROM `orders` WHERE order_ID = $_GET['order_ID'] ";

}


  if(mysqli_query($con,$sql))
        header("refresh:1; url=view_order_kitchen.php");

    else{
        echo "Fehler";
}

  ?>

trotzdem ist immernoch der wurm drin.

MfG
BWDS

Gelöschtes Mitglied 36401 · 6 Juli 2020

BWDS schrieb:
trotzdem ist immernoch der wurm drin.

Och menno, das ist doch kein Problembeschreibung und "Richtig debuggen" hast du auch nicht angewendet. Ist das wirklich so schwer???

PHP:

<?php
  error_reporting(-1);
  ini_set('display_errors', true);


  if (isset($_GET['order_id']))
  {
     $con = mysqli_connect('localhost','root','', 'foodorder');
 
     if (!$con)
     {
       echo "Fehler: konnte nicht mit MySQL verbinden." . PHP_EOL;
       echo "Debug-Fehlernummer: " . mysqli_connect_errno() . PHP_EOL;
       echo "Debug-Fehlermeldung: " . mysqli_connect_error() . PHP_EOL;
       exit;
     }

     mysqli_select_db($con,'orders')
       or die (mysqli_error($con));

     $sql = "DELETE FROM `orders` WHERE order_ID = $_GET['order_ID'] ";

     mysqli_query($con,$sql)
       or die (mysqli_error($con));
      
     header("refresh:1; url=view_order_kitchen.php");
  }
  else
    die ("Parameter nicht vorhanden");
?>

qwertz · 6 Juli 2020

wenn ich genau diesen code in delete.php einfüg zeigt er mir folgende fehlermeldung an:

der code ist aber soweit ich sehe richtig.

MfG
BWDS

Gelöschtes Mitglied 36401 · 6 Juli 2020

PHP:

$sql = "DELETE FROM `orders` WHERE order_ID = " . $_GET['order_ID'];

qwertz · 6 Juli 2020

Jetzt sagt er mir, dass der parameter nicht vorhanden ist, obwohl der parameter vorhanden ist.

was nun?

Gelöschtes Mitglied 36401 · 6 Juli 2020

Wenn PHP bei dem Script ausgibt, dass der Parameter nicht vorhanden ist, dann glaube ich PHP mehr als dir.

Hängt die ID an der URL?
Hast du das Script 1:1 übernommen?

qwertz · 6 Juli 2020

Die ID hängt an der url (siehe oben)
Alle parameter sind aber vorhanden.

MfG
BWDS

Gelöschtes Mitglied 36401 · 6 Juli 2020

Mach doch mal die Augen auf!!! Du hast wieder am HTML-Code rumgespielt, in der URL steht jetzt id und nicht order_id

Lösch Button für mysql Datensatz in Tabelle

Neues Mitglied

Neues Mitglied

Neues Mitglied

Gelöschtes Mitglied 36401

Guest

Neues Mitglied

Gelöschtes Mitglied 36401

Guest

Neues Mitglied

Gelöschtes Mitglied 36401

Guest

Neues Mitglied

Gelöschtes Mitglied 36401

Guest

Neues Mitglied

Gelöschtes Mitglied 36401

Guest

Neues Mitglied

Gelöschtes Mitglied 36401

Guest

Neues Mitglied

Gelöschtes Mitglied 36401

Guest

Neues Mitglied

Gelöschtes Mitglied 36401

Guest

Neues Mitglied

Gelöschtes Mitglied 36401

Guest