Топ дня: Lovecs

$arr = array('su' => $su, 'color' => $color, 'gold' => $gold);
$stmt = $connect->prepare("UPDATE `settings` SET `value` = ? WHERE `name` = ?");
foreach ($arr as $name => $value) {
$stmt->execute([$value, $name]);
}

(PDO)
Помогите найти ошибку/говнокодистость в этом запросе. Если она есть, не трольте бессмысленно, а обоснуйте.
p.s. прошу без оскорблений и т. п. как это обычно бывает:)

В массиве 3 ключа, в запросе 2 плейсхолдера

Avenax, я знаю, но ведь через цикл foreach каждый ключ обрабатывается по отдельности

Может так правильнее?
$arr = array('su' => $su, 'color' => $color, 'gold' => $gold);
$stmt = $connect->prepare("UPDATE `settings` SET `value` = ? WHERE `name` = ?");
foreach ($arr as $name => $value) {
$stmt->execute(array($value, $name));
}

$arr = array(
'su' => $su, 'color' => $color, 'gold' => $gold
);

$stmt = $connect->prepare('UPDATE settings SET value = ? WHERE name = ?');
foreach ($arr as $id => $bonus)
{
$stmt->execute([$bonus,$id]);
}

Отличается от моего только "хорошим оформлением кода" для читабельности, а ошибки там нет? как думаеш?

я даже знаю откуда ты это "перерисмовал"

rekmixa, http://phpfaq.ru/pdo#connect

rekmixa, если структура таблицы ок, то все должно быть ок

Avenax, ладно. спасибо за помощь