ID löschen in MySQL

RockHound · 19.08.2011, 19:05

Hallo zusammen!
Ich fange mal so an: google, foren, 10.000 versuche - hat alles nichts gebracht.
Auch hier habe ich fleißig geschaut, aber die codes unterscheiden sich immer wieder auf's neue - insofern gab es keine Lösung für mich.

Wie die Überschrift schon sagt, möchte ich per 'ID' löschen,
das Formular soll den Wert mit 'POST' an eine andere PHP übergeben,
aber wenn ich den Datensatz neu abrufe ist nichts gelöscht.

--Datei: del.php--

PHP-Code:

  <form method="post" action="l.php">

  <table width="351" border="0">

    <tr>

      <td valign="bottom">Welche ID soll gelöscht werden?<br></td>

      <td valign="top"><br></td>

    </tr>

    <tr>

      <td valign="bottom"><input type="text" name="nr" value="" size="3" />

      <input type="submit" name="button" value="OK" /></td>

    </tr>    

  </table>

  </form>

--Datei: l.php--

PHP-Code:

  <?php

$host = "localhost";     

$user = "****";         

$pass = "****";         

$dbase = "fleet";         

 
$connection = mysql_connect("$host" , "$user" , "$pass") 

              OR die ("Verbindung zur Datenbank nicht m&ouml;lich.");

$db = mysql_select_db($dbase , $connection) 

              OR die ("<h1>Auswahl der Datenbank nicht m&ouml;glich.<br>Datenbank existiert nicht oder keine definiert!<h1>");      

 
 

    $nr = $_POST["nr"];

 
  

$nr = "DELETE FROM wood WHERE id = '$nr'";

$loesch = mysql_query($loeschen);

 
mysql_close();

?>

Ich hoffe ihr könnt mir helfen und würde mich freuen,
wenn die Auflistung brauchbar war.

LG

Capfly · 19.08.2011, 19:15

$loeschen stimmt nicht im query

smon · 19.08.2011, 19:21

Hallo RockHound,

das SQL-Statement zum Löschen wird überhaupt nicht ausgeführt. Du übergibst eine nicht-existende Variable ($loeschen) an die mysql_query(...)-Funktion, dein Statement ist allerdings in der Variable $nr gespeichert.

Darüber hinaus ist, bzw. wäre, wenn das SQL-Statement so ausgeführt würde, der gepostete Code anfällig für SQL-Injection-Angriffe, da eine Benutzereingabe ungefiltert in das SQL-Statement eingefügt wird.

Viele Grüße
smon

eagle275 · 20.08.2011, 00:19

na, wo haste das Script kopiert ? *g*

in der Zeile drüber heißt der Abfrage-Text noch $nr .. und direkt da drunter auf einmal $loeschen ?.. klar soweit ?
allerdings solltest du nun nicht $loeschen ändern, sondern $nr - denn $nr transportiert ja deine $_POST-Variable ...

außerdem DRINGENST anschauen .. SQL-Injections .. sonst lösch ich dir deine komplette Tabelle innerhalb von ein paar Sekunden

nikosch · 20.08.2011, 00:22

Zitat:

$connection = mysql_connect("$host" , "$user" , "$pass")

Auch hierzu gleich mal: PHP.de Wiki | Code Smells

RockHound · 20.08.2011, 03:50

Danke erst mal für die vielen und schnellen Antworten!

Okay, werde mich gleich mal dran machen den query zu ändern

(Vor lauter Bäume den Wald nicht mehr sehen - kannte jeder, nicht wahr?)

In der Tat, gewisse Fragmente aus div. vollständigen PHP schnipsel sind kopiert, na klar - es schleichen sich Fehler ein, aber ist ja auch recht bequem...

Über die Injection Geschichte muss ich mir später gedanken machen - allerdings weiß ja keiner um welche URL es sich handelt

Jedenfalls dient das Script im Augenblick nur zum eintragen von Ruhestörungen
(automatische Datums & Zeiteintragung in de db - das Textfeld entsprechend zum eintragen der Fälle)

