[PHP/MYSQL] Datensatz aus Liste löschen und zurückkehren

[Luckie]

Ich habe ein Problem eher konzeptioneller Natur.

Es gibt zwei Skripte. In dem einem wird eine Auswahl getroffen, diese Auswahl wird an ein zweites Skript übergeben. Dieses zweite Skript führt mit Hilfe dieser Auswahl eine Abfrage durch und stellt das Ergebnis in einer Tabelle dar. In dieser Tabelle gibt es für jede Zeile Links, um den entsprechenden Datensatz anzuzeigen, zu bearbeiten oder zu löschen. Anzeigen und bearbeiten ist kein Problem, da dazu weitere Skripte aufgerufen werden. Probleme habe ich mit dem Löschen.

Bisher wird dazu ein drittes Skript aufgerufen, welches den Datensatz löscht und dann eine separate Seite anzeigt. Von dort kann man jetzt zur Startseite zurück oder, wenn man zurück navigiert zur Seite mit der Ergebnistabelle der Abfrage. Das ist unschön.

Was ich gerne hätte: Man löscht einen Datensatz und dann wird wieder die Seite mit der aktualisierten Ergebnistabelle geladen. Nur wie bekomme ich das hin? Die Daten werden per Post an das zweite Skript übergeben.

Etwas Quellcode:
Erste Skript:

Code:

<?php         
         $query = "SELECT name FROM adr_kategorien GROUP BY name ASC";
         $resultset = mysql_query($query);                        
         echo "<form action='katlist.php' method=\"POST\">\n";      
         echo "<select name='kategorie' size='1'>\n";
         echo "<option value=''>keine Auswahl</option>\n";      
         while($row = mysql_fetch_object($resultset))
         {
            echo "<option value='".$row->name."'>".$row->name."</option>\n";
         }      
         echo "</select>\n";   
         echo "<input type=\"submit\" value=\"OK\"></input>\n";
         echo "</form>\n";
      ?>

Zweite Skript:

Code:

$query = "SELECT a.*, k.name as k_name
         FROM adressen a
         LEFT OUTER JOIN adr_kategorien k on k.id = a.kategorie_id
         WHERE k.name='".$kat."'
         ORDER BY a.gesch_firma, a.name, a.vorname";
      $resultset = mysql_query($query);
      if (!$resultset)
      {
         echo "<p class='error'>".mysql_error()."</p>";
         mysql_close($connection);
         include("footer.html");
         die();
      }
      
      echo "<table>";
      echo "<tr><th>Nr</th><th>Name</th><th>Vorname</th><th>Firma</th><th>Aktionen</th><tr>";
      $nr = 1;
      while($row = mysql_fetch_object($resultset))
      {
         echo "<tr>
               <td>".$nr."</td>
               <td>".$row->name."</td>
               <td>".$row->vorname."</td>
               <td>".$row->gesch_firma."</td>               
               <td>
                  [url='./details.php?action=show&id=".$row->id."'][img]./images/view.jpg[/img][/url]
                  [url='./form.php?id=".$row->id."'][img]./images/edit.jpg[/img][/url]
                  [url='./delete.php?id=".$row->id."'][img]./images/delete.jpg[/img][/url]
               </td>
            </tr>\n";
         $nr++;
      }
      echo "</table>";

Wie man sieht, wird zum Löschen das Skript delete.php aufgerufen. Ich würde jetzt aber gerne ohne den Umweg über das dritte Skript einen Datensatz löschen und die Ergebnistabelle wieder anzeigen.

[jfheins]

Was hindert dich daran, eine Seite - nennen wir sie action.php - zu machen, die die Datenmanipulation macht und den Benutzer wieder zurückschickt?

Oder du kannst das auch in verschiedenen Dateien machen - und am Ende der Delete.php kannste ja einen Redirect-Header einfügen, sodass man direkt wieder auf die Seite mit der Tabelle geleitet wird

Hier mal ein Beispiel, wie ich es mal gemacht hab:

Code:

<?php   

// including neccessary files
include ("includes/inc_verify.php");
include("../includes/inc_function.php");

