database update met datum

Ik gebruik zelf geen PDO, maar moet je niet ook bind_param() gebruiken?

Het grootste probleem is dat je :started probeert te binden, maar die bestaat niet binnen je query, en je bindt :username niet en die bestaat WEL binnen je query. Waarom gebruik je trouwens niet gewoon de datumtypen van je database? DATETIME in dit geval, en updaten met NOW().

On a sidenote, het is ook mogelijk om aan execute() een genummerd of associatief array mee te geven (zie voorbeeld #2 en #3) in plaats van alles te binden. Scheelt je wel wat code, maar dat heeft natuurlijk wel wat implicaties, als er fouten in het formaat van je invoer zitten dan loopt de code pas spaak als je de query probeert uit te voeren. Dit in tegenstelling tot wanneer je eerst bind(), PHP zal dan al eerder zeuren dat e.e.a. niet goed geformateerd is.

Voeg nooit variabelen rechtstreeks in de SQL-string toe, hier zijn prepared statements niet voor bedoeld!

En wat @Ben zegt: sla geen geformatteerde datums op (Y/m/d H:i:s) maar maak van de daarvoor bestemde kolomtypen gebruik met de daarbij behorende formats.

Op het moment dat je van PDO gebruik maakt ben je wel gebonden (:D) aan wat regels. Zo zou je voor de goede orde ook van een try-catch blok gebruik moeten maken om eventuele exceptions op te vangen.

Ook is PDO ietwat verraderlijk in die zin dat het handjevol klasses niet echt ingewikkeld is, het venijn zit em in de staart (specifiek, in de configuratie en het gebruik van de specifieke database-driver).

[size=xsmall](en dan is PDO ook nog eens niet geoptimaliseerd voor gebruik van een specifieke database, simpelweg omdat PDO niet geschreven is voor gebruik van een specifieke database)[/size]

Ik probeer jullie adviezen op te volgen. Heb nu onderstaand script.

Recoverable fatal error: Object of class DateTime could not be converted to string in safe.php on line 41
line 41 is: $pdoResult->bindValue(':datum', $datum);

In phpmyadmin in de tabel staat: lastactive - datetime - Nee - 0000-00-00 00:00:00
Wat doe ik dan fout met die datum?

        try
        {
        $update_query = "UPDATE gebruikers SET lastactive=:datum WHERE username=:username";
        $datum = new DateTime();
//        $dateTime->format('Y/m/d H:i:s');
        $pdoResult->bindValue(':datum', $datum);
        $pdoResult->bindValue(':username', $_SESSION['username']);
        $pdoResult = $pdoConnect->prepare($update_query);
        $pdoResult->execute();
          }     // einde try

            catch(PDOException $e){
            echo $e->getMessage();
            }

DATETIME als database type dus. Je moet wel lezen wat er gezegd wordt voordat je wat doet.

Ben, ik ben een beginner hier in, zeker in PDO. Ik zou willen dat ik alles al begreep wat er gezegd werd.
Dus ja .. ik doe veel trial en error.

Wat bedoel je precies met je advies? Kun je misschien in code aangeven wat is verkeerd doe?

Volgens myphpadmin staat in de tabel staat het als datetime.
naam veld : lastactive
type : DATETIME
standaardwaarde : 0000-00-00 00:00:00

Dan is het helemaal mooi, daar kun je geen standaard tijd in zetten, maar een tijd die de database herkent. Gelukkig heeft de database daar zijn eigen voorzieningen voor dus kun je iets als volgt doen:
<?php
$update_query = "UPDATE gebruikers SET lastactive = NOW() WHERE username = :username";
$pdoResult = $pdoConnect->prepare($update_query);
$pdoResult->bindValue(':username', $_SESSION['username']);
$pdoResult->execute();
?>
Let vooral op de volgorde van wat je doet, je kunt geen parameters binden voordat je hebt geprepared.
1. Prepare
2. Bind
3. Execute

Of nog wat korter:
<?php
$update_query = "UPDATE gebruikers SET lastactive = NOW() WHERE username = :username";
$pdoResult = $pdoConnect->prepare($update_query);
$pdoResult->execute(array(':username' => $_SESSION['username']));
?>

Bedankt Ben, die volgorde had ik nog niet ergens op internet gelezen. Weer wat geleerd.
Dit werkt !

Volgens mij heeft @Theo niets verkeerd gedaan @Ben, behalve dit:

Recoverable fatal error: Object of class DateTime could not be converted to string in safe.php on line 41

Het enige wat @Theo daar nog hoefde te doen is een DateTime-object omzetten naar een datum-string.

Dit doe je met de format()-methode.

En dan was je klaar geweest.

Iets waar je misschien nog over na moet denken is het volgende: wie dicteert hoe laat het is? PHP of je database? Want er kunnen best (kleine) verschillen zijn tussen deze twee klokken. Zorg dus altijd dat je consequent van ofwel het een (PHP) ofwel het ander (database) gebruik maakt, maar niet van een combinatie van beide.