[PHP] Beiträge listen und Löschbutton

mastermo · 11/28/2014, 17:29

Heyho Leute,

ich stehe wiedermal vor einem Problem und brauche euren Rat. Ich möchte Datenbankeinträge in einer Tabelle listen und daneben einen Löschbutton womit man den Wunscheintrag löschen kann.

Das ganze sollte in etwa so aussehen:

Code:

id | title | [URL="#"]Löschbutton[/URL]

Kann mir da wer von euch nen Codesnippet geben womit ich das umsetzen kann?

Mein bisheriger Code:

PHP Code:


			
<html>

<head>

    <title>Test</title>

</head>

<body>

<?php 



require_once('./config.php');



$query = mysql_query("SELECT * FROM events ORDER BY 'id' ASC");



while($row = mysql_fetch_assoc($query))

{

    $id = $row['id'];        

    $title = $row["title"];

    echo "



    

    <h3>ID {$id} <font color='red'>am</font> Event {$title}</h3>

    ";    

}

?>

</body>

</html>

Gruß Mo

~ JWonderpig ~ · 11/28/2014, 18:07

z.B.

PHP Code:


			
echo "<a href='delete.php?id={$id}'>Löschen</a>"

Das gleich dann mit Edit - Jedoch solltest du aufpassen, welcher User Rechte für diese Datei hat.

mastermo · 11/28/2014, 19:44

Ich bräuchte ja das eigentliche Snippet womit ich den Beitrag auch löschen kann. Sprich die delete.php

Bin leider noch nicht besonders weit mit PHP

Guron · 11/28/2014, 19:53

Quote:

Originally Posted by mastermo

Ich bräuchte ja das eigentliche Snippet womit ich den Beitrag auch löschen kann. Sprich die delete.php Bin leider noch nicht besonders weit mit PHP

So weit ich weiß brauchst du keine delet.php - du kannst auch deine aktuelle Datei löschen und eine Succes Message einblenden lassen. Das Script packst du eigentlich nach oben.

Wie bereits erwähnt solltest du noch absichern welcher Rank was löschen kann. Einfach die aktuelle SESSION abfragen was für einen Rank der angemeldetete Benutzer hat.

Mikesch01 · 11/28/2014, 20:02

@Threadersteller

Warnuk3r · 11/28/2014, 20:29

Falls du es so machen willst wie Caiv es dir vorschlägt:

Bei delete.php?id=4 wird über die GET-Methode der Wert 4 übermittelt. Durch $_GET['id'] kannst du ihn ansprechen.

Code:

$id = $_GET['id'];
$query = mysql_query("DELETE FROM events WHERE id = $id");

Evtl. " ' . um $id herum, falls es nicht als Variable wahrgenommen wird.

An deiner Stelle würde ich eher ein Formular aus der Ausgabe machen und wenn man den Button zum löschen klickt, wird er abgeschickt und löscht direkt. So in etwa wie Guron schrieb. Du musst halt wie eben bereits oft gesagt aufpassen, wer zu der Datei Zugang hat.

Übrigens würde ich Mysqli verwenden.

NotEnoughForYou · 11/28/2014, 20:41

Quote:
Originally Posted by Warnuk3r
Falls du es so machen willst wie Caiv es dir vorschlägt:

Bei delete.php?id=4 wird über die GET-Methode der Wert 4 übermittelt. Durch $_GET['id'] kannst du ihn ansprechen.
Code:
$id = $_GET['id'];
$query = mysql_query("DELETE FROM events WHERE id = $id");
Evtl. " ' . um $id herum, falls es nicht als Variable wahrgenommen wird.

An deiner Stelle würde ich eher ein Formular aus der Ausgabe machen und wenn man den Button zum löschen klickt, wird er abgeschickt und löscht direkt. So in etwa wie Guron schrieb. Du musst halt wie eben bereits oft gesagt aufpassen, wer zu der Datei Zugang hat.

Übrigens würde ich Mysqli verwenden.

Vorher aber bitte noch escapen sonst ist ein kompletter DB Zugriff möglich.

mastermo · 11/28/2014, 23:24

Ich werds morgen mal austesten. Erstmal ein großes Danke an die tatkräftigen Lösungsvorschläge

Zur Berechtigung: Das Vorhaben ist nur für ein Adminpanel gedacht, wozu der Admin Zugang hat. Sprich wer kein Admin ist kommt auch nicht hin (hab ne Session vom Login davor, nur nicht im Snippet gepostet

)

.Kirataro · 11/28/2014, 23:44

Tipp: Gestalte es als Tabelle!

