web-dev-qa-db-ja.com

配列をIN()条件にバインドできますか?

PDOを使用して値の配列をプレースホルダーにバインドできるかどうかを知りたいのですが。ここでの使用例は、IN()条件で使用するために値の配列を渡すことを試みています。

こんなことができるようになりたいです。

<?php
$ids=array(1,2,3,7,8,9);
$db = new PDO(...);
$stmt = $db->prepare(
    'SELECT *
     FROM table
     WHERE id IN(:an_array)'
);
$stmt->bindParam('an_array',$ids);
$stmt->execute();
?>

そしてPDOをバインドして、配列内のすべての値をクォートします。

現時点で私はやっている:

<?php
$ids = array(1,2,3,7,8,9);
$db = new PDO(...);
foreach($ids as &$val)
    $val=$db->quote($val); //iterate through array and quote
$in = implode(',',$ids); //create comma separated list
$stmt = $db->prepare(
    'SELECT *
     FROM table
     WHERE id IN('.$in.')'
);
$stmt->execute();
?>

確かにそれは仕事をしますが、私が見逃している内蔵のソリューションがあるかどうか疑問に思いますか?

536
Andru

soulmergeは正しいと思います。あなたはquery-stringを構築する必要があります。

<?php
$ids     = array(1, 2, 3, 7, 8, 9);
$inQuery = implode(',', array_fill(0, count($ids), '?'));

$db = new PDO(...);
$stmt = $db->prepare(
    'SELECT *
     FROM table
     WHERE id IN(' . $inQuery . ')'
);

// bindvalue is 1-indexed, so $k+1
foreach ($ids as $k => $id)
    $stmt->bindValue(($k+1), $id);

$stmt->execute();
?>

修正:ダン、あなたは正しかった。コードを修正しました(テストはしていません)

edit:chris(comments)とsomebodyisintroubleの両方がforeachループを提案しました.

(...)
// bindvalue is 1-indexed, so $k+1
foreach ($ids as $k => $id)
    $stmt->bindValue(($k+1), $id);

$stmt->execute();

...は冗長かもしれないので、foreachループと$stmt->executeは単に...に置き換えることができます。

<?php 
  (...)
  $stmt->execute($ids);
?>

(繰り返しますが、テストしませんでした)

247
stefs

手短に言うと:

//$db = new PDO(...);
//$ids = array(...);

$qMarks = str_repeat('?,', count($ids) - 1) . '?';
$sth = $db->prepare("SELECT * FROM myTable WHERE id IN ($qMarks)");
$sth->execute($ids);
166
DonVaughn

INステートメントを使用することはそれほど重要ですか? FIND_IN_SET opを使用してみてください。

たとえば、PDOには次のようなクエリがあります。

SELECT * FROM table WHERE FIND_IN_SET(id, :array)

次に、このようにカンマで分解した値の配列をバインドするだけです。

$ids_string = implode(',', $array_of_smth); // WITHOUT WHITESPACES BEFORE AND AFTER THE COMMA
$stmt->bindParam('array', $ids_string);

そしてそれは完了です。

UPD:この回答に対するコメントで指摘しているように、明示的に述べるべき問題がいくつかあります。

  1. FIND_IN_SETはテーブルでインデックスを使用しません、そしてそれはまだ実装されていません - MYSQLバグトラッカーの このレコードを見てください 。通知をくれた@BillKarwinに感謝します。
  2. あなたは検索のための配列の値としてコンマの中の文字列を使うことはできません。セパレータとしてカンマ記号を使用するため、implodeの後でそのような文字列を正しい方法で解析することは不可能です。メモしてくれた@VaLに感謝します。

結局のところ、あなたがインデックスに強く依存しておらず、検索のためにカンマ付きの文字列を使用していないのであれば、私の解決策は上に挙げた解決策よりはるかに簡単、単純、そして速いでしょう。

45

私は多くの動的な問い合わせをするので、これは私が作った超単純なヘルパー関数です。

public static function bindParamArray($prefix, $values, &$bindArray)
{
    $str = "";
    foreach($values as $index => $value){
        $str .= ":".$prefix.$index.",";
        $bindArray[$prefix.$index] = $value;
    }
    return rtrim($str,",");     
}

