sqlstate hy093 invalid parameter number parameter was not defined

[danlupo]

Salve qualcuno mi può spiegare questo errore ??

Ho questo codice:

   $postdata = array('email' => $email,'mobilephone' => $phone);
            $where = "username = '{$username}'";
            $this->db->update('utenti', $postdata, $where); 

che fa riferimento a questa funzione:

   public function update($table, $data, $where) {
       ksort($data);
      try { 

            $set = ''; 
            foreach ($data as $key => $value) { 
                $set.="$key = :$key"; 
            } 
            $sql = "UPDATE $table" 
                    . " SET $set" 
                    . " WHERE $where"; 
            session::set('campi', $sql); 
            $sth = $this->prepare($sql); 
            foreach ($data as $key => $value) { 
                $sth->bindValue(":$key", $value); 
            } 
      $sth->execute(); 
           session::set('risultato', $sth->execute()); 
        } catch (PDOException $e) { 
            echo  $e->getMessage(); 
            session::set('risultato', $e->getMessage()); 
        } 
    } 

Se faccio l'update di un campo alla volta funziona, se uso due o più campi va in errore come da titolo.

Qual è il possibile problema?

Grazie !

[boots]

foreach ($data as $key => $value) { 
                $set.="$key = :$key"; 
            } 

Dovresti separare i vari assegnamenti con la virgola, altrimenti ti viene tutto attaccato

es:
email = :emailphone = hone

[danlupo]

Mhh perchè il mio codice è quello della prima parte che ho postato (quella del $postdata) il resto fa parte di un template MVC.

Sai dirmi, se non è un problema, come modificarlo in pratica.

Devo aggiungere un

$set.=","; 

??

Grazie Daniele

[OT] ho visto che il mio post è stato inviato due volte, è possibile rimuovere l'altro ??

[boots]

prova così:

try { 

            $set = array(); 
            foreach ($data as $key => $value) { 
                $set[] ="$key = :$key"; 
            } 
            $sql = "UPDATE $table" 
                    . " SET ".implode(', ', $set)
                    . " WHERE $where"; 
            session::set('campi', $sql); 
            $sth = $this->prepare($sql); 
            foreach ($data as $key => $value) { 
                $sth->bindValue(":$key", $value); 
            } 

[luca200]

Ma siamo sicuri di quel $key = :$key ?

[boots]

Originariamente inviato da luca200
Ma siamo sicuri di quel $key = :$key ?

Direi di si, visto che poi fa il bind con i valori

foreach ($data as $key => $value) {  
                $sth->bindValue(":$key", $value);  
            }