Error in der SQL syntax

[Sr2406]

^^ Ok. Also:

Vorher:

PHP-Code:

//Anfrage zusammenstellen die an die DB geschickt werden soll
                                        $senden = "INSERT INTO `gaestebuch`
                                                            ( `vorname`, `nachname`, `inhalt`, `datum`, `zeit`)
                                                            VALUES(
                                                                '" .mysql_real_escape_string( $vorname ). "',
                                                                '" .mysql_real_escape_string( $nachname ). "',
                                                                '" .mysql_real_escape_string( $inhalt ). "',
                                                                '" .$datum. "',
                                                                '" .$zeit. "'
                                                                )";
                                    
                                    //Schickt die Anfrage an die DB und schreibt die Daten in die Tabelle
                                        mysql_query( $senden ) or die(mysql_error());

                                    // Pruefen ob der neue Datensatz tatsaechlich eingefuegt wurde
                                        if (mysql_affected_rows() == 1) 
                                        {
                                        echo mysql_error();
                                        include ('content.php');
                                        exit;
                                        }
                                        else 
                                        {
                                        echo "<h3>Es ist ein Fehler bei der Datenübertragung aufgetreten!</h3>"; 


Nacher:

PHP-Code:

//Anfrage zusammenstellen die an die DB geschickt werden soll
                                        $senden = "INSERT INTO `gaestebuch`
                                                            ( `vorname`, `nachname`, `inhalt`, `datum`, `zeit`)
                                                            VALUES(
                                                                '" .mysql_real_escape_string( $vorname ). "',
                                                                '" .mysql_real_escape_string( $nachname ). "',
                                                                '" .mysql_real_escape_string( $inhalt ). "',
                                                                '" .date("d.m.Y"). "',
                                                                '" .date("H:i"). "'
                                                                )";
                                    
                                    //Schickt die Anfrage an die DB und schreibt die Daten in die Tabelle
                                        mysql_query( $senden ) or die(mysql_error());

                                    // Pruefen ob der neue Datensatz tatsaechlich eingefuegt wurde
                                        if (mysql_affected_rows() == 1) 
                                        {
                                        echo mysql_error();
                                        include ('content.php');
                                        exit;
                                        }
                                        else 
                                        {
                                        echo "<h3>Es ist ein Fehler bei der Datenübertragung aufgetreten!</h3>"; 


Es geht um die letzten beiden Einträge in der Variablen $senden!

[vt1816]

Zitat von Sr2406

[..]
Es geht um die letzten beiden Einträge in der Variablen $senden!

.. und um es nun genau beurteilen zu können, bräuchten wir auch die Struktur der Tabelle gaestebuch.

[knight1]

@Sr2406
1. Wieso nutzt Du die date()-Funktion von PHP und nicht die von Datums- und Zeitfunktionen von MySQL? Guckst Du: http://dev.mysql.com/doc/refman/5.1/...functions.html
Tipp: now()
2. Entferne mal die unötigen Leerzeichen innerhalb des Statements.


Kai aka Knight1

[Sr2406]

Hier die Struktur:

id int(5)
vorname varchar(100)
nachname varchar(150)
datum varchar(12)
zeit varchar(8)
inhalt text

reicht euch das so?

Ich habe die date() funktion benutzt, weil ich so sehr einfach das Datumsformat bestimmen kann. Wie das mit MySQL geht muss ich mir dann noch mal anschauen.

@knight1: Hat das mit den leerzeichen entfernen irgend eine bewandnis? Ich fand das so eigendlich übersichtlich.

[knight1]

Zitat von Sr2406

Hier die Struktur:

id int(5)
vorname varchar(100)
nachname varchar(150)
datum varchar(12)
zeit varchar(8)
inhalt text

reicht euch das so?

Ich habe die date() funktion benutzt, weil ich so sehr einfach das Datumsformat bestimmen kann. Wie das mit MySQL geht muss ich mir dann noch mal anschauen.

@knight1: Hat das mit den leerzeichen entfernen irgend eine bewandnis? Ich fand das so eigendlich übersichtlich.

MySQL bietet sehr schöne Datums- und Zeit-Feldtypen.
Wie wäre es wenn Du dir das MySQL-Manual mal durchlesen würdest?
http://dev.mysql.com/doc/refman/5.1/...ime-types.html
Dort wird dann schon beim anlegen das jeweilige Format festgelegt.

Das hier " . ist wohl nicht so schlimm.
Aber das ganze einrücken innerhalb des Statements ist falsch. Ich hatte das auch mal. MySQL hängt sich da an den ganzen Leerzeichen bzw. Tabulatoren innerhalb der Value-Liste auf. Nicht immer ist die Übersichtlichkeit der funktional bessere Weg.


Kai aka Knight1

[Sr2406]

Zitat von knight1

Das hier " . ist wohl nicht so schlimm.
Aber das ganze einrücken innerhalb des Statements ist falsch. Ich hatte das auch mal. MySQL hängt sich da an den ganzen Leerzeichen bzw. Tabulatoren innerhalb der Value-Liste auf. Nicht immer ist die Übersichtlichkeit der funktional bessere Weg.

OK. Das mit den MySQL Datums und Zeitformaten werde ich mir mal anschauen und dann umsetzten.

Bezüglich des einrückens ist es also besser alle Werte im value() direkt hintereinander zu schreiben? Also ungefähr so?

PHP-Code:

$senden = "INSERT INTO `gaestebuch`( `vorname`, `nachname`, `inhalt`, `datum`, `zeit`)
VALUES('" .mysql_real_escape_string( $vorname ). "','" .mysql_real_escape_string( $nachname ). "','" .mysql_real_escape_string( $inhalt ). "','" .date("d.m.Y"). "','" .date("H:i"). "')"; 


[DokuLeseHemmung]

Aber das ganze einrücken innerhalb des Statements ist falsch. Ich hatte das auch mal. MySQL hängt sich da an den ganzen Leerzeichen bzw. Tabulatoren innerhalb der Value-Liste auf. Nicht immer ist die Übersichtlichkeit der funktional bessere Weg.

Sorry aber hier muss ich mal ernsthaft Einspruch erheben und reproduzierbare Beweise verlangen.

Bezüglich des einrückens ist es also besser alle Werte im value() direkt hintereinander zu schreiben? Also ungefähr so?

Lass dich nicht von so einem Unsinn kirre machen!
(es sei denn, mir wird das Gegenteil bewiesen)

[knight1]

Ja, das ist zumindest meine eigene Erfahrung.



Kai aka Knight1

[Sr2406]

Ich habe das bisher auch nur in meinem Beitrag so angepasst. Habe noch ein anderes Formular, welches bisher ohne Probleme läuft.

Um nochmal auf das Datums und Zeitformat zu kommen. Es werden dort ja keine Daten vom Benutzer eingetragen, also müsste es doch eigendlich egal sein, wie ich die Daten an die Datenbank übergebe, oder? Falls nicht, kann mir das irgendwer kurz erklären?

[knight1]

@DokuLeseHemmung
Auch wenn es nicht explizit in der Dokumentation drin steht, so habe ich aber trotzdem die Erfahrung gemacht dass sowas

$query = mysql_query("
INSERT INTO
`tabelle`
(
`feld1`,
`feld2`,
`feld3`,
`feld4`
) VALUES (
  'daten1',
  'daten2',
  'daten3',
  'daten4'
");

wegen der Leerzeichen in der Value-Liste nicht funktioniert. Der wird dann wegen der Leerzeichen meckern.

Kai aka Knight1