このように使用してください。

$bindString = helper::bindParamArray("id", $_GET['ids'], $bindArray);
$userConditions .= " AND users.id IN($bindString)";

文字列:id1,:id2,:id3を返し、またクエリを実行するときに必要となるバインディングの$bindArrayも更新します。簡単です!

30
prograhammer

postgresの非常にきれいな方法はpostgres-array( "{}")を使うことです:

$ids = array(1,4,7,9,45);
$param = "{".implode(', ',$ids)."}";
$cmd = $db->prepare("SELECT * FROM table WHERE id = ANY (?)");
$result = $cmd->execute(array($param));
17
ESCOBAR

EvilRygyからの解決策は私のために働いていませんでした。 Postgresでは、他の回避策があります。


$ids = array(1,2,3,7,8,9);
$db = new PDO(...);
$stmt = $db->prepare(
    'SELECT *
     FROM table
     WHERE id = ANY (string_to_array(:an_array, ','))'
);
$stmt->bindParam(':an_array', implode(',', $ids));
$stmt->execute();
16
Sergey Galkin

私はPDOをstefsが提案するのと同じようなことをするように拡張しました、そしてそれは長期的に私にとってより簡単でした:

class Array_Capable_PDO extends PDO {
    /**
     * Both prepare a statement and bind array values to it
     * @param string $statement mysql query with colon-prefixed tokens
     * @param array $arrays associatve array with string tokens as keys and integer-indexed data arrays as values 
     * @param array $driver_options see php documention
     * @return PDOStatement with given array values already bound 
     */
    public function prepare_with_arrays($statement, array $arrays, $driver_options = array()) {

        $replace_strings = array();
        $x = 0;
        foreach($arrays as $token => $data) {
            // just for testing...
            //// tokens should be legit
            //assert('is_string($token)');
            //assert('$token !== ""');
            //// a given token shouldn't appear more than once in the query
            //assert('substr_count($statement, $token) === 1');
            //// there should be an array of values for each token
            //assert('is_array($data)');
            //// empty data arrays aren't okay, they're a SQL syntax error
            //assert('count($data) > 0');

            // replace array tokens with a list of value tokens
            $replace_string_pieces = array();
            foreach($data as $y => $value) {
                //// the data arrays have to be integer-indexed
                //assert('is_int($y)');
                $replace_string_pieces[] = ":{$x}_{$y}";
            }
            $replace_strings[] = '('.implode(', ', $replace_string_pieces).')';
            $x++;
        }
        $statement = str_replace(array_keys($arrays), $replace_strings, $statement);
        $prepared_statement = $this->prepare($statement, $driver_options);

        // bind values to the value tokens
        $x = 0;
        foreach($arrays as $token => $data) {
            foreach($data as $y => $value) {
                $prepared_statement->bindValue(":{$x}_{$y}", $value);
            }
            $x++;
        }

        return $prepared_statement;
    }
}

あなたはこのようにそれを使うことができます:

$db_link = new Array_Capable_PDO($dsn, $username, $password);

$query = '
    SELECT     *
    FROM       test
    WHERE      field1 IN :array1
     OR        field2 IN :array2
     OR        field3 = :value
';

$pdo_query = $db_link->prepare_with_arrays(
    $query,
    array(
        ':array1' => array(1,2,3),
        ':array2' => array(7,8,9)
    )
);

$pdo_query->bindValue(':value', '10');

$pdo_query->execute();
12
Chris

これが私の解決策です:

$total_items = count($array_of_items);
$question_marks = array_fill(0, $total_items, '?');
$sql = 'SELECT * FROM foo WHERE bar IN (' . implode(',', $question_marks ). ')';

$stmt = $dbh->prepare($sql);
$stmt->execute(array_values($array_of_items));

Array_valuesの使用に注意してください。これはキーの順序付けの問題を解決することができます。

私はIDの配列をマージしてから重複した項目を削除していました。私はこんな感じでした:

$ids = array(0 => 23, 1 => 47, 3 => 17);

そしてそれは失敗しました。

12
Progrock

他のパラメータがある場合は、次のようにします。

