Как реализовать поиск по нескольким полям?

Question

[Михаил Смирнов]

Как реализовать поиск в базе если нужно найти по одному или нескольким введённым данным. Делаю следующим образом
данные подготавливаю следующим образом:

function getParam($param, $conn, $default = '') {
    (isset($_POST[$param])) ? $data = '%'.mysqli_real_escape_string($conn, $_POST[$param]).'%' : $data = $default;
    return $data;
}

сам запрос:

$query = sprintf(
        "SELECT * FROM `people` WHERE 
        (LOWER(`surname`) LIKE LOWER('%s')
        OR LOWER('name') LIKE LOWER('%s')
        OR LOWER('fathername') LIKE LOWER('%s')
        OR LOWER('phone') LIKE ('%s'))",
        $data['surname'],
        $data['name'],
        $data['fathername'],
        $data['phone']
    );

В чём ошибка?

Comments

[Rsa97]

Основная ошибка - изобретение велосипеда вместо использования стандартных подготовленных запросов.

[alexalexes]

И будьте внимательными с кавычками.
LOWER(`surname`)
Тут опускается регистр значения поля surname - работаете с колонкой таблицы (обратные кавычки на клавише Ё).
LOWER('name')
Тут опускается регистр у строки 'name' - работаете не с колонкой таблицы, а буквально со строкой 'name' (обычные кавычки на клавише Э).

[Akina]

нужно найти по одному или нескольким введённым данным.

Если введено два значения - искать на соответствие обоим или хотя бы одному из двух?

Вообще - читать теорию по термину "фасетный поиск".

[Михаил Смирнов]

Спасибо большое

Answer 1

[Ипатьев]

Если вычистить из кода неверные кавычки и бессмысленные телодвижения, и переделать на "использование стандартных подготовленных запросов" (с) @Rsa97

$data = [
    "%{$_POST['surname']}%",
    "%{$_POST['name']}%", 
    "%{$_POST['fathername']}%",
    "%{$_POST['phone']}%",
];
$query = "SELECT * FROM `people` 
          WHERE `surname` LIKE ? OR `name` LIKE ? OR `fathername` LIKE ? OR `phone` LIKE ?";
$result = $conn->execute_query($query, $data);

Если используется устаревшая версия РНР, то либо добавить в неё execute_query руками, либо написать по-старинке,

$stmt = $conn->prepare($query);
$stmt->bind_param("ssss", ...$data);
$stmt->execute();
$result = $stmt->get_result();

Одно из преимуществ такого подхода в том, что мы как раз не путаемся в кавычках. Одинарные в запросе становятся не нужны

Comments

[Rsa97]

Если сохранить логику автора вопроса, то LIKE CONCAT('%', ?, '%')

[Ипатьев]

Rsa97, ну у него там уже это делается, хотя с ненужной mysqli_real_escape_string
но по хорошему да, это все тоже надо было выкинуть