Doppelter Eintrag

[mewking]

Hallo, ich nutze folgenden SQL-Code, um in meine Tabelle einen Eintrag zu schreiben:

$sql = "INSERT INTO `items` (`itemID` ,`itemname` ,`user` ,`used`)VALUES (NULL , 'Trank', 'mewking', '0');"; 

Jedoch kommt es dann zu einem doppelten Eintrag. mysql_error() gibt dazu nichts aus und auch mysql_affected_rows gibt 1 zurück.
Wo liegt da das Problem =?

[Manko10]

Ich könnte wetten, du hast irgendwo ungewollt eine Schleife.

[mewking]

Nein, im Script ist keine Schleife.
nopaste.info - free nopaste script and service
/E: mit break versucht:

Fatal error:  Cannot break/continue 1 level in C:\xampp\htdocs\mewland.de\root\index.php  on line 434

[lstegelitz]

Wird $sql nochmals ausgeführt, ohne es vorher neu belegt zu haben?

[mewking]

nein, das hat nix gebracht ein $sql = '' zu definieren

[ChrisB]

Dann wird vermutlich dein Script unbeabsichtigt zwei mal aufgerufen.


Btw.:

$thing = addslashes(htmlspecialchars($_GET['thing'])); 

Das sieht so aus, als hätte da jemand wenig Ahnung, was er tut ...

[mewking]

@chrisb:
im header:

ini_set('magic_quotes_gpc', 0); 

dann bringt addslashes was

[nikosch]

Nein.

[ChrisB]

@chrisb:
im header: ini_set('magic_quotes_gpc', 0); dann bringt addslashes was

Mit dieser Aussage bestätigst du meine Vermutung, dass wenig Ahnung vorhanden ist.

[soundbear]

Hallo, ich nutze folgenden SQL-Code, um in meine Tabelle einen Eintrag zu schreiben:

$sql = "INSERT INTO `items` (`itemID` ,`itemname` ,`user` ,`used`)VALUES (NULL , 'Trank', 'mewking', '0');"; 

Jedoch kommt es dann zu einem doppelten Eintrag. mysql_error() gibt dazu nichts aus und auch mysql_affected_rows gibt 1 zurück.
Wo liegt da das Problem =?

Pack einfach eine ID dazu, die den Primary Index und Autoincrement erhält und schon ist die Sache erledigt. Sollte man sowieso nutzen!

[nikosch]

Schlechte Lösung, weil Symptombekämpfung.

mit break versucht:

Eine Schleife kann bspw. auch durch ein mehrfaches Include eines Scripts eintstehen. Das solltest Du prüfen, bspw. nach der Query mal eine Testausgabe machen.

[soundbear]

Aso, dachte, dass der Eintrag auch doppelt rein soll.

Dann entweder den Fehler in der Schleife suchen oder ganz einfach vorher eine Select-Abfrage mit LIMIT 0,1 zum Testen. Falls es den Datensatz schon gibt, einfach weitermachen, wenn nicht anlegen ...

[ChrisB]

ganz einfach vorher eine Select-Abfrage mit LIMIT 0,1 zum Testen. Falls es den Datensatz schon gibt, einfach weitermachen, wenn nicht anlegen ...

Das ist immer noch Symptombekämpfung.

[soundbear]

Wenn die Symptombekämpfung weniger CPU beansprucht und/oder übersichtlicher umzusetzen ist, sollte man diese vorziehen.

Aber wie auch immer: Er wollte Lösungen und keine Grundsatzdiskussion.

[mewking]

wie schon gesagt:
keine schleife!
ich habe jetzt herausgefunden was das problem war, jedoch nicht warum.
die seite ruft sich beim laden zweimal auf, aber ich finde dafür keine ursache außerhalb
des if-bereichs, da es ja bei allen ifs der fall ist.