$ids = array(1,2,3,7,8,9);
$db = new PDO(...);
$query = 'SELECT *
            FROM table
           WHERE X = :x
             AND id IN(';
$comma = '';
for($i=0; $i<count($ids); $i++){
  $query .= $comma.':p'.$i;       // :p0, :p1, ...
  $comma = ',';
}
$query .= ')';

$stmt = $db->prepare($query);
$stmt->bindValue(':x', 123);  // some value
for($i=0; $i<count($ids); $i++){
  $stmt->bindValue(':p'.$i, $ids[$i]);
}
$stmt->execute();

PDOを見ると:定義済み定数にリストされているように必要となるPDO :: PARAM_ARRAYはありません> bindParam

ブールPDOStatement :: bindParam(混合$パラメーター、混合&$変数[、int $ data_type[、int $ length [、混合$ driver_options]]]))

だから私はそれが達成可能だとは思わない。

10
user83632

私にとってセクシーな解決策は動的連想配列を作成してそれを使うことです

// A dirty array sent by user
$dirtyArray = ['Cecile', 'Gilles', 'Andre', 'Claude'];

// we construct an associative array like this
// [ ':name_0' => 'Cecile', ... , ':name_3' => 'Claude' ]
$params = array_combine(
    array_map(
        // construct param name according to array index
        function ($v) {return ":name_{$v}";},
        // get values of users
        array_keys($dirtyArray)
    ),
    $dirtyArray
);

// construct the query like `.. WHERE name IN ( :name_1, .. , :name_3 )`
$query = "SELECT * FROM user WHERE name IN( " . implode(",", array_keys($params)) . " )";
// here we go
$stmt  = $db->prepare($query);
$stmt->execute($params);
10

私もこのスレッドは古くなっていることを認識していますが、間もなく廃止予定のmysqlドライバをPDOドライバに変換する際に、同じパラメータから通常のパラメータとINの両方を動的に構築できる関数を作成する必要がありましたパラメータ配列だから私はすぐにこれを構築しました:

/**
 * mysql::pdo_query('SELECT * FROM TBL_WHOOP WHERE type_of_whoop IN :param AND siz_of_whoop = :size', array(':param' => array(1,2,3), ':size' => 3))
 *
 * @param $query
 * @param $params
 */
function pdo_query($query, $params = array()){

    if(!$query)
        trigger_error('Could not query nothing');

    // Lets get our IN fields first
    $in_fields = array();
    foreach($params as $field => $value){
        if(is_array($value)){
            for($i=0,$size=sizeof($value);$i<$size;$i++)
                $in_array[] = $field.$i;

            $query = str_replace($field, "(".implode(',', $in_array).")", $query); // Lets replace the position in the query string with the full version
            $in_fields[$field] = $value; // Lets add this field to an array for use later
            unset($params[$field]); // Lets unset so we don't bind the param later down the line
        }
    }

    $query_obj = $this->pdo_link->prepare($query);
    $query_obj->setFetchMode(PDO::FETCH_ASSOC);

    // Now lets bind normal params.
    foreach($params as $field => $value) $query_obj->bindValue($field, $value);

    // Now lets bind the IN params
    foreach($in_fields as $field => $value){
        for($i=0,$size=sizeof($value);$i<$size;$i++)
            $query_obj->bindValue($field.$i, $value[$i]); // Both the named param index and this index are based off the array index which has not changed...hopefully
    }

    $query_obj->execute();

    if($query_obj->rowCount() <= 0)
        return null;

    return $query_obj;
}

論理はまだあるように見えますが、まだテストされていません。

それが同じ立場の誰かに役立つことを願っています、

編集:いくつかのテストの後、私は見つけました:

  • PDOは '。'が好きではありません。彼らの名前の中に(あなたが私に尋ねるとちょっとばかげている)
  • bindParamは間違った関数で、bindValueは正しい関数です。

実用版に編集されたコード。

9
Sammaye

Schnalleのコードについて少し編集

<?php
$ids     = array(1, 2, 3, 7, 8, 9);
$inQuery = implode(',', array_fill(0, count($ids)-1, '?'));

$db   = new PDO(...);
$stmt = $db->prepare(
    'SELECT *
     FROM table
     WHERE id IN(' . $inQuery . ')'
);

foreach ($ids as $k => $id)
    $stmt->bindValue(($k+1), $id);

