Datenbankeinträge per Link löschen!

Sp1n · 11.08.2009, 09:28

Hallo Leute!
Ich weis ich nerve, aber ich bekomm da etwas nicht ganz hin!

Ich habe eine Übersichts-Datei wo meine Projekte aufgerufen werden, neben jedem Projekt befindet sich ein Button der diesen Eitnrag löscht von der Datenbank!
Der Link sieht so aus: "http://linux-dev/artika_form/delete.php?id=11" natürlich ist die ID vom Projekt abhängig!

Jetzt habe ich die delete.php erstellt die so aussieht:

Code:

PHP-Code:

  <?php

$host = "localhost";
$user = "dbuser";
$password = "dbpass";
$dbname = "***";

$dbverbindung = mysql_connect ($host, $user, $password);
mysql_select_db($dbname);

 $search_id = $_GET[ 'id' ];
    echo $search_id;
    if ($search_id  != "0" && $search_id !="") 

$loeschen = mysql_query('DELETE FROM Projekt WHERE id =$id');

    if (mysql_query($loeschen)) {
print ("Ihr Eingabe wurde erfolgreich geändert!.");
} else {
print ("Es trat ein Fehler auf! Bitte wenden Sie sich an den Administrator!.");
}
mysql_close ($dbverbindung);
$abfrage = "SELECT * FROM Projekt";
?>

Ich weis das die glaube komplett falsch ist aber könnt ihr mir bitte helfen?
Mit Zitat antworten

Edit Chriz: Datenbankname entfernt

taurus · 11.08.2009, 09:48

ohne dein code jetzt genau zu studieren.. welcher fehler kommt denn?
und auf die schnelle... das hier

PHP-Code:

  ('DELETE FROM Projekt WHERE id =$id');

ist falsch und ausserdem nicht sicher.
prüfe die id vorher ob es eine zahl is und lass es erst dann in die db stichwort sql injection
korrekt wäre diese form

PHP-Code:

  "DELETE FROM ProjektWHERE f_id = " . $id . "
                 LIMIT 1";

oder anstatt " halt '

Asipak · 11.08.2009, 09:48

PHP-Code:

  $search_id = $_GET[ 'id' ];

// ...

$loeschen = mysql_query("DELETE FROM Projekt WHERE id = " . $id);

Fällt dir da nichts auf?

Sp1n · 11.08.2009, 09:55

Zitat:

Zitat von Asipak

PHP-Code:

  $search_id = $_GET[ 'id' ];

 
// ...

 
$loeschen = mysql_query("DELETE FROM Projekt WHERE id = " . $id);

Fällt dir da nichts auf?

?

Gemouen · 11.08.2009, 09:55

Hmm... warum führst du die mysql_query zweimal aus ? einmal hier:

PHP-Code:

  $loeschen = mysql_query('DELETE FROM Projekt WHERE id =$id');

Da verstehe iochs noch, aber dann machst du dasselbe nochmal mit dem Rückgabewert ?

