Löschen eines DB-Eintrages und Live Aktualisierung bei der Seite

Stef · 7. April 2018

Hey,

ich habe ein Problem.

Und zwar habe ich Termine. Man kann diese erstellen, editieren und löschen. Alles funktioniert auch. Nur ich brauche den Array ($alleTermine), welcher alle aus der DB gezogenen Termindaten enthält, schon vor dem löschen und editieren. Somit wird nach dem Löschen erst beim 2ten Seitenaufruf dieser gelöschte Termin nicht mehr angezeigt.

Eine Idee wäre Ajax aber ich möchte dies so lösen. Weiß nur nicht wie ich es machen soll. Hoffe ihr könnt mir hilfreiche Tipps geben.

Der Code:

PHP

try{

    $selectTermin = $dbv->prepare("SELECT terminId, grund, beschreibung, datum, uhrzeit FROM termine");
    $selectTermin->execute();

    $alleTermine = $selectTermin->fetchAll(PDO::FETCH_ASSOC);

    for($i = 0; $i < count($alleTermine); $i++){

        if(isset($_POST['deleteTermin' . $i])){

            $terminId = $_POST['terminId'];

            

            $deleteTermin = $dbv->prepare("DELETE FROM termine WHERE terminId = :terminId");
            $deleteTermin->execute(array(":terminId" => $terminId));


    } 
    else if(isset($_POST['editTermin' . $i])){

        $terminId = $_POST['terminId'];

        header("LOCATION: termine/edit_termin.php?id=" . $terminId);
    }

    }
    
    


}catch(EXCEPTION $z){
            echo "Ein Fehler ist aufgetreten: " . $z->getMessage();
            exit();
}

Alles anzeigen

Die Ausgabe der Termine erfolgt so:

PHP

<?php 
                foreach ($alleTermine as $key => $value) {
                    echo "<div class='alleTermine'>
                            <ul>
                                <li>Termingrund: " . htmlspecialchars($value['grund']) . "</li>
                                <li>Terminbeschreibung: " . htmlspecialchars($value['beschreibung']) . "</li>
                                <li>Am: " . htmlspecialchars($value['datum']) . "</li>
                                <li>Um: " . htmlspecialchars($value['uhrzeit']). " Uhr </li>
                            </ul>

                            <form method='post'>

                                <input type='hidden' value='" . $value['terminId'] . "' name='terminId'>

                                <button name='deleteTermin" . $key . "'>Termin löschen</button>
                                <button name='editTermin" . $key . "'>Termin editieren</button>
                            </form>
                        </div>    
                    ";
                }
            ?>

Alles anzeigen

Gruß,

Stef

m.scatello · 7. April 2018

Das hört sich für mich so an, als wenn du dich nicht an das EVA-Prinzip hälst. Aber das kann man bei den hier geposteten Code-Fetzen nicht beurteilen.

cottton · 7. April 2018

Erst delete, edit und create.

Dann den aktuellen Stand laden (und ausgeben).

=)

Code

// pseudo code
isDelete = /*f.e.$_POST[action] === delete*/
isEdit = /*f.e.$_POST[action] === edit*/
isCreate /*f.e. $_POST[action] === create */

if isDelete
    if /*have id to delete and is valid*/
        if entityExists(/*id*/)
            deleteEntity(id)

else if isEdit
    if /*have id to edit and is valid*/
        if entityExists(/*id*/)
            editEntity(id, /*data ...*/)

else if isCreate
    if /*have id to create and is valid*/
        if entityExists(/*id*/)
            createEntity(/*data ...*/)

// and NOW load current data 
entities = loadEntities()
foreach entities as entity
    // ...

Alles anzeigen

Stef · 7. April 2018

Hey,

m.scatello : Ich würde schon sagen, dass ich mich so gut wie möglich an das EVA-Prinzip halte.

& cottton :

Okay. Ich habe erst create, dann delete und dann edit gemacht. Aber es kommt nach deinem Schema das selbe raus.

Das Problem ist, dass ich ja mehrere Termine habe. Ich gebe diese dann aus. Nun hat jeder Termin ein löschen und editieren Button. Aber die ganzen Buttons müssen einen unterschiedlichen Namen haben weil es sonst nicht funktioniert. Da dachte ich, dass ich dann an den Namen den Schlüssel hänge und dann mittels einer for-schleife alle buttons auch abdecke und ausführbahr mache. Aber für die for-schleife brauche ich ja die Schlüssel aus dem Array $alleTermine. Also brauche ich diesen schon vor dem Löschen des Datensatzes. Sonst sind die Schlüssel nicht vorhanden und es funktioniert nicht.

