keine Eintragung in MySQL-Tabelle

andy74 · 09.01.2006, 21:47

Hallo,

ich habe in nachstehendem Script einen Fehler. Ich kann zwar die eine Tabelle (nl_email) abfragen und bekomme die richtigen Daten, kann aber keine Daten in die Tabelle (tmp_nl_email) eintragen. Woran könnte das liegen?

PHP-Code:


			
    else {

        $email = ($_POST['email']);

        $name = ($_POST['name']);

        $handy = ($_POST['handy']);

    // Abfrage, ob Email-Adresse bereits existiert

    $query = "SELECT * FROM nl_email Where email = '$email'";    

    $rs = mysql_query($query, $conn);

    if (mysql_num_rows($rs) > 0)     {

        echo 'Sie haben sich bereits angemeldet';

    } else {

        $query = "INSERT INTO tmp_nl_email(email,handy,name) VALUES ($email,$handy,$name)";

        echo 'Vielen Dank für die Registrierung';

    }

Lars · 09.01.2006, 21:51

Du speichert den Query zwar in $query, führst ihn aber nicht mitteln mysql_query() aus.

09.01.2006, 21:53

Ich bin mir nicht sicher, ob das der Fehler ist, aber muss es nicht

PHP-Code:


			
VALUES ('$email', '$handy', '$name')

heissen?

mfg flupsi

Lars · 09.01.2006, 21:57

Das kommt ncoh dazu.

Außerdem solltest du auch SQL-Injection verhindern.

andy74 · 09.01.2006, 21:57

jepp, beide Tipps haben weitergeholfen. Ich sage wieder einmal: Recht herzlichen Dank.

Das mit dem mysql_query hätte mir selbst auffallen müssen. Werde zukünftig noch genauer hinschauen - versprochen.

Jann Hendrik · 10.01.2006, 07:53

ok, dann lege ich noch einen drauf

du solltest dann auch stets mysql_error() verwenden! Mögliche Fehler werden dir dann ausgegeben.

Lars · 10.01.2006, 13:17

Optimalerweise so:

PHP-Code:


			
$query = "{query}";

if ( !( $result = @mysql_query($query, $db) ) )

{

    trigger_error(mysql_errno().': '.mysql_error(), E_USER_ERROR);

}

Noch jemand? :]

Jann Hendrik · 10.01.2006, 16:38

ja ich!

warum nicht mit einem einfachen

mysql_query($sql) or die(...);

???
warum über die trigger_error() ?

Jojo · 10.01.2006, 18:52

Warum willst du das Script abbrechen, "nur" weil deine MySQL-Query nicht funktioniert?
Und warum das ganze nicht noch ein wenig formatieren und mit der Möglichkeit versehen, die Fehlerausgabe zu optimieren bzw. zu personalisieren?
@Lars
coole Fehlerbehandlung.
Kannte ich noch nich.
Danke dafür

Jann Hendrik · 12.01.2006, 08:38

Zitat:

Zitat von Jojo

Warum willst du das Script abbrechen, "nur" weil deine MySQL-Query nicht funktioniert?

Wahrscheinlich, weil ich mir nie darüber weitere Gedanken gemacht habe und die die()-Art mir bisher ausreichend gefallen hat...

ich werde die Lars-Methode mal ausprobieren!