$stmt->execute();
?>

//implode(',', array_fill(0, count($ids)-1), '?')); 
//'?' this should be inside the array_fill
//$stmt->bindValue(($k+1), $in); 
// instead of $in, it should be $id
8

どのデータベースを使用していますか? PostgreSQLでは、ANY(配列)を使うのが好きです。だからあなたの例を再利用する:

<?php
$ids=array(1,2,3,7,8,9);
$db = new PDO(...);
$stmt = $db->prepare(
    'SELECT *
     FROM table
     WHERE id = ANY (:an_array)'
);
$stmt->bindParam('an_array',$ids);
$stmt->execute();
?>

残念ながら、これはかなり移植性がありません。

他のデータベースでは他の人が言及しているようにあなた自身の魔法を作り上げる必要があるでしょう。もちろん、プログラム全体で再利用可能にするために、そのロジックをクラス/関数に入れたいと思うでしょう。 PHP.NETのmysql_queryページのコメントを見て、このシナリオの主題と例に関するいくつかのさらなる考察を得てください。

7
Ryan Bair

同じ問題を経験した後、私はもっと単純な解決策に進みました(PDO::PARAM_ARRAYほどエレガントではありませんでしたが)。

$ids = array(2, 4, 32)という配列が与えられます。

$newparams = array();
foreach ($ids as $n => $val){ $newparams[] = ":id_$n"; }

