[ERLEDIGT] PHP-Variablen Problem in MySql

[Nakroma]

Als allererstes: Ja ich weiß ich sollte iMysql benutzen.

So nun zu meinem Problem, ich hab hier ein Skript welches dich automatisch in einen von 9 Teamplätzen einsortieren soll, natürlich nicht in welche die schon vergegeben sind. Das Problem ist, das Skript wird ausgeführt und alle MySql Sachen klappen, außer eine. Es hängt definitiv mit der Variable $i_member=$Name zusammen, weil wenn ich manuell Member2 eintrage, klappt es. Hat da irgendjemand eine Idee wie ich das mach?
Das hier ist mein Code. Bei $i_member kommt definitiv (überprüft) 'Member2' raus.

    $aendern2 = "UPDATE Squads SET '$i_member'='$Name' WHERE Name='$Squad'";
    $aendern_query2 = mysql_query($aendern2);

Hier ist meine Datenbank:

Mfg!

 

[threadi]

Ich würde in PHP die Zeile eher so schreiben:

$aendern2 = "UPDATE Squads SET '".$i_member."'='".$Name."' WHERE Name='".$Squad."'";

Außerdem solltest Du mit mysql_error() prüfen, ob es einen SQL-Fehler gibt. Wenn ja, dann schau dir genau das erzeugte SQL-Statement an, z.B. per

echo $aendern2;

 

[Nakroma]

Ich würde in PHP die Zeile eher so schreiben:

$aendern2 = "UPDATE Squads SET '".$i_member."'='".$Name."' WHERE Name='".$Squad."'";

Außerdem solltest Du mit mysql_error() prüfen, ob es einen SQL-Fehler gibt. Wenn ja, dann schau dir genau das erzeugte SQL-Statement an, z.B. per

echo $aendern2;

Danke erstmal für die Antwort, aber es wirft keinen Mysql Fehler aus und bei deinen Änderungen passiert auch nix ._.

 

[Cheffchen]

Hallo,

in der DB sehe ich keine Spalte "Name"?
$aendern2 wird aber auch als sql ausgeführt?

Cheffchen

 

[Nakroma]

Hallo,

in der DB sehe ich keine Spalte "Name"?
$aendern2 wird aber auch als sql ausgeführt?

Cheffchen

Ich denke schon, wie kann man das überprüfen?

 

[Cheffchen]

Hallo,

gibts das den Datensatz denn der bei "Name" gesucht wird?
sql in de art vorhanden

if (!$mysqli->query($aendern2)) {printf("DB Fehler: %s\n", $mysqli->error);}

ohne mehr infos kann man nicht helfen.

Cheffchen

 

[Nakroma]

Hallo,

gibts das den Datensatz denn der bei "Name" gesucht wird?
sql in de art vorhanden

if (!$mysqli->query($aendern2)) {printf("DB Fehler: %s\n", $mysqli->error);}

ohne mehr infos kann man nicht helfen.

Cheffchen

Ja den Datensatz gibt es, zum wiederholten mal, wenn ich statt $i_member Member2 hinschreibe klappt es ja, ich will nur wissen wie ich das mit der Variable richtig mache.

 

[threadi]

Wie sieht das Statement denn aus, wenn Du es mit echo ausgibst?

Btw.: besteht überhaupt die Datenbankverbindung?

 

[Nakroma]

Hört Ihr mir überhaupt zu? Sobald ich

'$i_member'

durch

Member2

ersetze klappt es ja, und alle anderen MySql Sachen davor im Code gehen ja auch, das heißt es ist 100% ein Problem wie ich die Variable im MySql angebe und dafür brauch ich ja grad Hilfe :/

 

[threadi]

Dennoch würde ich gerne das per PHP zusammengestellte Statement mal sehen.

 

[Nakroma]

Ich habs Problem gelöst... hat vorher nicht funktioniert jetzt aber plötzlich lol.

$i_member='$Name'

Also einfach ohne die Striche... naja trotzdem Danke für die Hilfe hier!

 

[threadi]

Die Lösung ist aus deiner Sicht also Quellcode den Du uns nicht gezeigt hast? Zudem verstehe ich deine Lösung nicht. Natürlich ist es besser so eine Wertzuweisung ohne Anführungszeichen zu schreiben. Aber es sollte auch nicht so ein Problem verursachen. Da Du uns das erzeugten SQL-Statement weiter vorenthalten hast, ist es aber weiterhin schwierig den richtigen Grund zu finden.

 

[Nakroma]

Nein, das Problem war das es nicht in die Datenbank eingetragen wurde, weil es versucht hat den Wert 'Member2' zuzuweisen, und nicht Member2.
Außerdem habe ich den Quellcode gezeigt, das hier war mein Ausgangsquellcode:

$aendern2 = "UPDATE Squads SET '$i_member'='$Name' WHERE Name='$Squad'";

und so funktioniert es:

$aendern2 = "UPDATE Squads SET $i_member='$Name' WHERE Name='$Squad'";

 

[threadi]

Ach jetzt verstehe ich. Logisch. Ich würde dennoch als Optimierung noch folgendes vorschlagen:

$aendern2 = "UPDATE Squads SET ".$i_member." ='".mysql_real_escape_string($Name)."' WHERE Name='".mysql_real_escape_string($Squad)."'";

 

[bdt600]

Es funktioniert, ist aber trotzdem falsch!!!
Korrekt schreibt man das:

$aendern2 = "UPDATE `Squads` SET `$i_member` ='$Name' WHERE `Name`='$Squad'";

Tabellen- und Spaltemnamen gehören zwischen Backticks

 

[threadi]

Ok, mit bdt600's Hinweis würde das optimal so aussehen:

$aendern2 = "UPDATE Squads SET `".$i_member."` ='".mysql_real_escape_string($Name)."' WHERE Name='".mysql_real_escape_string($Squad)."'";

 

[bdt600]

Ok, mit bdt600's Hinweis würde das optimal so aussehen:

Fast, da fehlen immer noch Backticks ;)