Как я могу написать меньше кода с помощью bindParam?

Question

Я использую PDO для вставки данных в таблицу. Проблема в том, что есть много столбцов, что означает, что запрос очень длинный.

Это код, который я в настоящее время:

$stmt = $con->prepare("INSERT INTO table (crips, biscuits, chocolate, cakes, smarties, yogurts, apples, oranges) VALUES (:crisps, :biscuits, :chocolate, :cakes, :smarties, :yogurts, :apples, :oranges)"); 
$stmt->bindParam(':crisps', $crisps); 
$stmt->bindParam(':biscuits', $biscuits); 
$stmt->bindParam(':chocolate', $chocolate); 
$stmt->bindParam(':cakes', $cakes); 
$stmt->bindParam(':smarties', $smarties); 
$stmt->bindParam(':yogurts', $yogurts); 
$stmt->bindParam(':apples', $apples); 
$stmt->bindParam(':oranges', $oranges); 
$stmt->execute(); 

Есть более простой способ, а не имея другой строки кода для каждого значения?

Возможно ли это как можно?

foreach(value) { 
    $stmt->bindParam(':value', $value); 
} 

Answer 1

Вы можете использовать ? идентификатор и покормить значения в следующем порядке:

$stmt = $con->prepare(" 
    INSERT INTO table 
    (crips, biscuits, chocolate, cakes, smarties, yogurts, apples, oranges) 
    VALUES 
    (?, ?, ?, ?, ?, ?, ?, ?) 
"); 

$stmt->execute([$crisps, $biscuits, $chocolate, $cakes, $smarties, $yogurts, $apples, $oranges]); 

Это экономит довольно много коды уже, :named параметров имеют свою область применения, но вы не можете использовать их в комбинации с ?.

---

Вы можете использовать цикл, но для этого требуется предварительно определенный массив.

Для именованных параметров создать массив следующим образом:

$list = [ 
    ':crisps' => $crisps, 
    ':biscuits' => $biscuits, 
    # etc 
]; 

foreach($list as $k => $v){ 
    $stmt->bindValue($k, $v); 
} 

$stmt->execute(); 

И это для ? индексированных параметров:

$list = [ 
    $crisps, 
    $biscuits, 
    #etc 
] 

foreach($list as $k => $v){ 
    $stmt->bindValue($k+1, $v); 
} 

$stmt->execute(); 

Пожалуйста, обратите внимание, что я изменил метод от bindParam() до bindValue() как в ваш код не имеет смысла использовать этот метод.

В этом случае возникает вопрос об использовании bindValue(), потому что этот дополнительный метод только для того, чтобы он мог присвоить значение определенному типу, используя параметр 3th, который вы ничего не делаете.

Таким образом, в сущности, вы можете избежать этого foreach петли все-вместе путем подачи $list в execute() метод непосредственно:

$stmt->execute($list); 

Если вы действительно хотите использовать bindParam(), то просто добавьте опорный символ перед именем переменной :

$list = [ 
    ':cakes' => &$cakes 
]; 

Answer 2

Ну ваш Еогеасп можно использовать, но вам нужно построить этот массив, который до сих пор занимает строки кода
- Марк Бейкер.

Это очень верно.

---

Примечание: Пожалуйста read here, почему я использую BindValue, а не BindParam. Благодаря Xorifelse для напоминания.

Codesee, Если вы действительно заботитесь о строках кода, я бы предложил вам создать себе класс взаимодействия с базой данных, который вы можете использовать для выполнения этих действий. После того, как класс написан, он сохранит ваше повторение макета кода.

Ваше представление об использовании foreach не за горами и может быть использовано для экономии времени, например, вы можете построить массив (в котором ссылочное имя переменной является ключом), чтобы тесно связать каждую переменную со своим соответствующим SQL заполнитель.

так:

$save['crisps'] = $crisps; 
$save['biscuits'] = $buiscuits; 
... 

Это заменить автономные переменные ($crisps и т.д.) с коллекцией (массив) переменных, с ключом массива является имя ссылки.

Тогда

foreach($save as $key=>$value){ 
    $stmt->bindValue(":".$key, $value); 
} 
unset($key,$value); 

ПРИМЕЧАНИЕ: Я не поставить в любой чистки безопасности или проверки процедур здесь, как это несколько выходит за рамки, но я рекомендую делать так (в preg_replace или аналогичный), чтобы обеспечить недопустимые символы предварительно удаляются из значения ключа.

Пример:

$key = preg_replace("/[^a-z]/i","",$key); 

---

Вы уверены, что мне нужно заранее определенный массив как таковой? Может ли это быть упрощено больше, поскольку $ v всегда будет таким же, как $ k, только с $ перед ним (если вы понимаете, что я имею в виду)?
- Codesee

Проблема с этим, как ваши переменные не собираются в массив или объект, как подразумевается код, чтобы узнать, какие переменные вы хотите добавить в INSERT SQL? Вам все равно нужна строка кода для каждого из них.

Answer 3

Xorifelse уже mede эту тему ясно, поэтому я могу сделать только небольшое дополнение. в PHP вы всегда можете найти нечетную функцию, которая может помочь вам сделать что-то экстравагантное.

Так что, если вы предпочитаете использовать именованные заполнители, вы можете держать с ними, но по-прежнему использовать ярлык с Execute():

$stmt = $con->prepare("INSERT INTO table (crips, biscuits, chocolate, cakes, smarties, yogurts, apples, oranges) VALUES (:crisps, :biscuits, :chocolate, :cakes, :smarties, :yogurts, :apples, :oranges)"); 
$stmt->execute(compact('crisps', 'biscuits', 'chocolate', 'cakes', 'smarties', 'yogurts', 'apples', 'oranges')); 

Но лично я предпочел бы позиционные заполнителей.