Ich habe jetzt eine Möglichkeit gefunden. Aber ich denke dies ist unsauber und nicht so gut.

Ich habe einfach nochmal zum Schluss alle Datensätze aus der DB-Tabelle gefetcht.

Was meint ihr dazu?

Gruß,

Stef

m.scatello · 7. April 2018

Doppelte Abfragen sind immer eine schlechte Idee.

Poste mal deinen kompletten Quellcode.

cottton · 7. April 2018

Ich sehe|verstehe gerade, dass Du mehrere löschen lassen willst.

Also ein mass-delete.

Dann solltest Du aber checkbox verwenden, und nicht button (es sei denn da steckt JS dahinter und Du kannst per button mehrere selecten - sieht aber nicht danach aus).

Also checkbox verwenden.

<input type='checkbox' name='ids[]' value='{ID ...}' />

Einfach mal im Netz suchen nach html multiple checkbox array.

Im Script checkst Du dann, ob es eine delete-Request ist $_POST[deleteTermin]

und gehst foreach $_POST[ids] as $id durch und löscht.

Stef · 7. April 2018

Ein mass-delete war direkt nicht geplant. Ich wollte auch, dass man die Termine wieder löschen kann.

Aber danke für den Tipp. Ich werde mich aufjedenfall jetzt daran orientieren. Ist viel besser als so viele Buttons. Wenn ich soweit fertig bin oder noch Probleme auftreten melde ich mich hier wieder.

Stef · 7. April 2018

cottton & m.scatello :

Vielen Dank für die Hilfe. Besoners danke Cottton für deinen super Tipp.

Es klappt ohne Probleme und ist auch viel besser.

PHP

if(isset($_POST['deleteTermin'])){

            
            $elements = isset($_POST['id']) ? $_POST['id'] : null;

            if(empty($elements)){
                $fehler = "<p class='fehler'>...</p>";
            }

            if(!isset($fehler)){

                foreach ($elements as $key => $terminId) {
                    $deleteTermin = $dbv->prepare("DELETE FROM termine WHERE terminId = :terminId");
                    $deleteTermin->execute(array(":terminId" => $terminId));
                }
            }
            
        
    }

Alles anzeigen

cottton · 7. April 2018

Nur am Rande:

man könnte jetzt eine delete-Query daraus machen.

Das hätte aber den Nachteil, dass Du nicht weißt, welche id nicht gelöscht werden konnte

(wenn zB nicht existiert ...).

Wenn aber sehr viele Einträge gelöscht werden können, dann wäre das aber zu empfehlen. Sonst rennt der SQL Server sich müde

Aber auf jeden Fall per transaction:

PHP

$dbh->beginTransaction();
try {
    // siehe https://stackoverflow.com/a/2373756/3411766
    $questionMarks = implode(',', array_fill(0, count($elements), '?'));
    // erstellt ?,?,? (nach anzahl der ids in $elements)

    $stmt = $dbh->prepare("DELETE FROM `foo` WHERE `id` IN ({$questionMarks});");

    // array_values stellt sicher, das wir numerische, fortlaufende keys haben (0 => , ... 1 =>, ...)
    $deleteTermin->execute(array_values($elements));


    // wenn du PDO error handling auf exception gesetzt hast,
    //      $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
    // dann fliegt bei ->execute eine exception und wir laden in dem catch block
    // sonst müsstest du prüfen, ob ->execute false war und eine eigene exception werfen.
    $dbh->commit();
} catch (Exception $e) {
    $dbh->rollBack();
}

Alles anzeigen

Stef · 8. April 2018

Zitat

Sonst rennt der SQL Server sich müde

Ja da hast du recht. Je nachdem wie viele checkboxen ausgewählt sind erfolgen Anfragen.

Danke für den Ratschlag. Dann hat man nur 1ne Query welche nur einmal ausgeführt wird und trotzdem alle ausgwählten Einträge löscht.

Ich werde es mal testen.

Löschen eines DB-Eintrages und Live Aktualisierung bei der Seite

Jetzt mitmachen!

Teilen