Daten aus Formular in DB schreiben

[mmoerth]

Ich habe mal folgendes zusammengeschrieben:

try {
                $pdotransaction = new PDO('mysql:host=localhost;dbname=autorenprojekt', 'root', 'root');
                $pdotransaction->query("set names 'utf8'");
}
catch(PDOException $e)
{
                die($e->getMessage());
}

 
 
 
        # pdo - easy way to insert form values into database
 
 
 
                // query
                $sql = "INSERT INTO buecher (id, username, buchtitel, link) VALUES ('NULL', $username, $titel, $link)";
                $stm = $pdotransaction->prepare($sql);
                $insert = $stm->execute($data);
 
                if( ! $insert ) die('insert failed !'); 

Doch wird kein Wert in die Datenbank geschrieben.

Die einzige Ausgabe: insert failed !

Wo liegt hierbei der Fehler?



lg mmoerth

[VPh]

Strings müssen in mysql von Hochkommas umgeben sein.
Prepared Statements werden anders genutzt, bzw du nutzt ihren Vorteil nicht.
Gibt es $data?

Ehrlich gesagt, das ist nichts wofür man in ein Forum posten sollte. Investier doch lieber mal eine Stunde um selbst herauszufinden warum etwas nicht funktioniert.

[hausl]

Investier doch lieber mal eine Stunde um selbst herauszufinden warum etwas nicht funktioniert.

Debugging-Leitfaden dafür:

PHP: http://php-de.github.io/jumpto/leitfaden/
SQL: http://php-de.github.io/jumpto/sql/

LG

[mmoerth]

$data sollte eigentlich $sql sein. Habe das mal korrigiert.
Nun habe ich die Strings in Hochkommas umgeben. Das heißt, dass der Abschnitt nun so aussieht:

$sql = "INSERT INTO buecher ('id', 'username', 'buchtitel', 'link') VALUES ('NULL', $username, $titel, $link)";
                $stm = $pdotransaction->prepare($sql);
                $insert = $stm->execute($sql); 

Oder habe ich einen Denkfehler?

[hausl]

"INSERT INTO buecher ('id', 'username', 'buchtitel', 'link') VALUES ('NULL', $username, $titel, $link)" 

Nicht die Spaltennamen in Strings, die (maximal) in `backticks`. Nur die Strings. Schau dir mal Beispiele genau an, oder nutze besser die schon ewähnten prep. Statements.

[tk1234]

Neben dem bereits gesagten zu den Anführungszeichen und der Tatsache dass du das Prinzip von prepared Statements noch nicht verstanden hast:

$sql = "INSERT INTO buecher ('id', 'username', 'buchtitel', 'link') VALUES ('NULL', $username, $titel, $link)"; 

Als Wert für die Spalte »id« musst du NULL übergeben, nicht den String »NULL« (also ohne Anführungszeichen), alternativ die Spalte einfach ganz weg lassen.

[lstegelitz]

Vor allem scheinst du zu versuchen, PHP Variablen ($username, $titel, $link) im SQL String einzufügen... PHP müsste das bereits anmeckern (error_reporting erhöhen!), wenn es diese Variablen nicht gibt. Platzhalter für prepared statements sollte man definitiv anders nennen...