PHP Code:


			
<table>
while($row = mysqli_fetch_object($query))
{
                        $sender = get_name($row->id_sender);
                        $empfaenger = get_name($row->id_empfaenger, $db);
                        echo "<tr><td>".$sender."</td><td>".$row->date."</td><td>".$row->time."</td><td>".$row->subject."</td><td>".'<a href="index.php?s=message&mode=read&id='.$row->id.'">Lesen</a>'."</td><td>".'<a href="index.php?s=message&mode=delete&id='.$row->id.'">Löschen</a>';
}
</table>

(ist von einem alten PN-System meines Mini-Browsergames, bau dir einfach Sachen um, wie du's brauchst).

~ JWonderpig ~ · 11/29/2014, 00:06

Oder man schreibt eine Funktion, die alle Daten aus der DB in einem Array speichert.
So muss man nicht den HTML Code in die while-Schleife schreiben und kann den selben Code an mehreren Stellen nutzen

NotEnoughForYou · 11/29/2014, 12:15

Quote:

Originally Posted by Caiv

Oder man schreibt eine Funktion, die alle Daten aus der DB in einem Array speichert.
So muss man nicht den HTML Code in die while-Schleife schreiben und kann den selben Code an mehreren Stellen nutzen

Um die Inhalt des Arrays aber wieder auszugeben braucht man wieder ne schleife (foreach / while / for whatever), von daher ist es grad egal

~ JWonderpig ~ · 11/29/2014, 13:22

Quote:

Originally Posted by NotEnoughForYou

Um die Inhalt des Arrays aber wieder auszugeben braucht man wieder ne schleife (foreach / while / for whatever), von daher ist es grad egal

Jain^^

Wenn er auf einer Startseite alle Einträge auslesen will und nochmal im Backend, müsste er 2x mit einer While-Schleife arbeiten.
Mit einem Array kann man nur die Werte ausgeben, die an der Stelle gebraucht werden. Auf der Startseite sollen ja z.B. nicht die Links zum Löschen angegeben werden^^

PHP Code:


			


$db = new mysqli(...);



function getBeitraege() {

    GLOBAL $db;



    $sql = "SELECT * FROM `tabelle`";

    $result = $db->query($sql);



    $results = array();



    while ($row = $result->fetch_assoc()) {

        $results[] = $row;

    }



    return $results;

}

Dann kann man später mit einer foreach-Schleife nur die Werte ausgeben die man braucht:

Startseite:

PHP Code:


			
foreach (getBeitraege() as $beitrag) {

    echo '<h2>' . $beitrag['headline'] . '</h2>';

   echo '<p>' . $beitrag['content'] . '</p>';

}

Backend:

PHP Code:


			
foreach (getBeitraege() as $beitrag) {

    //Alles was du im Backend brauchst

    echo '<a href="delete.php?id=' . $beitrag['id'] . '">Löschen</a>';

    ...

}

NotEnoughForYou · 11/29/2014, 13:26

Quote:

Originally Posted by Caiv

Jain^^

Wenn er auf einer Startseite alle Einträge auslesen will und nochmal im Backend, müsste er 2x mit einer While-Schleife arbeiten.
Mit einem Array kann man nur die Werte ausgeben, die an der Stelle gebraucht werden. Auf der Startseite sollen ja z.B. nicht die Links zum Löschen angegeben werden^^

PHP Code:

$db = new mysqli(...); function getBeitraege() {     GLOBAL $db;     $sql = "SELECT * FROM `tabelle`";     $result = $db->query($sql);     $results = array();     while ($row = $result->fetch_assoc()) {         $results[] = $row;     }     return $results; }

Dann kann man später mit einer foreach-Schleife nur die Werte ausgeben die man braucht:

Startseite:

PHP Code:

foreach (getBeitraege() as $beitrag) {     echo '<h2>' . $beitrag['headline'] . '</h2>';    echo '<p>' . $beitrag['content'] . '</p>'; }

Backend:

PHP Code:

foreach (getBeitraege() as $beitrag) {     //Alles was du im Backend brauchst     echo '<a href="delete.php?id=' . $beitrag['id'] . '">Löschen</a>';     ... }

Wenn Frontend und Backend allerdings getrennt sind (so wie es sein sollte), dann würde sowieso niemals beides gleichzeitig aufgerufen werden, was somit die Performanceverbesserung durch die einmalige Zwischenspeicherung im Array wieder zu nicht machen würde.

Außerdem würde es auf der Startseite nicht viel bringen alle Einträge zu selectieren (was man vermeiden sollte) sondern dort auch nur die, die benötigt werden. Fürs Backend bräuchte man dann schon alle, deshalb würden sich hier unterschiedliche Funktionen anbieten.