[Erledigt] SQL Update Problem HILFE!!! :-)

SilferSurver · 29.11.2008, 14:33

Hallo! ich hab hier irgend ein Fehler drin und komm nicht drauf!

Möchte was aus einer SQL auslesen,bearbeiten und wieder schreiben!

das auslesen und bearbeiten geht aber wenn ich dann auf aktualisieren gehe passiert gar nichts! auch kein Fehler! die Daten bleiben die selben!
Bin echt verzweifelt Bitte helft mir!

PHP-Code:

  $aktion = $_GET['aktion'];
$id = $_GET['id'];
if ($aktion == "Delete") {
 $query = "DELETE FROM kfz WHERE id = $id";
 $dummy = mysql_query($query);
 unset($bild, $hersteller, $fabrikat, $baujahr, $kilometer, $hubraum, $leistung, $kraftstoff, $getriebe, $schadstoff, $farbe, $beschreibung, $preis, $id);
}elseif ($aktion == "Edit") {
  $query = "SELECT * FROM kfz WHERE id = $id";
  $sql = mysql_query($query);
  $ds = mysql_fetch_object($sql);
  $id = $ds ->id;
  $bild = $ds ->bild;
  $hersteller = $ds ->hersteller;
  $fabrikat = $ds ->fabrikat;
  $baujahr = $ds ->baujahr;
  $kilometer = $ds ->kilometer;
  $hubraum = $ds ->hubraum;
  $leistung = $ds ->leistung;
  $kraftstoff = $ds ->kraftstoff;
  $getriebe = $ds ->getriebe;
  $schadstoff = $ds ->schadstoff;
  $farbe = $ds ->farbe;
  $beschreibung = $ds ->beschreibung;
  $preis = $ds ->preis;
 } else {
  $id = $_POST['id'];
  $bild = trim($_POST['bild']);
  $hersteller = trim($_POST['hersteller']);
  $fabrikat = trim($_POST['fabrikat']);
  $baujahr = trim($_POST['baujahr']);
  $kilometer = trim($_POST['kilometer']);
  $hubraum = trim($_POST['hubraum']);
  $leistung = trim($_POST['leistung']);
  $kraftstoff = trim($_POST['kraftstoff']);
  $getriebe = trim($_POST['getriebe']);
  $schadstoff = trim($_POST['schadstoff']);
  $farbe = trim($_POST['farbe']);
  $beschreibung = trim($_POST['beschreibung']);
  $preis = trim($_POST['preis']);

if (!empty($id)) 
   $query = "UPDATE kfz SET bild = '$bild', hersteller = '$hersteller', fabrikat = '$fabrikat', baujahr = '$baujahr', kilometer = '$kilometer', hubraum = '$hubraum', leistung = '$leistung', kraftstoff = '$kraftstoff', getriebe = '$getriebe', schadstoff = '$schadstoff', farbe = $farbe, beschreibung = '$beschreibung', preis = '$preis'WHERE id = $id";
   
   
   $dummy = mysql_query($query);
   unset($bild, $hersteller, $fabrikat, $baujahr, $kilometer, $hubraum, $leistung, $kraftstoff, $getriebe, $schadstoff, $farbe, $beschreibung, $preis, $id);
}
?>

Vielen Dank schonmal für Eure Hilfe!

PS: ups denke bin im falschen thread gelandet, denke das in den sql thread!

Könnte bitte ein Admin den Thread verschieben! Danke

caipi · 29.11.2008, 14:39

ich hätte erst lesen, dann antworten sollen. sry

David · 29.11.2008, 14:42

Damit wir überhaupt erstmal anfangen können bzw Du anfangen kannst, den Fehler zu beschreiben, ein paar kleine (immer wiederverwendbare) Änderungen am Skript.
Ganz am Anfang, nach dem ersten <?php

PHP-Code:

  error_reporting(E_ALL); ini_set('display_errors', 1);

einfügen. Damit sendet PHP alle Fehler-/Warn-/Hinweismeldungen an den Client/Browser, so dass Du sie lesen kannst.

Jedes mysql_query(...); durch

PHP-Code:

  mysql_query($query) or die(mysql_error().': '.htmlentities($query));

ersetzen.
Wenn mysql_query() fehlschlägt wird der or die(...) Teil der Anweisung ausgeführt. die(...) gibt ... aus und beendet dann das Skript. mysql_error() liefert den Fehlertext der letzten MySQL-Operation zurück -> Fehlermeldung+SQL-Anweisung wird ausgegeben und das Skript stirbt.

Beides sind keine perfekten Mechanismen, aber erstmal besser als nichts.

p.s.: So, und jetzt suche ich mir den Link zur entsprechenden Wiki-Seite heraus

SilferSurver · 29.11.2008, 14:56

