it-roy-ru.com

Используйте связанный параметр несколько раз

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

Однако возвращаемые данные выбираются из разных таблиц.

Каждый запрос использует $ term для поиска совпадений, и я привязал его к «: term» в качестве подготовленного параметра.

Теперь руководство говорит:

Вы должны включить уникальный маркер параметра для каждого значения, которое вы хотите передать в оператор при вызове PDOStatement :: execute (). Нельзя использовать именованный маркер параметра с одним и тем же именем дважды в подготовленном операторе.

Я решил, что вместо замены каждого параметра: term на: termX (x для term = n ++) должно быть лучшее решение?

Или мне просто нужно связать число Х: termX?

Редактировать Опубликовать мое решение для этого:

$query = "SELECT ... FROM table WHERE name LIKE :term OR number LIKE :term";

$term = "hello world";
$termX = 0;
$query = preg_replace_callback("/\:term/", function ($matches) use (&$termX) { $termX++; return $matches[0] . ($termX - 1); }, $query);

$pdo->prepare($query);

for ($i = 0; $i < $termX; $i++)
    $pdo->bindValue(":term$i", "%$term%", PDO::PARAM_STR);

Хорошо, вот образец. У меня нет времени на sqlfiddle, но я добавлю его позже, если это необходимо.

(
    SELECT
        t1.`name` AS resultText
    FROM table1 AS t1
    WHERE
        t1.parent = :userID
        AND
        (
            t1.`name` LIKE :term
            OR
            t1.`number` LIKE :term
            AND
            t1.`status` = :flagStatus
        )
)
UNION
(
    SELECT
        t2.`name` AS resultText
    FROM table2 AS t2
    WHERE
        t2.parent = :userParentID
        AND
        (
            t2.`name` LIKE :term
            OR
            t2.`ticket` LIKE :term
            AND
            t1.`state` = :flagTicket
        )
)
26
Daniel

Я сталкивался с одной и той же проблемой пару раз, и думаю, что нашел довольно простое и хорошее решение. Если я хочу использовать параметры несколько раз, я просто сохраняю их в MySQL User-Defined Variable.
Это делает код намного более читабельным, и вам не нужны никакие дополнительные функции в PHP:

$sql = "SET @term = :term";

try
{
    $stmt = $dbh->prepare($sql);
    $stmt->bindValue(":term", "%$term%", PDO::PARAM_STR);
    $stmt->execute();
}
catch(PDOException $e)
{
    // error handling
}


$sql = "SELECT ... FROM table WHERE name LIKE @term OR number LIKE @term";

try
{
    $stmt = $dbh->prepare($sql);
    $stmt->execute();
    $stmt->fetchAll();
}
catch(PDOException $e)
{
    //error handling
}

Единственным недостатком может быть то, что вам нужно выполнить дополнительный запрос MySQL - но, тем не менее, это того стоит.
Так как User-Defined Variables привязаны к сеансу в MySQL, также нет необходимости беспокоиться о переменной @term, вызывающей побочные эффекты в многопользовательской среде. 

15
low_rents

Я создал две функции, чтобы решить проблему, переименовав дважды используемые термины. Один для переименования SQL и один для переименования привязок.

    /**
     * Changes double bindings to seperate ones appended with numbers in bindings array
     * example: :term will become :term_1, :term_2, .. when used multiple times.
     *
     * @param string $pstrSql
     * @param array $paBindings
     * @return array
     */
    private function prepareParamtersForMultipleBindings($pstrSql, array $paBindings = array())
    {
        foreach($paBindings as $lstrBinding => $lmValue)
        {
            // $lnTermCount= substr_count($pstrSql, ':'.$lstrBinding);
            preg_match_all("/:".$lstrBinding."\b/", $pstrSql, $laMatches);

            $lnTermCount= (isset($laMatches[0])) ? count($laMatches[0]) : 0;

            if($lnTermCount > 1)
            {
                for($lnIndex = 1; $lnIndex <= $lnTermCount; $lnIndex++)
                {
                    $paBindings[$lstrBinding.'_'.$lnIndex] = $lmValue;
                }

                unset($paBindings[$lstrBinding]);
            }
        }

        return $paBindings;
    }

    /**
     * Changes double bindings to seperate ones appended with numbers in SQL string
     * example: :term will become :term_1, :term_2, .. when used multiple times.
     *
     * @param string $pstrSql
     * @param array $paBindings
     * @return string
     */
    private function prepareSqlForMultipleBindings($pstrSql, array $paBindings = array())
    {
        foreach($paBindings as $lstrBinding => $lmValue)
        {
            // $lnTermCount= substr_count($pstrSql, ':'.$lstrBinding);
            preg_match_all("/:".$lstrBinding."\b/", $pstrSql, $laMatches);

            $lnTermCount= (isset($laMatches[0])) ? count($laMatches[0]) : 0;

            if($lnTermCount > 1)
            {
                $lnCount= 0;
                $pstrSql= preg_replace_callback('(:'.$lstrBinding.'\b)', function($paMatches) use (&$lnCount) {
                    $lnCount++;
                    return sprintf("%s_%d", $paMatches[0], $lnCount);
                } , $pstrSql, $lnLimit = -1, $lnCount);
            }
        }

        return $pstrSql;
    }