Klappt soweit Prima... also mache ich mich gleich an die Arbeit - Melde mich dann wieder!

LG

__________________________________________

Gaaaanz lieben Dank noch mal! Hat auf anhieb das Problem gelöst -na logo!

Jetzt habe mal eben via google und div. wiki's herausgefunden, das mysql die ID beim löschen nicht automatisch runtersetzt - schlecht.
Ich war der meinung bei einer älternen mqsql version wäre das mal gegangen... autoincrement und unique meine ich...
Aber ich schätze das es tatsächlich das damalige script war, das die ID's wieder runtergesetzt hat nach dem löschen.
Kann das einer bestätigen?

Aufjeden fall bräuchte ich das noch, dass die ID nach dem klöschen kleiner wird, sodass beim löschen von z.B. 45 dann 44 ist, aber nicht auf 46 weiter zählt...

ChrisB · 20.08.2011, 10:45

Zitat:

Zitat von RockHound

Jetzt habe mal eben via google und div. wiki's herausgefunden, das mysql die ID beim löschen nicht automatisch runtersetzt - schlecht.

Nein - nicht „schlecht“.

Beim Googlen solltest du auch auf die Aussage gestossen sein, dass a) eine ID ausschließlich zur Identifikation eines Datensatzes dienen soll, und für nichts anderes, und b) eine einmal „benutzte“ ID nicht erneut vergeben wird, um die Datenintegrität zu gewährleisten.

Zitat:

Aufjeden fall bräuchte ich das noch, dass die ID nach dem klöschen kleiner wird

Nein, bräuchtest du nicht.
Wenn du meinst, du bräuchtest, machst du etwas falsch, und misst der ID eine Bedeutung bei, die sie nicht haben sollte.

RockHound · 20.08.2011, 22:34

Hi ChrisB,
danke für deine Antwort.

Ja, doch, brauche ich - zumindest hatte ich es beim damaligen script so und es war auch ganz nützlich.
denn wenn ich ein array habe und aus dem array einen Eintrag lösche, ist es nur logisch den neuen Eintrag
mit der letzten id zu versehen.

Wer hat in seinem array schon gerne 32,33,35,39,40,41... also immense Lücken?

nikosch · 20.08.2011, 23:09

Arrays kann man problemlos auch mit Lücken verarbeiten.

RockHound · 21.08.2011, 05:00

Natürlich "kann" man das... ->soll<- aber nicht.

Ich möchte ja die ID's anzeigen und bei gelöschten Einträgen sollen eben keine
Lücken vorhanden sein.

Jedenfalls, wenn hier keiner dazu bereit ist mir nützliche Tipps
bezüglich dessen zu geben, muss ich natürlich schauen wie ich weiter komme.

Aber trotzdem, war total nett von Euch und ich kann mich nur noch mal
ganz Herzlich für die schnellen Antworten bedanken.

Jedenfalls bleibe ich diesem Forum hier treu!

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
Mysql einträge mit ner Confirmbox löschen	~fransen~	PHP Einsteiger	12	10.01.2011 09:15
Scriptsuche MySQL und CSV vergleichen u. nicht vorhandenes löschen	headbannger83	Scriptbörse	3	09.01.2011 13:45
Scriptsuche Daten aus MySQl Datenbank und CSV Datei vergleichen u. ggf übertragen oder löschen	headbannger83	Scriptbörse	0	07.01.2011 01:00
MySQL Datentransfer (Dateigroesse 4MB) via Shell Zugang oder einfacher	promoterlv	Datenbanken	1	14.11.2010 22:12
[Erledigt] PHP bekommt keine Verbindung zu MySQL	SaKe	Datenbanken	15	20.09.2010 14:45
[Erledigt] Kann keine Umlaute im mysql client eingeben	Oger	Datenbanken	9	02.04.2009 11:54
MySQL Konsole und Umlaute unter Windows [LÖSUNG]	f4ckm5	Datenbanken	8	30.03.2009 22:10
MySQL Tabelleninhalt löschen	Paddy	PHP Tipps 2008	19	10.12.2008 11:59
Lerne Grundlagen \| Quellensammlung	cycap	PHP Einsteiger	0	12.11.2008 16:23
MySQL Server startet nicht mehr richtig...		Datenbanken	16	03.03.2006 19:40
MySQL per PHP zeilenweise löschen...		PHP Tipps 2006	7	21.02.2006 15:29
Dateien über MySQL löschen	Charly Shop Sui	PHP Tipps 2005	4	06.05.2005 15:42
mysql query... alle datensäze älter als x tage löschen...	Sclot	Datenbanken	2	05.03.2005 23:04
Nur markierte Einträge löschen mysql php		PHP-Fortgeschrittene	2	19.09.2004 21:53
LÖSCHEN VON MySQL Daten.		PHP Tipps 2004	6	01.09.2004 17:17

