problem mit mysql

[bumeno]

hey,

ich hab folgendes problem: jedesmal wenn ich einen string mit einem datenbankeintrag vergleiche bekomme ich den fehler zurück, dass der eintrag nicht existiert.

zb:

SELECT * FROM `$login` WHERE `username` = `$username`

gibt folgenden fehler:

Unknown column 'username' in 'where clause'

ich hab dermaßen oft gekuckt: ich hab nirgends tippfehler drin. es geht nur wenn ich einen integer benutze.

dave

 

[Dadelu]

Hi

Nun, ohne Zugriff auf meine Home Daten, kann ich dir kein Beispiel nennen. Jedoch so auf den ersten Blick, nimm die ` vom Spaltennamen weg. Sie sind zwar nicht falsch, aber sie werden nicht gebraucht

Bin mir nicht sicher, aber wenn du einen String vergleichen willst, musst Hochkommas (') machen und nicht `.

Hastu du mal $Login ausgegeben vor dem Query? Wenn dies natürlich leer, oder einen falschen Wert besitzt, findet er die DB nicht (sollte jedoch einen anderen SQL - Error auswerfen)

Und dann , wenn du Tabelle / Spalte dynamisch machen willst, müsstest du den String konkatenieren!

Hier dein Beispiel:

$SelectQuery = "SELECT * FROM".$login." WHERE username ='".$username."'";

und dann

$Resultat = mysql_query($SelectQuery);

Probiers mal, sollte funktionieren.

Gruss Dario

 

[Nogger]

Und dann , wenn du Tabelle / Spalte dynamisch machen willst, müsstest du den String konkatenieren!

PHP kann durchaus auch Stringinterpolation.

 

[Skeeve]

Bitte solchen Code nicht so einfach verwenden. Immer an die Gefahr von SQL Injection denken!

 

[jesusjones]

hey,

ich hab folgendes problem: jedesmal wenn ich einen string mit einem datenbankeintrag vergleiche bekomme ich den fehler zurück, dass der eintrag nicht existiert.

zb:

SELECT * FROM `$login` WHERE `username` = `$username`

gibt folgenden fehler:

Unknown column 'username' in 'where clause'

ich hab dermaßen oft gekuckt: ich hab nirgends tippfehler drin. es geht nur wenn ich einen integer benutze.

dave

und, welches forum hat dir nun die besseren antworten geliefert

 

[Mitglied 7974]

Ich spiel nur ungern den Korinthenkacker, aber dieser Code würde nur zu einem Fehler führen, weil ein Leerzeichen zwischen FROM und $login fehlt.

$SelectQuery = "SELECT * FROM".$login." WHERE username ='".$username."'";

Richtig wär's so:

$SelectQuery = "SELECT * FROM ".$login." WHERE username ='".$username."'";

Nichts für ungut,
Matthias

 

[Hilarious]

Auch ganz ohne Korinthengekacke:

Schöner wär's so:

$SelectQuery = sprintf("SELECT * FROM %s WHERE username='%s LIMIT 1"
                                , mysql_real_escape_string($login)
                                , mysql_real_escape_string($username) );

 

[Skeeve]

Auch ganz ohne Korinthengekacke:

kleine Korrektur:

$SelectQuery = sprintf("SELECT * FROM %s WHERE username='%s' LIMIT 1"
                                , mysql_real_escape_string($login)
                                , mysql_real_escape_string($username) );

Nebenbei: man sollte statt "SELECT *" immer die Spalten explizit angeben, da man so unabhängig von der tatsächlichen Spaltenreihenfolge in dr DB ist.

 

[Trapper]

Nebenbei: man sollte statt "SELECT *" immer die Spalten explizit angeben, da man so unabhängig von der tatsächlichen Spaltenreihenfolge in dr DB ist.

Was bei der Nutzung von assoziativen Arrays (wie sie in den meisten Skriptsprachen angeboten werden) ziemlich egal ist.

 

[Skeeve]

Was bei der Nutzung von assoziativen Arrays (wie sie in den meisten Skriptsprachen angeboten werden) ziemlich egal ist.

Stimmt. Davon steht da aber nix.

Randbemerkung: Ich ziehe dennoch die Angabe der Spalten vor:

• Es wird bereits beim SELECT angegeben, welche Spalten man erwartet
• Man kann die Ausgabe aufs Minimum der benötigten Spalten einschränken

 

[Trapper]

Stimmt. Davon steht da aber nix.

Die Variablenbezeichner der Form $XXXX deuten aber schon sehr darauf hin und die Möglichkeit des assoziativen Zugriffs kennt er spätestens jetzt, aber ich weiß, du bist ja der Klugscheißer hier

Randbemerkung: Ich ziehe dennoch die Angabe der Spalten vor:

• Es wird bereits beim SELECT angegeben, welche Spalten man erwartet
• Man kann die Ausgabe aufs Minimum der benötigten Spalten einschränken

Diese Argumente hingegen kann ich vollends unterstützen

 

[Skeeve]

aber ich weiß, du bist ja der Klugscheißer hier

Endlich jemand, der es einsieht. Ich hoffe, das beschert Dir kein negatives Karma! Prophylaktisch gab es positives von mir.

 

[Hilarious]

Danke für die Korrektur, Skeeve. Und darüber hinaus ein prima Beitrag!
Wenn's keine Karmalatten mehr gibt, mach ich das eben hier öffentlich.

 

[Hobbes_]

Wenn's keine Karmalatten mehr gibt, mach ich das eben hier öffentlich.

Ja, weshalb gibt's die eigentlich nicht mehr?

 

[Hilarious]

Ja, weshalb gibt's die eigentlich nicht mehr?

Gute Frage, vermutlich eine »lange Geschichte«...