Zitat:
|
Das Problem war bei der Abfrage und nicht weil ich fast nichts von deinen Tipps/Anmerkungen umgesetzt habe! Wir haben das so in der Schule gelernt und dort hat es einwandfrei funktioniert. Ich habe nur gefragt, was falsch daran ist und nicht was ich noch besser machen könnte, wie Schutz vor SQL Injections usw.
|
Das Problem ist nicht
nur bei der Abfrage! Deine IF-Bedingung mit if ($action="update") macht auch keinen Sinn...wo kommt $action her???
SQL Injections ist auch wichtig und gehört zu gutem Code mit dazu!
Zitat:
|
Ich habe nur gefragt, was falsch daran ist und nicht was ich noch besser machen könnte
|
Widerspricht sich, da du es besser machen musst, damit es funktioniert!
Zitat:
|
action="<?php echo $PHP_SELF; ?>"
|
-> so: action=""
-> so: <?php echo $showlogo; ?>
-> so: komplett weglassen!
Bei deinen mysql_query Geschichten ein " or die("Fehler: ".mysl_error()); " hinterhängen usw. Nochmal zur deiner IF Bedingung, mein Zitat:
Zitat:
|
Nicht wirklich! Dein SQL Statement wird jedesmal zuerst ausgeführt! Wenn du ein Update Statement ausführst, wird der SELECT Befehl ja beim neuladne praktisch auch "neugeladen", also macht die IF Bedingung hier keinen Sinn!
|
mfg Wolf29
P.S. Wenn Du meinen, einfachen Beispielcode anpasst, bist du in 10 Minuten mit allem fertig - nur als Tipp!