@ Caipi = Kein Problem trotzdem danke!

@ David = Mein Held

das hat die Lösung gebracht!!!!

der fehler lag an :

PHP-Code:

  schadstoff = '$schadstoff', farbe = $farbe, beschre

es muss :

PHP-Code:

  schadstoff = '$schadstoff', farbe = '$farbe', beschre

heissen!

da hab ich wohl nicht aufgepast! war auch schon spät!

David · 29.11.2008, 15:01

Deshalb immer Fehlerbehandlung einbauen.
Und immer ein Auge auf die Fehlermeldungen haben. Auf dem Entwicklungsserver kann man error_reporting=E_ALL und display_errors=On getrost in der php.ini setzen.
Es spart so viel Zeit. Edit: Und ich wirke so schlau, wenn ich sowas tippe *hust*

David · 29.11.2008, 15:15

So dann ist es jetzt Zeit (genau jetzt, nicht morgen, nicht nächste Woche, jetzt

) das Problem mit den sql injections anzupacken.
Und gerade wenn Du auf _GET setzt, ist das Thema noch brennender.

SQL-Injection – Wikipedia
www.php.net/security.database.sql-injection

und danach sollte dann "Warum man DELETE nicht an _GET knüpfen sollte" auf der Liste stehen...

nikosch · 29.11.2008, 15:50

Ehrlich. Spar Dir dieses blöde

Zitat:

HILFE!!!

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
[Erledigt] sql update problem	malisha	Datenbanken	3	31.10.2008 16:36
Problem mit Update	taucher	PHP Tipps 2008	6	01.07.2008 05:50
mysql problem: mysql update wird nicht ausgeführt	schmiddy	PHP Tipps 2007	4	27.12.2005 20:35
Mysql update problem		PHP Tipps 2007	8	21.12.2005 15:56
Update Problem (Postgres)	DER_Brain	PHP Tipps 2005-2	2	19.10.2005 15:50
Problem mit osCommerce brauche dringend Hilfe...		PHP Tipps 2005	12	13.05.2005 08:40
Brauche hilfe bei komplexem problem mit PHP und MySQL	Dragon26mFR	PHP Tipps 2005	2	02.05.2005 13:58
[Erledigt] Hilfe - Problem mit Warenkorb		PHP Tipps 2004	1	26.11.2004 11:32
Problem bei MYSQL / Update Resource #id Fehler		PHP Tipps 2004	2	29.09.2004 13:09
PHP Update Problem!		PHP Tipps 2004	1	28.09.2004 23:13
[Erledigt] UPDATE = NO Update aber kein fehler..dringend hilfe suchend		Datenbanken	2	25.09.2004 16:28
[Erledigt] UPDATE Syntax problem		Datenbanken	6	17.08.2004 16:18
mysql !!! Update problem!!!		PHP Tipps 2004	2	18.07.2004 20:41
[Erledigt] brauche Hilfe beim update...	18inch	Datenbanken	3	07.07.2004 17:04
[Erledigt] Problem bei Tabellenerstellung, bitte um Hilfe		Datenbanken	4	01.07.2004 15:19

