UPDATE ein Array mit PDO

Question

Ich erstelle ein mehrstufiges Formular für meine Benutzer. Sie können einige oder alle Felder aktualisieren. Also, ich muss die Werte senden, überprüfen, ob sie eingestellt sind und wenn ja, führen Sie eine UPDATE. Hier ist, was ich bisher:

public function updateUser($firstName, $lastName, $streetAddress, $city, $state, $zip, $emailAddress, $industry, $password, $public = 1, 
    $phone1, $phone2, $website,){ 

    $updates = array(
     'firstName'    => $firstName, 
     'lastName'    => $lastName, 
     'streetAddress'   => $streetAddress, 
     'city'     => $city, 
     'state'     => $state, 
     'zip'     => $zip, 
     'emailAddress'   => $emailAddress, 
     'industry'    => $industry, 
     'password'    => $password, 
     'public'    => $public, 
     'phone1'    => $phone1, 
     'phone2'    => $phone2, 
     'website'    => $website, 

); 

Hier ist mein PDO (na ja, der Anfang Versuch)

$sth = $this->dbh->prepare("UPDATE user SET firstName = "); //<---Stuck here 
    $sth->execute(); 
    $result = $sth->fetchAll(PDO::FETCH_ASSOC); 
    return $result; 

Grundsätzlich, wie kann ich die UPDATE Anweisung erstellen, damit es nur die Elemente in dem Updates Array, die nicht NULL sind?

Ich dachte über eine foreach Schleife wie folgt ausgeführt werden:

foreach($updates as $key => $value) { 
     if($value == NULL) { 
      unset($updates[$key]); 
     } 
    } 

aber wie würde ich die prepare Aussage schreiben, wenn ich von den Werten nicht sicher bin?

Wenn ich das komplett falsch mache, bitte weisen Sie mich in die richtige Richtung. Vielen Dank.

Answer 1

Zunächst einmal verwenden array_filter alle NULL-Werte zu entfernen:

$updates = array_filter($updates, function ($value) { 
    return null !== $value; 
}); 

Zweitens binden Parameter, die Ihre Live viel einfacher macht:

$query = 'UPDATE table SET'; 
$values = array(); 

foreach ($updates as $name => $value) { 
    $query .= ' '.$name.' = :'.$name.','; // the :$name part is the placeholder, e.g. :zip 
    $values[':'.$name] = $value; // save the placeholder 
} 

$query = substr($query, 0, -1).';'; // remove last , and add a ; 

$sth = $this->dbh->prepare($query); 

$sth->execute($values); // bind placeholder array to the query and execute everything 

// ... do something nice :) 

Answer 2

Die unten optimiert werden kann:

$i = 0; $query = array(); 
foreach($updates as $key => $value) { 
    if ($value != NULL) { 
     $query[] = "{$key} = :param_{$i}"; 
     $i++; 
    } 
} 

if (! empty($query)) { 
    $finalQuery = implode(",", $query); 
    $sth = $this->dbh->prepare('UPDATE user SET ' . $finalQuery); 

    $i = 0; 
    foreach($updates as $key => $value) { 
    if ($value != NULL) { 
     $sth->bindParam(':param_'.$i, $value, PDO::PARAM_STR); 
     $i++; 
    } 
    } 
}