Как правильно избежать запроса «поиск / лайк» в MySQL? - PullRequest
4 голосов
/ 21 декабря 2011

Сводка

В настоящее время я использую "$search_field LIKE '$this->db->escape_like_str($search_string)%'"; для экранирования динамически создаваемых поисковых запросов.Созданные в результате операторы SQL не выдают никаких ошибок, но также не дают никаких результатов.Ниже приводится подробное описание того, что я делаю.

Подробности

Я использую jqGrid и его функцию поиска.Когда пользователь вводит условия поиска, он отправляет объект $filters json на мой сервер.Затем я анализирую его и создаю инструкцию SQL для получения запрошенных данных.

Вот код для экранирования входящих данных поиска (это также проблемная область):

$search_string_like = $this->CI->db->escape_like_str($search_string);
$operator['bw'] = "$search_field LIKE '$search_string_like%'"; //begins with

Вот результирующий оператор SQL:

SELECT *
FROM player_data_temp_table
WHERE first_name LIKE '\'zech\'%' AND last_name LIKE '\'camp\'%'
ORDER BY date_won desc
LIMIT 0 , 15

Этот запрос не выдает никаких ошибок, но он также не работает.Когда я запускаю аналогичный запрос непосредственно в phpmyadmin, я получаю MySQL returned an empty result set (i.e. zero rows)., хотя я знаю, что результаты можно найти.Если я просто уберу обратную косую черту и одинарные кавычки из first_name LIKE '\'zech\'%', чтобы сделать его first_name LIKE 'zech%', я получу ожидаемые результаты.Мое беспокойство в том, что это больше не правильно избежать, верно?

Код, использованный для построения запроса

Сводка

Переменная $filters с такими данными, как {"groupOp":"AND","rules":[{"field":"first_name","op":"bw","data":"zech"}]}, передается в build_where_clause($filters).build_where_clause возвращает полный оператор $where и затем используется в модели для создания окончательного оператора поиска SQL.

jqgrid_lib.php

class jqgrid_lib 
{
private $CI;

public function __construct()
{
    $this->CI =& get_instance();
}

/**
 * Function takes a json string with search rules and turns it into an sql statement.
 *
 * To use this function make sure you set stringResult: true, see example below:
 *   $("#list").jqGrid('filterToolbar',{stringResult: true});
 * @param   json string     
 * @author zechdc
 */
public function build_where_clause($filters)
{
    $sql_fragments = array();

    $filters = json_decode($filters);
    $rules = $filters->rules;
    $group_op = $filters->groupOp;

    //loop through each rule and create an sql statement
    foreach($rules as $rule)
    {
        $temp_sql = $this->create_search_field($rule->field, $rule->data, $rule->op);
        array_push($sql_fragments, $temp_sql);
    }

    //combine all sql fragments with the group_operator
    $data['sql'] = implode(' ' . $group_op . ' ', $sql_fragments);

    return $data;
}

/**
 * Takes a field, string and search condition and turns it into a sql search statement
 *
 * To use this function make sure you set stringResult: true, see example below:
 *   $("#list").jqGrid('filterToolbar',{stringResult: true});
 * @param   json string 
 * @return  string  
 * @author  zechdc
 */
public function create_search_field($search_field, $search_string, $search_operator)
{   
    //$search_field = $this->CI->db->escape($search_field); //escaping the column breaks it.
    $search_string = $this->CI->db->escape($search_string);
    $search_string_like = $this->CI->db->escape_like_str($search_string);
    //$search_string_like = $search_string;

    $operator['eq'] = "$search_field=$search_string"; //equal to
    $operator['ne'] = "$search_field<>$search_string"; //not equal to
    $operator['lt'] = "$search_field < $search_string"; //less than
    $operator['le'] = "$search_field <= $search_string "; //less than or equal to
    $operator['gt'] = "$search_field > $search_string"; //less than
    $operator['ge'] = "$search_field >= $search_string "; //less than or equal to
    $operator['bw'] = "$search_field LIKE '$search_string_like%'"; //begins with
    $operator['bn'] = "$search_field NOT LIKE '$search_string_like%'"; //not begins with
    $operator['in'] = "$search_field IN ($search_string)"; //in
    $operator['ni'] = "$search_field NOT IN ($search_string)"; //not in
    $operator['ew'] = "$search_field LIKE '%$search_string_like'"; //ends with
    $operator['en'] = "$search_field NOT LIKE '%$search_string_like%'"; //not ends with
    $operator['cn'] = "$search_field LIKE '%$search_string_like%'"; //in
    $operator['nc'] = "$search_field NOT LIKE '%$search_string_like%'"; //not in
    $operator['nu'] = "$search_field IS NULL"; //is null
    $operator['nn'] = "$search_field IS NOT NULL"; //is not null

    if(isset($operator[$search_operator])) 
    {
        //set the sql search statement
        return $operator[$search_operator];
    } 
}
}

Модель

/*
 * Gets all columns from table with limit and sort order set dynamically
 */
function get_specific($sidx, $sord, $start, $limit, $where = NULL)
{
    $result = FALSE;

    if($where)
    {
        $where = ' WHERE ' . $where;
    }

    // usually I dont do select all but since this whole table is temp and only holds the needed data
    // then just do select all.
    $sql = "SELECT *
            FROM player_data_temp_table
            $where
            ORDER BY $sidx $sord
            LIMIT $start , $limit"; 

    $q = $this->db->query($sql);

    if($this->db->affected_rows() > 0)
    {
        $result = $q->result();
    }

    return $result;
}

ОБНОВЛЕНИЕ / Ответ:

Похоже, я исправил проблему.В модели я удалил сделанный вручную оператор $ sql и заменил его на

    if($where)
    {
        $this->db->where($where);
    }

    $this->db->order_by($sidx, $sord);
    $q = $this->db->get('player_data_temp_table', $limit, $start);

, который, похоже, правильно экранировал все переменные, включая имена моих столбцов в операторе $ where.

1 Ответ

1 голос
/ 21 декабря 2011

Руководство

$ this-> db-> escape_like_str () Этот метод следует использовать, когда строки должны использоваться в условиях LIKE, чтобы подстановочные знаки LIKE ('%', '_') в строке также правильно экранировались.

$search = '20% raise';
$sql = "SELECT id FROM table WHERE column LIKE '%".$this->db->escape_like_str($search)."%'";

Итак, вы делаете это правильно в последних двух строках кода.


Что делает для вас следующий код?

$search_string = 'zech';
$search_string_like = $this->CI->db->escape_like_str($search_string);
$operator['bw'] = "$search_field LIKE '$search_string_like%'";
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...