19.08.2011, 19:05
RockHound Neuer Benutzer Registriert seit: 19.08.2011 Beiträge: 4 PHP-Kenntnisse: Anfänger	ID löschen in MySQL Hallo zusammen! Ich fange mal so an: google, foren, 10.000 versuche - hat alles nichts gebracht. Auch hier habe ich fleißig geschaut, aber die codes unterscheiden sich immer wieder auf's neue - insofern gab es keine Lösung für mich. Wie die Überschrift schon sagt, möchte ich per 'ID' löschen, das Formular soll den Wert mit 'POST' an eine andere PHP übergeben, aber wenn ich den Datensatz neu abrufe ist nichts gelöscht. --Datei: del.php-- PHP-Code: `<form method="post" action="l.php"> <table width="351" border="0"> <tr> <td valign="bottom">Welche ID soll gelöscht werden?<br></td> <td valign="top"><br></td> </tr> <tr> <td valign="bottom"><input type="text" name="nr" value="" size="3" /> <input type="submit" name="button" value="OK" /></td> </tr> </table> </form>` --Datei: l.php-- PHP-Code: <?php $host = "localhost"; $user = "**"; $pass = "**"; $dbase = "fleet"; $connection = mysql_connect("$host" , "$user" , "$pass") OR die ("Verbindung zur Datenbank nicht mölich."); $db = mysql_select_db($dbase , $connection) OR die ("<h1>Auswahl der Datenbank nicht möglich.<br>Datenbank existiert nicht oder keine definiert!<h1>"); $nr = $_POST["nr"]; $nr = "DELETE FROM wood WHERE id = '$nr'"; $loesch = mysql_query($loeschen); mysql_close(); ?> Ich hoffe ihr könnt mir helfen und würde mich freuen, wenn die Auflistung brauchbar war. LG


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

19.08.2011, 19:15
Capfly Erfahrener Benutzer Registriert seit: 25.01.2009 Beiträge: 1.027 PHP-Kenntnisse: Fortgeschritten	$loeschen stimmt nicht im query __________________ MfG ~Capfly Jetzt NEU! Cpix & Wbits Katahlan.de - Das Browsergame

19.08.2011, 19:21
smon Neuer Benutzer Registriert seit: 17.08.2011 Beiträge: 9 PHP-Kenntnisse: Fortgeschritten	Hallo RockHound, das SQL-Statement zum Löschen wird überhaupt nicht ausgeführt. Du übergibst eine nicht-existende Variable ($loeschen) an die mysql_query(...)-Funktion, dein Statement ist allerdings in der Variable $nr gespeichert. Darüber hinaus ist, bzw. wäre, wenn das SQL-Statement so ausgeführt würde, der gepostete Code anfällig für SQL-Injection-Angriffe, da eine Benutzereingabe ungefiltert in das SQL-Statement eingefügt wird. Viele Grüße smon