09.01.2006, 21:47	Nach oben #1
andy74 Benutzer Registriert seit: 27.05.2005 Beiträge: 34	keine Eintragung in MySQL-Tabelle Hallo, ich habe in nachstehendem Script einen Fehler. Ich kann zwar die eine Tabelle (nl_email) abfragen und bekomme die richtigen Daten, kann aber keine Daten in die Tabelle (tmp_nl_email) eintragen. Woran könnte das liegen? PHP-Code: else { $email = ($_POST['email']); $name = ($_POST['name']); $handy = ($_POST['handy']); // Abfrage, ob Email-Adresse bereits existiert $query = "SELECT * FROM nl_email Where email = '$email'"; $rs = mysql_query($query, $conn); if (mysql_num_rows($rs) > 0) { echo 'Sie haben sich bereits angemeldet'; } else { $query = "INSERT INTO tmp_nl_email(email,handy,name) VALUES ($email,$handy,$name)"; echo 'Vielen Dank für die Registrierung'; }

09.01.2006, 21:51	Nach oben #2
Lars me pro ok? Registriert seit: 07.09.2005 Ort: Pulheim bei Köln Beiträge: 964	Du speichert den Query zwar in $query, führst ihn aber nicht mitteln mysql_query() aus. __________________ Gedanken aus Draht stricken einen Zaun.

09.01.2006, 21:53	Nach oben #3
flupsi Gast Beiträge: n/a	Ich bin mir nicht sicher, ob das der Fehler ist, aber muss es nicht PHP-Code: `VALUES ('$email', '$handy', '$name')` heissen? mfg flupsi

09.01.2006, 21:57	Nach oben #4
Lars me pro ok? Registriert seit: 07.09.2005 Ort: Pulheim bei Köln Beiträge: 964	Das kommt ncoh dazu. Außerdem solltest du auch SQL-Injection verhindern. __________________ Gedanken aus Draht stricken einen Zaun.

10.01.2006, 07:53	Nach oben #6
Jann Hendrik Projektleiter Registriert seit: 02.12.2004 Ort: Wildeshausen Beiträge: 2.225	ok, dann lege ich noch einen drauf du solltest dann auch stets mysql_error() verwenden! Mögliche Fehler werden dir dann ausgegeben. __________________ Umfragen: bitte beachten: Vorschläge für künftige Umfragen Woher weißt du vom developers-guide? Wenn du dich in ein interessantes Thema eingearbeitet hast, dann lass andere daran teilhaben! Schreibe ein Tutorial und beschreibe, wie es geht, was nicht klappt, wo man aufpassen muss usw. Danke!

09.01.2006, 21:57	Nach oben #5
andy74 Benutzer Registriert seit: 27.05.2005 Beiträge: 34	jepp, beide Tipps haben weitergeholfen. Ich sage wieder einmal: Recht herzlichen Dank. Das mit dem mysql_query hätte mir selbst auffallen müssen. Werde zukünftig noch genauer hinschauen - versprochen.

10.01.2006, 13:17	Nach oben #7
Lars me pro ok? Registriert seit: 07.09.2005 Ort: Pulheim bei Köln Beiträge: 964	Optimalerweise so: PHP-Code: `$query = "{query}"; if ( !( $result = @mysql_query($query, $db) ) ) { trigger_error(mysql_errno().': '.mysql_error(), E_USER_ERROR); }` Noch jemand? :] __________________ Gedanken aus Draht stricken einen Zaun.

10.01.2006, 16:38	Nach oben #8
Jann Hendrik Projektleiter Registriert seit: 02.12.2004 Ort: Wildeshausen Beiträge: 2.225	ja ich! warum nicht mit einem einfachen mysql_query($sql) or die(...); ??? warum über die trigger_error() ? __________________ Umfragen: bitte beachten: Vorschläge für künftige Umfragen Woher weißt du vom developers-guide? Wenn du dich in ein interessantes Thema eingearbeitet hast, dann lass andere daran teilhaben! Schreibe ein Tutorial und beschreibe, wie es geht, was nicht klappt, wo man aufpassen muss usw. Danke!

10.01.2006, 18:52	Nach oben #9
Jojo Irgendwas mit e Registriert seit: 26.08.2005 Ort: Mannheim Beiträge: 393	Warum willst du das Script abbrechen, "nur" weil deine MySQL-Query nicht funktioniert? Und warum das ganze nicht noch ein wenig formatieren und mit der Möglichkeit versehen, die Fehlerausgabe zu optimieren bzw. zu personalisieren? @Lars coole Fehlerbehandlung. Kannte ich noch nich. Danke dafür __________________ In the beginning was the word and the word was content-type: plain/text heute code ich, morgen debug ich und uebermorgen cast ich die koenigin auf int

Aktive Benutzer in diesem Thema: 1 (Registrierte Benutzer: 0, Gäste: 1)

Themen-Optionen
Druckbare Version zeigen Diese Seite per E-Mail verschicken

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
[Suche] MySQL Tool ähnlich MySQL Front	ex³	Gesuche	5	22.12.2006 18:52
ssh tunnel zu einer mysql datenbank	beny_mcde	Datenbanken	4	07.06.2006 16:05
MySQL 5.1 kommt in die Beta-Phase	Ben	Nachrichten	1	02.03.2006 14:31
MySQL: Abfrage aus zweiter Tabelle.	Sebastian	PHP-Programmierung	6	16.12.2005 00:01
[SQL] Zufallsdatensatz auslesen - MySQL 3.23	Ben	Tutorials	0	16.03.2005 00:06