Используйте один bind_param () с переменным количеством входных переменных - PullRequest
8 голосов
/ 27 апреля 2009

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

$stmt = $mysqli->prepare("UPDATE mytable SET myvar1=?, myvar2=... WHERE id = ?")) {
$stmt->bind_param("ss...", $_POST['myvar1'], $_POST['myvar2']...);

но некоторые из $ _POST ['...'] могут быть пустыми, поэтому я не хочу обновлять их в БД.

Непрактично учитывать все различные комбинации пустых $ _POST ['...'], и хотя я могу построить строку "UPDATE mytable SET ..." для своих нужд, bind_param () - это другое зверь.

Я мог бы попытаться построить его вызов как строку и использовать eval () на нем, но это не правильно: (

Ответы [ 5 ]

25 голосов
/ 27 апреля 2009

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

$paramNames = array('myvar1', 'myvar2', /* ... */);
$params = array();
foreach ($paramNames as $name) {
    if (isset($_POST[$name]) && $_POST[$name] != '') {
        $params[$name] = $_POST[$name];
    }
}
if (count($params)) {
    $query = 'UPDATE mytable SET ';
    foreach ($params as $name => $val) {
        $query .= $name.'=?,';
    }
    $query = substr($query, 0, -1);
    $query .= 'WHERE id = ?';
    $stmt = $mysqli->prepare($query);
    $params = array_merge(array(str_repeat('s', count($params))), array_values($params));
    call_user_func_array(array(&$stmt, 'bind_param'), $params);
}
4 голосов
/ 27 апреля 2009

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

public function __construct($con, $query){
    $this->con = $con;
    $this->query = $query;
    parent::__construct($con, $query);
    //We check for errors:
    if($this->con->error) throw new Exception($this->con->error);
}

protected static $allowed = array('d', 'i', 's', 'b'); //allowed types

protected static function mysqliContentType($value) {
    if(is_string($value)) $type = 's';
    elseif(is_float($value)) $type = 'd';
    elseif(is_int($value)) $type = 'i';
    else throw new Exception("type of '$value' is not string, int or float");
    return $type;
}

//This function checks if a given string is an allowed mysqli content type for prepared statement (s, d, b, or i)
protected static function mysqliAllowedContentType($s){
    return in_array($s, self::$allowed);
}

public function feed($params){
    //These should all be empty in case this gets used multiple times
    $this->paramArgs = array();
    $this->typestring = '';
    $this->params = $params;
    $this->paramArgs[0] = '';
    $i = 0;
    foreach($this->params as $value){
        //We check the type:
        if(is_array($value)){
            $temp = array_keys($value);
            $type = $temp[0];
            $this->params[$i] = $value[$type];
            if(!self::mysqliAllowedContentType($type)){
                $type = self::mysqliContentType($value[$type]);
            }
        }
        else{
            $type = self::mysqliContentType($value);
        }
        $this->typestring .= $type;
        //We build the array of values we pass to the bind_params function
        //We add a refrence to the value of the array to the array we will pass to the call_user_func_array function. Thus say we have the following
        //$this->params array:
            //$this->params[0] = 'foo';
            //$this->params[1] = 4;
        //$this->paramArgs will become: 
            //$this->paramArgs[0] = 'si'; //Typestring
            //$this->paramArgs[1] = &$this->params[0];
            //$this->paramArgs[2] = &$this->params[1].
        //Thus using call_user_func_array will call $this->bind_param() (which is inherented from the mysqli_stmt class) like this:
            //$this->bind_param( 'si', &$this->params[0], &$this->params[1] );
        $this->paramArgs[] = &$this->params[$i];
        $i++;
    }
    unset($i);
    $this->paramArgs[0] = $this->typestring;
    return call_user_func_array(array(&$this, 'bind_param'), $this->paramArgs);
}

Вы используете это так:

 $prep = new theClassAboveHere( $mysqli, $query );
 $prep->feed( array('string', 1, array('b', 'BLOB DATA') );

Класс должен расширять класс mysqli_stmt.

Надеюсь, это поможет вам в правильном направлении.
Если вы не хотите, я также могу опубликовать весь класс, он включает в себя привязку переменных результатов.

1 голос
/ 27 апреля 2009

Намного понятнее построить ваш оператор с использованием массива:

$params = array();
$fragments = array();
foreach($_POST as $col => $val)
{
  $fragments[] = "{$col} = ?";
  $params[] = $val;
}

$sql = sprintf("UPDATE sometable SET %s", implode(", ", $fragments));
$stmt = $mysqli->prepare($sql);
$stmt->bind_param($params);
0 голосов
/ 19 июня 2013

array_insert не существует, я предполагаю, что он ссылается на какую-то самодельную функцию, но я не уверен точно, что она делает ... вставляет типы параметров в массив где-то в начале, я думаю, так как значение 0 пройдено, но эй, это тоже может быть в конце;)

0 голосов
/ 27 апреля 2009

Создайте его как строку, но поместите ваши значения в массив и передайте его в bindd_param. (и подставьте? для значений в вашей строке SQL.

$ stmt = $ mysqli-> prepare ("ОБНОВИТЬ mytable SET myvar1 = ?, myvar2 = ... WHERE id =?")) { $ stmt-> bind_param ("ss ...", $ _POST ['myvar1'], $ _POST ['myvar2'] ...);

Например:

$args = array();
$sql = "UPDATE sometable SET ";
$sep = "";
$paramtypes = "";
foreach($_POST as $key => $val) {
  $sql .= $sep.$key." = '?'";
  $paramtypes .= "s"; // you'll need to map these based on name
  array_push($args, $val);
  $sep = ",";
}
$sql .= " WHERE id = ?";
array_push($args, $id);
array_insert($args, $paramtypes, 0);

$stmt = $mysqli->prepare($sql);
call_user_func_array(array(&$stmt, 'bindparams'), $array_of_params);
$stmt->bind_param($args);
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...