try {
    $stmt = $conn->prepare("DELETE FROM $table WHERE ($table.id IN (" . implode(", ",$newparams). "))");
    foreach ($ids as $n => $val){
        $stmt->bindParam(":id_$n", intval($val), PDO::PARAM_INT);
    }
    $stmt->execute();

... 等々

したがって、混在値配列を使用している場合は、param型を割り当てる前に、値をテストするためのコードがさらに必要になります。

// inside second foreach..

$valuevar = (is_float($val) ? floatval($val) : is_int($val) ? intval($val) :  is_string($val) ? strval($val) : $val );
$stmt->bindParam(":id_$n", $valuevar, (is_int($val) ? PDO::PARAM_INT :  is_string($val) ? PDO::PARAM_STR : NULL ));

しかし、私はこれをテストしていません。

4
alan_mm

私が知っているように、PDOステートメントに配列をバインドする可能性はありません。

しかし、2つの一般的な解決策があります。

  1. 位置指定プレースホルダ(?、?、?、?)または名前付きプレースホルダ(:id1、:id2、:id3)を使用する

    $ whereIn = implode( '、'、array_fill(0、count($ ids)、 '?'));

  2. 配列を早く引用

    $ whereIn = array_map(array($ db、 'quote')、$ ids);

どちらの方法も優れていて安全です。私はそれがより短いので私は第二の方が好きです、そして私がそれを必要とするならば私はvar_dumpパラメータができます。プレースホルダを使用して値をバインドする必要があり、最終的にはSQLコードは同じになります。

$sql = "SELECT * FROM table WHERE id IN ($whereIn)";

そして私にとって最後かつ重要なことは、「束縛変数の数がトークンの数と一致しない」というエラーを回避することです。

Doctrineは位置のプレースホルダを使う良い例です。それは入ってくるパラメータを内部的に制御するからです。

4
Oleg Matei

列に整数しか含めることができない場合は、おそらくプレースホルダーなしでこれを行い、クエリに直接IDを入れるだけで済みます。配列のすべての値を整数にキャストするだけです。このような:

$listOfIds = implode(',',array_map('intval', $ids));
$stmt = $db->prepare(
    "SELECT *
     FROM table
     WHERE id IN($listOfIds)"
);
$stmt->execute();

これはSQLインジェクションに対して脆弱ではないはずです。

4
Kodos Johnson

これが私の解決策です。私はまたPDOクラスを拡張しました:

class Db extends PDO
{

    /**
     * SELECT ... WHERE fieldName IN (:paramName) workaround
     *
     * @param array  $array
     * @param string $prefix
     *
     * @return string
     */
    public function CreateArrayBindParamNames(array $array, $prefix = 'id_')
    {
        $newparams = [];
        foreach ($array as $n => $val)
        {
            $newparams[] = ":".$prefix.$n;
        }
        return implode(", ", $newparams);
    }

    /**
     * Bind every array element to the proper named parameter
     *
     * @param PDOStatement $stmt
     * @param array        $array
     * @param string       $prefix
     */
    public function BindArrayParam(PDOStatement &$stmt, array $array, $prefix = 'id_')
    {
        foreach($array as $n => $val)
        {
            $val = intval($val);
            $stmt -> bindParam(":".$prefix.$n, $val, PDO::PARAM_INT);
        }
    }
}

上記のコードの使用例は次のとおりです。

$idList = [1, 2, 3, 4];
$stmt = $this -> db -> prepare("
  SELECT
    `Name`
  FROM
    `User`
  WHERE
    (`ID` IN (".$this -> db -> CreateArrayBindParamNames($idList)."))");
$this -> db -> BindArrayParam($stmt, $idList);
$stmt -> execute();
foreach($stmt as $row)
{
    echo $row['Name'];
}

どう考えているか教えてください

3
Lippai Zoltan

PDOではそのような配列を使用することはできません。

値ごとにパラメータを使って(または?を使って)文字列を作成する必要があります。例えば:

:an_array_0, :an_array_1, :an_array_2, :an_array_3, :an_array_4, :an_array_5

これが例です:

<?php
$ids = array(1,2,3,7,8,9);
$sqlAnArray = join(
    ', ',
    array_map(
        function($index) {
            return ":an_array_$index";
        },
        array_keys($ids)
    )
);
$db = new PDO(
    'mysql:dbname=mydb;Host=localhost',
    'user',
    'passwd'
);
$stmt = $db->prepare(
    'SELECT *
     FROM table
     WHERE id IN('.$sqlAnArray.')'
);
foreach ($ids as $index => $id) {
    $stmt->bindValue("an_array_$index", $id);
}

bindParamを使い続けたい場合は、代わりにこれを行うことができます。

foreach ($ids as $index => $id) {
    $stmt->bindParam("an_array_$index", $ids[$id]);
}

?プレースホルダーを使用したい場合は、次のようにします。

<?php
$ids = array(1,2,3,7,8,9);
$sqlAnArray = '?' . str_repeat(', ?', count($ids)-1);
$db = new PDO(
    'mysql:dbname=dbname;Host=localhost',
    'user',
    'passwd'
);
$stmt = $db->prepare(
    'SELECT *
     FROM phone_number_lookup
     WHERE country_code IN('.$sqlAnArray.')'
);
$stmt->execute($ids);

$idsが空かどうかわからない場合は、それをテストしてそれに応じて対処する必要があります(空の配列を返すか、Nullオブジェクトを返すか、例外をスローするなど)。

1

プレースホルダーを使用してパラメーターをバインドするという最初の質問に答えを近づけるには、もう少し時間がかかりました。

この答えは、クエリで使用される配列を2回ループする必要があります。しかし、それはより選択的なクエリのために他の列のプレースホルダーを持つという問題を解決します。

//builds placeholders to insert in IN()
foreach($array as $key=>$value) {
    $in_query = $in_query . ' :val_' . $key . ', ';
}

//gets rid of trailing comma and space
$in_query = substr($in_query, 0, -2);

$stmt = $db->prepare(
    "SELECT *
     WHERE id IN($in_query)";

//pind params for your placeholders.
foreach ($array as $key=>$value) {
    $stmt->bindParam(":val_" . $key, $array[$key])
}

$stmt->execute();
0
Joseph_J

あなたは最初に "?"の番号を設定しますクエリの中で、そして "for"によって次のようなパラメータを送信します。

require 'dbConnect.php';
$db=new dbConnect();
$array=[];
array_Push($array,'value1');
array_Push($array,'value2');
$query="SELECT * FROM sites WHERE kind IN (";

foreach ($array as $field){
    $query.="?,";
}
$query=substr($query,0,strlen($query)-1);
$query.=")";
$tbl=$db->connection->prepare($query);
for($i=1;$i<=count($array);$i++)
    $tbl->bindParam($i,$array[$i-1],PDO::PARAM_STR);
$tbl->execute();
$row=$tbl->fetchAll(PDO::FETCH_OBJ);
var_dump($row);
0
Ali Chegini