[MySQL 4.0] Datensatz löschen

[nate10]

Hallo hab einen Fehle in meiner SQL Sytnax und kann diesen nicht finden,
hier das Codefragment

    if(isset($del_spieler)) {
    $sql = "DELETE from spieler WHERE id=$del_spieler";
    if (@mysql_query($sql)) 

Fehlermeldung:

Fehler beim löschen: You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

hier zeige ich dann den link an:

<td class = 'tab1'>" . "<a href='$PHP_SELF?del_spieler=$del_spieler'>" . "Spieler löschen" .  "</td>" 

hier ein test link:
http://www.svschwabelweis.de/test.php

[wahsaga]

Re: [MySQL 4.0] Datensatz löschen

Kontrollausgabe Query?

[nate10]

wie meinst du dass jetzt?

[Abraxax]

und kann diesen nicht finden

wie willst du auch einen fehler finden, wenn du @ vor mysql_query() schreibst, um fehler zu unterdrücken und nicht einmal mysql_error() einsetzt`?

[KamiKatze]

Fehler beim löschen: You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

das erste " wirds wohl nicht sein, also ist es wohl das letzte ". und was sagt uns das? der fehler liegt unmittelbar davor, also schauen wir mal nach.

id=$del_spieler

was sagt das sql-handbuch zu wertzuweisungen? richtig wäre

id='$del_spieler'

[nate10]

so hab ichs komplett geschrieben um auch eine fehlüberrüfung zu bekommen

if(isset($del_spieler)) {
    $del = "DELETE from spieler WHERE id=$del_spieler";
    if (@mysql_query($del)) {
        echo("<p>Der Spieler wurde gelöscht.</p>");
    } else {
        echo("<p>Fehler beim löschen: " . mysql_error() . "</p>");
    }
} 

[nate10]

ok super aber gelöscht wird deswegen immer noch nichts!

[KamiKatze]

dann mach mal bitte ein echo für $del_spieler und gucke, ob die übergebene id richtig ist und in der datenbank überhaupt existiert.

hilfreich wäre übrigens immer nen fehlercode mit anzugeben, gerade wenn du schon mysql_error() benutzt. hilft uns, dir zu helfen.

[thommyk]

Richtig wäre id='$del_spieler'

Nicht unbedingt. id=$del_spieler ist dann richtig wenn es ein Integer ist, was ich mal annehme. Sonst wird nämlich kein Index benutzt und macht Query langsam!

1. Vermute, $del_spieler ist leer. Deswegen geht die Query schief.
2a. Testen ob $del_spieler überhaupt einen Wert hat, also

if ($del_spieler) 

2b. Oder die Query so umschreiben, dass sie auch dann nicht fehlschlägt

WHERE id = 0".$del_spieler." 

Dann wird der Datensatz mit der ID=0 gelöscht, den es normalerweise nicht gibt (bei autoincrement)

[KamiKatze]

OffTopic:

---

@thommyk

Nicht unbedingt. id=$del_spieler ist dann richtig wenn es ein Integer ist, was ich mal annehme.

tja und scheinbar liegst du falsch, denn dann hätte er wohl nicht

ok super

geschrieben, denn ich glaube nicht, dass sich das problem durch "mysql_error" gelöst hat.

2a. Testen ob $del_spieler überhaupt einen Wert hat, also ....

$del_spieler wird er sich doch hoffentlich vorher aus der db holen und nicht eingeben lassen. damit sollte der wert vorbelegt sein, damit fällt vorschlag 2b weg, 2a und 1. erledigen sich mit einem simplen echo. das einzigste was mir dein posting bringt ist (neben nachgequatsche ne gute halbe stunde später), das eine indexsuche mit integer schneller geht, als mit char. wow, da haben wir jetzt aber was gelernt und vorallem ist

macht Query langsam!

so aussagekräftig.

---

[wahsaga]

@KamiKatze: Was hattest du denn heute morgen im Futter?
thommyks Hinweise bzgl. der Übergabe der ID waren gut und berechtigt.

[KamiKatze]

ich hasse es, wenn leute inhaltlich das selbe einfach nur anders wiederholen und dann auch noch ne halbe stunde später. ich habe doch nicht aus langerweile gesagt er soll mal ein echo davor schreiben. doch dann hätte er es selber gesehen und mal drüber nachgedacht, warum die query nicht funktioniert. außerdem übergibt er die id sicherlich irgendwo und da wird der fehler liegen. du schreibst doch auch nicht drei vermutungen, die nicht mal zutreffen müssen, sondern stuppst die leute auf das was falsch ist.

Was hattest du denn heute morgen im Futter?

kitekat, aber nur in soße, nicht in gelee. das wirds sein.

[hoffmann-c]

ist doch logisch warum nicht gelöscht wird.
die variable $del_spieler ist nicht zugewiesen.

Seht selber!

OffTopic:

---

hab mir den thread noch nicht richtig durchgelesen! lediglich das problem

---

[KamiKatze]

OffTopic:

---

@wahsaga

hab mir den thread noch nicht richtig durchgelesen!

ist doch toll so etwas, oder? ich geh jetzt in den keller.

---

[thommyk]

Was ich ja mal gerne von nate10 wüßte ist, ob er jetzt den Fehler gefunden hat oder nicht.
Auch wenn für KamiKatze der Informationsgehalt meines Posting wieder gleich Null ist.