if(!isset($_REQUEST['action']))
{
         header("Location: home.php");
}

// quick actions, of which the user won't see anything -> redirection
if($_REQUEST['action'] == "new") // insert new page ->home
{
   $res = mysql_query("INSERT INTO de (menue) VALUES ('...")
   or die("unable to access database
" . mysql_error());
   header("Location: " . $_GET['where'] . ".php");
   exit();
}
// Bei mehreren Möglichkeiten kommmt auch ein switch-case in Frage
?>

Ganz ohne Umweg über eine dritte Datei kommst du aus indem du den Link so machst: $_SERVER['PHP_SELF'] . '?action=delete&id=56'
Und dann am Anfang überprüfen ob in action was drinsteht und ggf. handeln - erst danach anzeigen


Due siehtst, es führen viele Wege nach Rom

[Luckie]

An Umleitungen hab eich auch schon gedacht, nur konnte ich mich mit der Idee nicht anfreunden. Ich meine, es müsste auch irgendwie anders, elegqanter gehen.

Zitat von jfheins:

Ganz ohne Umweg über eine dritte Datei kommst du aus indem du den Link so machst: $_SERVER['PHP_SELF'] . '?action=delete&id=56'
Und dann am Anfang überprüfen ob in action was drinsteht und ggf. handeln - erst danach anzeigen

Ja, so arbeite ich schon in dem Projekt. Aber die übergeben Daten sind dann doch wohl weg oder irre ich mich da? Das heißt, ich könnte die Tabelle nicht wieder erzeugen. Das ist eigentlich mein Problem.

[mirage228]

Du könntest es über AJAX lösen. In der Ergebnistabelle wird beim "Löschen"-Link eine JS-Funktion aufgerufen, die intern ein PHP-Skript aufruft, was den Datensatz löscht. Ist die Operation abgeschlossen lädt sich die Ergebnisliste einfach neu bzw. manipuliert die Tabelle, sodass der Datensatz dort auch weg ist.

Viele Grüße

[Luckie]

Das ganze muss auch mit deaktivierten Javascript funktionieren. Aber ich könnte ja die daten als Parameter beim Löschen übergeben.

[jfheins]

Zitat von Luckie:

An Umleitungen hab eich auch schon gedacht, nur konnte ich mich mit der Idee nicht anfreunden. Ich meine, es müsste auch irgendwie anders, eleganter gehen.

Freunde dich damit ruhig an, da ist nichts verwerfliches dabei

Zitat:

Ja, so arbeite ich schon in dem Projekt. Aber die übergeben Daten sind dann doch wohl weg oder irre ich mich da? Das heißt, ich könnte die Tabelle nicht wieder erzeugen. Das ist eigentlich mein Problem.

Naja, du kannst ja die Daten die du zur Erzeugung der Tabelle brauchst ebenfalls mitgeben.

Also dann sowas: zweitesscript.php?category=3&action=delete&id=12
Dann löschst du den Datensatz mit der Id 12 und zeigst anschlißend die Tabelle mit der Kategorie an
Da die Daten dann aber als GET übergeben werden, würde ich die Combobox auch auf GET umstellen

[Luckie]

Ja, ich denke, so werde ich es machen. Danke für die unterstützenden Überlegungen.

[generic]

Tipp noch:
Du solltest Code vom Design trennen z.B. durch Nutzung einer Templateengine.
Das macht das Leben echt leichter.

Dreamteam - Smarty und PearDB

[Luckie]

Danke für den Hinweis. Aber das ist mir bekannt. Nur wäre für diesen Zweck ein Templatengine etwas Overkill. Es ist nur eine kleine private Adressdatenbank.

[generic]

Früher oder später wirst du eine Wartung dran vornehmen.
Wenn du keine TE nutzen möchtest, dann lagere doch einfach die Ausgabe in
einzelne PHP-Dateien aus. Diese bindest du dann der Include ein.
In den Dateien kannst du dann auch die Kurzformen für die Ausgabe nutzen.

Code:

<?=$Varname?>

Ist immer noch sauberer.