You have a error in your SQL syntax..

wolf · 20. Mai 2014

Abend,
Ich vertsehe das Problem nicht

Code

1 -> 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 'WHERE `username` = 'wolf' SET `lastseen` = '2014-05-20 21:33:16'' at line 1 -- Code : 42000 -- in localhost /var/www/UMAG/classes/class.usermag.helper.php on line 206. Query: REPLACE `useractions` WHERE `username` = :username SET `lastseen` = :time

Hier Der Code dazu

PHP

function setLastSeen(){
    $SQL['setLastSeen']['query'] = "REPLACE `useractions` WHERE `username` = :username SET `lastseen` = :time";
    $SQL['setLastSeen']['stmt'] = $this->SYS['conf']['db']['conect']->prepare($SQL['setLastSeen']['query']);
    if (
        !$SQL['setLastSeen']['stmt']->execute(
            array(
                ':username' => $this->username,
                ':time' => date("Y-m-d H:i:s")
            )
        )
    ){#wenn fehler beim ausführen
        generate_query_ErrorMessage($SQL['setLastSeen']['stmt']->errorInfo(),__LINE__,__FILE__,$SQL['setLastSeen']['query']);
    }
}

Alles anzeigen

EDIT:
Tabellen Spalte: `username` VARCHAR(50) UNIQUE NOT NULL

cottton · 21. Mai 2014

REPLACE verhält sich wie INSERT INTO.
Ein WHERE ist nicht vorgesehen.

SQL

REPLACE INTO
`useractions`  
(`lastseen`)
VALUES(:time)




-- oder 
REPLACE INTO
`useractions`  
SET `lastseen` = :time

Alles anzeigen

http://dev.mysql.com/doc/refman/5.6/en/replace.html

Warum eigtl kein UPDATE ?

wolf · 21. Mai 2014

Ich weis halt nicht ob es schon einen Datensatz mit dem username bereits gibt...

lauras · 21. Mai 2014

INSERT mit ON DUPLICATE KEY UPDATE?

cottton · 21. Mai 2014

SQL

UPDATE`useractions` SET `lastseen` = :time WHERE `username` = :username;




-- wenn kein datensatz mit `username` = :username existiert, dann wird nix geupdated

wolf · 21. Mai 2014

Joa, danke Lauras, denke das passt
@Cotton je aben... dann wird nix geschrieben... ich will ja eben das Entweder neu geschrieben wird (wenn noch nicht vorhanden) oder eben geupdated wird

So, hier noch die Lösung

PHP

function setLastSeen(){
    $SQL['setLastSeen']['query'] = "
        INSERT INTO `useractions`(
            `username`,
            `lastseen`
            )
        VALUES (
            :username,
            :time
        )
        ON DUPLICATE KEY UPDATE `lastseen` = :time;
    ";
    $SQL['setLastSeen']['stmt'] = $this->SYS['conf']['db']['conect']->prepare($SQL['setLastSeen']['query']);
    if (
        !$SQL['setLastSeen']['stmt']->execute(
            array(
                ':username' => $this->username,
                ':time' => date("Y-m-d H:i:s")
            )
        )
    ){#wenn fehler beim ausführen
        generate_query_ErrorMessage($SQL['setLastSeen']['stmt']->errorInfo(),__LINE__,__FILE__,$SQL['setLastSeen']['query']);
    }
}

Alles anzeigen

Danke für die Hilfe

cottton · 21. Mai 2014

Bedenke, dass die auto_increment id (wenn vorhanden) hochgezählt wird, falls es den Eintrag schon gab.
Soll heißen
angenommen der `username` existiert schon, dann macht mysql folgendes:
~versuche den INSERT
~id auto_increment +1
~"oh usernameis unique und existiert schon"
~on duplicate key update username
~fertsch

Kurz: es entstehen "gaps" (Lücken)

wolf · 21. Mai 2014

Danke, aber zum glück habe ich mir id's gespart, da username schon unique genug ist

cottton · 23. Mai 2014

(für den Fall der Name ist primary key) Wird bei großen Datenmengen benachteiligend sein:

Zitat von http://dev.mysql.com/doc/refman/5.1-olh/de/innodb-tuning.html

In InnoDB verschwendet ein langer PRIMARY KEY viel Plattenplatz, weil sein Wert mit jedem sekundären Indexdatensatz gespeichert werden muss. (Siehe auch Abschnitt 9.10, „InnoDB-Tabellen- und Index-Strukturen“. Richten Sie eine AUTO_INCREMENT-Spalte als Primärschlüssel ein, wenn Ihr Primärschlüssel lang ist.

You have a error in your SQL syntax..

Jetzt mitmachen!

Teilen