PHP-Code:

  if (mysql_query($loeschen)) {

?

Entferne esrtmal das. (Warum setzt du eigentlich $abfrage, nachdem du die Datenbankverbindung geschlossen hast ?)

Mfg
Gemouen

Edit:
wtf!? War ich da lahm.. hab das mit der id jetzt rausgenommen

Sp1n · 11.08.2009, 09:56

Zitat:

Zitat von taurus

ohne dein code jetzt genau zu studieren.. welcher fehler kommt denn?
und auf die schnelle... das hier

PHP-Code:

  ('DELETE FROM Projekt WHERE id =$id');

ist falsch und ausserdem nicht sicher.
prüfe die id vorher ob es eine zahl is und lass es erst dann in die db stichwort sql injection
korrekt wäre diese form

PHP-Code:

  "DELETE FROM ProjektWHERE f_id = " . $id . "

                 LIMIT 1";

oder anstatt " halt '

Fehlermeldung kommt natürlich

PHP-Code:

  else {

print ("Es trat ein Fehler auf! Bitte wenden Sie sich an den Administrator!.");

}

Asipak · 11.08.2009, 09:56

Zitat:

Zitat von Sp1n

?

PHP-Code:

  // Ausgabe: false

if ('$search_id' == '$id') {

  return true;

} else {

  return false;

}

Sp1n · 11.08.2009, 09:58

Zitat:

Zitat von Gemouen

Hmm... warum führst du die mysql_query zweimal aus ? einmal hier:

PHP-Code:

  $loeschen = mysql_query('DELETE FROM Projekt WHERE id =$id');

Da verstehe iochs noch, aber dann machst du dasselbe nochmal mit dem Rückgabewert ?

PHP-Code:

  if (mysql_query($loeschen)) {

?

Entferne esrtmal das. (Warum setzt du eigentlich $abfrage, nachdem du die Datenbankverbindung geschlossen hast ?)

Mfg
Gemouen

Edit:
wtf!? War ich da lahm.. hab das mit der id jetzt rausgenommen

Habe ich entfernt!
Als Info!
Wenn ich den Link zum löschen anklicken bekomm ich nur die Ausgabe "11" also die ID die ich durch echo $search_id; ausgeben lasse!

Gemouen · 11.08.2009, 09:59

Vllt. solltest du mal schauen, was in deiner Abfrage genau für Werte benutzt werden ?...

Sp1n · 11.08.2009, 10:01

ich bin glaube zu dumm! ich check nix mehr!

Mein Code sieht jetzt so aus

PHP-Code:

  <?php

$host = "localhost";
$user = "dbuser";
$password = "dbpass";
$dbname = "artika_form";

$dbverbindung = mysql_connect ($host, $user, $password);
mysql_select_db($dbname);

 $search_id = $_GET[ 'id' ];
    echo $search_id;
    if ($search_id  != "0" && $search_id !="") 

$loeschen = mysql_query('DELETE FROM Projekt WHERE id =$id');

    $abfrage = "SELECT * FROM Projekt";
mysql_close ($dbverbindung);

?>

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
erst beim zweiten Login funktioniert Link	scarry_77	PHP Tipps 2009	5	14.04.2009 00:35
Klick auf Seitenobjekt (Link mit onClick Ereigniss) mit JScript simulieren	Stephan_87	JavaScript, Ajax und mehr	1	22.12.2008 21:50
[Erledigt] bild aus db und verzeichn is mit einem link löschen!	litterauspirna	PHP Tipps 2008	3	15.09.2008 15:46
$_POST Daten löschen ohne über einen Link zu gehen	Blank	PHP Tipps 2006	10	29.05.2006 17:00
Datei löschen per IMG link	Soese	PHP Tipps 2006	3	20.02.2006 23:37
[Erledigt] Löschen aus Tabelle mit ID-Übergabe		Datenbanken	8	20.10.2005 22:10
Daten aus MySQL-abfrage zum Link mit id zusammensetzen	maccoder	Datenbanken	2	13.09.2005 16:49
Vor dem löschen abfragen ob man wirklich löschen will		PHP Tipps 2005-2	1	05.09.2005 11:54
Suche Tipps für Persormance-Steigerung (Geld für Nützliches)		Beitragsarchiv	18	16.08.2005 10:57
Link zum löschen eines Datensatzes automatisch generieren		PHP Tipps 2005	2	27.04.2005 19:38
Variabel in link übergeben und Datensatz löschen		Datenbanken	4	17.12.2004 13:08
Über Link einen Zeile aus Tabelle löschen	Calli	PHP Tipps 2004-2	24	03.12.2004 13:35
[Erledigt] datensatz löschen		Datenbanken	3	16.08.2004 10:03
PHP Link ???		PHP Tipps 2004	4	25.06.2004 17:24
Datensatz löschen: Datenbankeinträge per Link löschen!?		PHP Tipps 2004	9	09.06.2004 01:44

11.08.2009, 09:28
Sp1n Erfahrener Benutzer Registriert seit: 30.07.2009 Beiträge: 276 PHP-Kenntnisse: Anfänger	Datenbankeinträge per Link löschen! Hallo Leute! Ich weis ich nerve, aber ich bekomm da etwas nicht ganz hin! Ich habe eine Übersichts-Datei wo meine Projekte aufgerufen werden, neben jedem Projekt befindet sich ein Button der diesen Eitnrag löscht von der Datenbank! Der Link sieht so aus: "http://linux-dev/artika_form/delete.php?id=11" natürlich ist die ID vom Projekt abhängig! Jetzt habe ich die delete.php erstellt die so aussieht: Code: PHP-Code: <?php $host = "localhost"; $user = "dbuser"; $password = "dbpass"; $dbname = "**"; $dbverbindung = mysql_connect ($host, $user, $password); mysql_select_db($dbname); $search_id = $_GET[ 'id' ]; echo $search_id; if ($search_id != "0" && $search_id !="") $loeschen = mysql_query('DELETE FROM Projekt WHERE id =$id'); if (mysql_query($loeschen)) { print ("Ihr Eingabe wurde erfolgreich geändert!."); } else { print ("Es trat ein Fehler auf! Bitte wenden Sie sich an den Administrator!."); } mysql_close ($dbverbindung); $abfrage = "SELECT FROM Projekt"; ?> Ich weis das die glaube komplett falsch ist aber könnt ihr mir bitte helfen? Mit Zitat antworten Edit Chriz: Datenbankname entfernt Geändert von Chriz (11.08.2009 um 09:38 Uhr).


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

11.08.2009, 09:48
taurus Erfahrener Benutzer Registriert seit: 01.11.2008 Beiträge: 825 PHP-Kenntnisse: Anfänger	ohne dein code jetzt genau zu studieren.. welcher fehler kommt denn? und auf die schnelle... das hier PHP-Code: `('DELETE FROM Projekt WHERE id =$id');` ist falsch und ausserdem nicht sicher. prüfe die id vorher ob es eine zahl is und lass es erst dann in die db stichwort sql injection korrekt wäre diese form PHP-Code: `"DELETE FROM ProjektWHERE f_id = " . $id . " LIMIT 1";` oder anstatt " halt '

11.08.2009, 09:48
Asipak Moderator Registriert seit: 18.07.2005 Beiträge: 4.072	PHP-Code: `$search_id = $_GET[ 'id' ]; // ... $loeschen = mysql_query("DELETE FROM Projekt WHERE id = " . $id);` Fällt dir da nichts auf?

11.08.2009, 09:55
Gemouen Neuer Benutzer Registriert seit: 01.02.2009 Beiträge: 23	Hmm... warum führst du die mysql_query zweimal aus ? einmal hier: PHP-Code: `$loeschen = mysql_query('DELETE FROM Projekt WHERE id =$id');` Da verstehe iochs noch, aber dann machst du dasselbe nochmal mit dem Rückgabewert ? PHP-Code: `if (mysql_query($loeschen)) {` ? Entferne esrtmal das. (Warum setzt du eigentlich $abfrage, nachdem du die Datenbankverbindung geschlossen hast ?) Mfg Gemouen Edit: wtf!? War ich da lahm.. hab das mit der id jetzt rausgenommen

11.08.2009, 09:59
Gemouen Neuer Benutzer Registriert seit: 01.02.2009 Beiträge: 23	Vllt. solltest du mal schauen, was in deiner Abfrage genau für Werte benutzt werden ?...

11.08.2009, 10:01
Sp1n Erfahrener Benutzer Registriert seit: 30.07.2009 Beiträge: 276 PHP-Kenntnisse: Anfänger	ich bin glaube zu dumm! ich check nix mehr! Mein Code sieht jetzt so aus PHP-Code: `<?php $host = "localhost"; $user = "dbuser"; $password = "dbpass"; $dbname = "artika_form"; $dbverbindung = mysql_connect ($host, $user, $password); mysql_select_db($dbname); $search_id = $_GET[ 'id' ]; echo $search_id; if ($search_id != "0" && $search_id !="") $loeschen = mysql_query('DELETE FROM Projekt WHERE id =$id'); $abfrage = "SELECT * FROM Projekt"; mysql_close ($dbverbindung); ?>`