29.11.2008, 14:33
SilferSurver Neuer Benutzer Registriert seit: 29.11.2008 Beiträge: 10	[Erledigt] SQL Update Problem HILFE!!! :-) Hallo! ich hab hier irgend ein Fehler drin und komm nicht drauf! Möchte was aus einer SQL auslesen,bearbeiten und wieder schreiben! das auslesen und bearbeiten geht aber wenn ich dann auf aktualisieren gehe passiert gar nichts! auch kein Fehler! die Daten bleiben die selben! Bin echt verzweifelt Bitte helft mir! PHP-Code: $aktion = $_GET['aktion']; $id = $_GET['id']; if ($aktion == "Delete") { $query = "DELETE FROM kfz WHERE id = $id"; $dummy = mysql_query($query); unset($bild, $hersteller, $fabrikat, $baujahr, $kilometer, $hubraum, $leistung, $kraftstoff, $getriebe, $schadstoff, $farbe, $beschreibung, $preis, $id); }elseif ($aktion == "Edit") { $query = "SELECT * FROM kfz WHERE id = $id"; $sql = mysql_query($query); $ds = mysql_fetch_object($sql); $id = $ds ->id; $bild = $ds ->bild; $hersteller = $ds ->hersteller; $fabrikat = $ds ->fabrikat; $baujahr = $ds ->baujahr; $kilometer = $ds ->kilometer; $hubraum = $ds ->hubraum; $leistung = $ds ->leistung; $kraftstoff = $ds ->kraftstoff; $getriebe = $ds ->getriebe; $schadstoff = $ds ->schadstoff; $farbe = $ds ->farbe; $beschreibung = $ds ->beschreibung; $preis = $ds ->preis; } else { $id = $_POST['id']; $bild = trim($_POST['bild']); $hersteller = trim($_POST['hersteller']); $fabrikat = trim($_POST['fabrikat']); $baujahr = trim($_POST['baujahr']); $kilometer = trim($_POST['kilometer']); $hubraum = trim($_POST['hubraum']); $leistung = trim($_POST['leistung']); $kraftstoff = trim($_POST['kraftstoff']); $getriebe = trim($_POST['getriebe']); $schadstoff = trim($_POST['schadstoff']); $farbe = trim($_POST['farbe']); $beschreibung = trim($_POST['beschreibung']); $preis = trim($_POST['preis']); if (!empty($id)) $query = "UPDATE kfz SET bild = '$bild', hersteller = '$hersteller', fabrikat = '$fabrikat', baujahr = '$baujahr', kilometer = '$kilometer', hubraum = '$hubraum', leistung = '$leistung', kraftstoff = '$kraftstoff', getriebe = '$getriebe', schadstoff = '$schadstoff', farbe = $farbe, beschreibung = '$beschreibung', preis = '$preis'WHERE id = $id"; $dummy = mysql_query($query); unset($bild, $hersteller, $fabrikat, $baujahr, $kilometer, $hubraum, $leistung, $kraftstoff, $getriebe, $schadstoff, $farbe, $beschreibung, $preis, $id); } ?> Vielen Dank schonmal für Eure Hilfe! PS: ups denke bin im falschen thread gelandet, denke das in den sql thread! Könnte bitte ein Admin den Thread verschieben! Danke Geändert von SilferSurver (29.11.2008 um 14:40 Uhr).


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

29.11.2008, 14:39
caipi Neuer Benutzer Registriert seit: 02.10.2008 Beiträge: 3	ich hätte erst lesen, dann antworten sollen. sry

29.11.2008, 14:42
David Erfahrener Benutzer Registriert seit: 05.09.2007 Beiträge: 5.044	Damit wir überhaupt erstmal anfangen können bzw Du anfangen kannst, den Fehler zu beschreiben, ein paar kleine (immer wiederverwendbare) Änderungen am Skript. Ganz am Anfang, nach dem ersten <?php PHP-Code: `error_reporting(E_ALL); ini_set('display_errors', 1);` einfügen. Damit sendet PHP alle Fehler-/Warn-/Hinweismeldungen an den Client/Browser, so dass Du sie lesen kannst. Jedes mysql_query(...); durch PHP-Code: `mysql_query($query) or die(mysql_error().': '.htmlentities($query));` ersetzen. Wenn mysql_query() fehlschlägt wird der or die(...) Teil der Anweisung ausgeführt. die(...) gibt ... aus und beendet dann das Skript. mysql_error() liefert den Fehlertext der letzten MySQL-Operation zurück -> Fehlermeldung+SQL-Anweisung wird ausgegeben und das Skript stirbt. Beides sind keine perfekten Mechanismen, aber erstmal besser als nichts. p.s.: So, und jetzt suche ich mir den Link zur entsprechenden Wiki-Seite heraus

29.11.2008, 14:56
SilferSurver Neuer Benutzer Registriert seit: 29.11.2008 Beiträge: 10	Vielen Dank! @ Caipi = Kein Problem trotzdem danke! @ David = Mein Held das hat die Lösung gebracht!!!! der fehler lag an : PHP-Code: `schadstoff = '$schadstoff', farbe = $farbe, beschre` es muss : PHP-Code: `schadstoff = '$schadstoff', farbe = '$farbe', beschre` heissen! da hab ich wohl nicht aufgepast! war auch schon spät!

29.11.2008, 15:01
David Erfahrener Benutzer Registriert seit: 05.09.2007 Beiträge: 5.044	Deshalb immer Fehlerbehandlung einbauen. Und immer ein Auge auf die Fehlermeldungen haben. Auf dem Entwicklungsserver kann man error_reporting=E_ALL und display_errors=On getrost in der php.ini setzen. Es spart so viel Zeit. Edit: Und ich wirke so schlau, wenn ich sowas tippe hust Geändert von David (29.11.2008 um 15:12 Uhr).

29.11.2008, 15:15
David Erfahrener Benutzer Registriert seit: 05.09.2007 Beiträge: 5.044	So dann ist es jetzt Zeit (genau jetzt, nicht morgen, nicht nächste Woche, jetzt ) das Problem mit den sql injections anzupacken. Und gerade wenn Du auf _GET setzt, ist das Thema noch brennender. SQL-Injection – Wikipedia www.php.net/security.database.sql-injection und danach sollte dann "Warum man DELETE nicht an _GET knüpfen sollte" auf der Liste stehen...