Сводка
В настоящее время я использую "$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.