Пример использования:

$lstrSqlQuery= $this->prepareSqlForMultipleBindings($pstrSqlQuery, $paParameters);
$laParameters= $this->prepareParamtersForMultipleBindings($pstrSqlQuery, $paParameters);
$this->prepare($lstrSqlQuery)->execute($laParameters);

Пояснение по поводу именования переменных:
p: параметр, l: локальный в функции
str: string, n: numeric, a: array, m: mixed

9
pascalvgemert

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

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

http://php.net/manual/en/pdo.prepare.php - (Выделение мое.)

Таким образом, технически разрешение эмулированной подготовки с использованием $PDO_obj->setAttribute( PDO::ATTR_EMULATE_PREPARES, true ); также будет работать; хотя это может не быть хорошей идеей (как обсуждалось в этот ответ , отключение эмулированных подготовленных утверждений является одним из способов защиты от определенных атак с использованием инъекций; хотя некоторые писали об обратном что это не имеет значения к безопасности, подражают ли приготовлениям или нет. (Я не знаю, но я не думаю, что последний имел в виду ранее упомянутую атаку.) 

Я добавляю этот ответ для полноты картины; когда я отключил emulate_prepares на сайте, над которым я работаю, это привело к сбою поиска, так как он использовал аналогичный запрос (SELECT ... FROM tbl WHERE (Field1 LIKE :term OR Field2 LIKE :term) ...), и он работал нормально, пока я явно не установил PDO::ATTR_EMULATE_PREPARES в false, затем он начал давать сбой ,.

(PHP 5.4.38, MySQL 5.1.73 FWIW) 

Этот вопрос подсказывает мне, что вы не можете использовать именованный параметр дважды в одном запросе (что мне кажется нелогичным, но, ну да ладно). (Каким-то образом я пропустил это в руководстве, хотя много раз просматривал эту страницу.)

5
Aaron Wallentine

Рабочий раствор:

$pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES, TRUE);
$query = "SELECT * FROM table WHERE name LIKE :term OR number LIKE :term";
$term  = "hello world";
$stmt  = $pdo->prepare($query);
$stmt->execute(array('term' => "%$term%"));
$data  = $stmt->fetchAll();
2
Your Common Sense

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

//Setting this doesn't work at all, I tested it myself 
$pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES, TRUE);

Я не хотел использовать определяемые пользователем переменные совсем как одно из решений, размещенных здесь. Я не хотел также делать переименование параметров как и другое решение, опубликованное здесь. Итак, вот мое решение, которое работает без использования пользовательских переменных и без переименования чего-либо в вашем запросе с меньшим количеством кода, и не заботится о том, сколько раз параметр используется в запросе. Я использую это на всех моих проектах, и это работает хорошо.

//Example values
var $query = "select * from test_table where param_name_1 = :parameter and param_name_2 = :parameter";
var param_name = ":parameter";
var param_value = "value";

//Wrap these lines of codes in a function as needed sending 3 params $query, $param_name and $param_value. 
//You can also use an array as I do!

//Lets check if the param is defined in the query
if (strpos($query, $param_name) !== false)
{
    //Get the number of times the param appears in the query
    $ocurrences = substr_count($query, $param_name);
    //Loop the number of times the param is defined and bind the param value as many times needed
    for ($i = 0; $i < $ocurrences; $i++) 
    {
        //Let's bind the value to the param
        $statement->bindValue($param_name, $param_value);
    }
}

И вот простое рабочее решение!

Надеюсь, это поможет кому-то в ближайшем будущем.

0
revobtz