20.08.2011, 00:19
eagle275 Erfahrener Benutzer Registriert seit: 01.09.2010 Beiträge: 4.561 PHP-Kenntnisse: Fortgeschritten	na, wo haste das Script kopiert ? g in der Zeile drüber heißt der Abfrage-Text noch $nr .. und direkt da drunter auf einmal $loeschen ?.. klar soweit ? allerdings solltest du nun nicht $loeschen ändern, sondern $nr - denn $nr transportiert ja deine $_POST-Variable ... außerdem DRINGENST anschauen .. SQL-Injections .. sonst lösch ich dir deine komplette Tabelle innerhalb von ein paar Sekunden __________________ "Irren ist männlich", sprach der Igel und stieg von der Drahtbürste

20.08.2011, 03:50
RockHound Neuer Benutzer Registriert seit: 19.08.2011 Beiträge: 4 PHP-Kenntnisse: Anfänger	Danke erst mal für die vielen und schnellen Antworten! Okay, werde mich gleich mal dran machen den query zu ändern (Vor lauter Bäume den Wald nicht mehr sehen - kannte jeder, nicht wahr?) In der Tat, gewisse Fragmente aus div. vollständigen PHP schnipsel sind kopiert, na klar - es schleichen sich Fehler ein, aber ist ja auch recht bequem... Über die Injection Geschichte muss ich mir später gedanken machen - allerdings weiß ja keiner um welche URL es sich handelt Jedenfalls dient das Script im Augenblick nur zum eintragen von Ruhestörungen (automatische Datums & Zeiteintragung in de db - das Textfeld entsprechend zum eintragen der Fälle) Klappt soweit Prima... also mache ich mich gleich an die Arbeit - Melde mich dann wieder! LG __________________________________________ Gaaaanz lieben Dank noch mal! Hat auf anhieb das Problem gelöst -na logo! Jetzt habe mal eben via google und div. wiki's herausgefunden, das mysql die ID beim löschen nicht automatisch runtersetzt - schlecht. Ich war der meinung bei einer älternen mqsql version wäre das mal gegangen... autoincrement und unique meine ich... Aber ich schätze das es tatsächlich das damalige script war, das die ID's wieder runtergesetzt hat nach dem löschen. Kann das einer bestätigen? Aufjeden fall bräuchte ich das noch, dass die ID nach dem klöschen kleiner wird, sodass beim löschen von z.B. 45 dann 44 ist, aber nicht auf 46 weiter zählt... Geändert von RockHound (20.08.2011 um 04:06 Uhr). Grund: Ergänzung...

20.08.2011, 22:34
RockHound Neuer Benutzer Registriert seit: 19.08.2011 Beiträge: 4 PHP-Kenntnisse: Anfänger	Hi ChrisB, danke für deine Antwort. Ja, doch, brauche ich - zumindest hatte ich es beim damaligen script so und es war auch ganz nützlich. denn wenn ich ein array habe und aus dem array einen Eintrag lösche, ist es nur logisch den neuen Eintrag mit der letzten id zu versehen. Wer hat in seinem array schon gerne 32,33,35,39,40,41... also immense Lücken? Geändert von RockHound (20.08.2011 um 22:35 Uhr). Grund: Änderung

20.08.2011, 23:09
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.987 PHP-Kenntnisse: Fortgeschritten	Arrays kann man problemlos auch mit Lücken verarbeiten. __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

21.08.2011, 05:00
RockHound Neuer Benutzer Registriert seit: 19.08.2011 Beiträge: 4 PHP-Kenntnisse: Anfänger	Natürlich "kann" man das... ->soll<- aber nicht. Ich möchte ja die ID's anzeigen und bei gelöschten Einträgen sollen eben keine Lücken vorhanden sein. Jedenfalls, wenn hier keiner dazu bereit ist mir nützliche Tipps bezüglich dessen zu geben, muss ich natürlich schauen wie ich weiter komme. Aber trotzdem, war total nett von Euch und ich kann mich nur noch mal ganz Herzlich für die schnellen Antworten bedanken. Jedenfalls bleibe ich diesem Forum hier treu!