Таблица products. В ней есть 3 поля по которым нужно выдавать конкретную строку. По GET запросу?

Question

[arsenaljek]

Дано: Таблица products. В ней есть 3 поля по которым нужно выдавать конкретную строку. Эти поля: idProduct(Это артикул товара может содержать, как цифры, так и буквы), linkProduct(это конкретный УРЛ товара) и поле name (это название товара)
Пользователь при вводе в форму своего запроса(обычно это или артикул или прямая ссылка) должен получить нужную ему строку из таблицы. Запрос идёт GET и я решаю это следующим образом

if (isset($_GET['sku'])) {
	if (is_numeric($sku)) { // если начало с цифры
	// if (mb_strlen($sku) >= 5 & mb_strlen($sku) <= 12) { // если запрос от 8 до 10 символов
		$sku = str_replace(" ","",$sku);
		$sql = "SELECT * FROM products WHERE idProduct = '".$sku."' LIMIT 0,1";
	} elseif ($sku == strpos($sku, 'https')) { // если начало с https
		$sku = str_replace(" ","",$sku);
		if ($_GET['sku'] = strpos($_GET['sku'], '?block_main')) {
			$sku = strstr($sku, '?block_main', true);
			$sql = "SELECT * FROM products WHERE linkProduct = '".$sku."' ";
		} elseif ($sku_new == true) {
			$sql = "SELECT * FROM products WHERE linkProduct = '".$sku."/' ";
		} else {
			$sql = "SELECT * FROM products WHERE linkProduct = '".$sku."' ";
		}
		// $sql = "SELECT * FROM products WHERE linkProduct LIKE '%".$sku."%' LIMIT 0,1";
	} else {
		$sql = "SELECT * FROM products WHERE name LIKE '%".$sku."%' LIMIT 0,1";
	}
}

Но мне это решение кажеться слишком корявым и оно не всегда отрабатывает т.к как задумано. Помогите его сделать более "уникальным" для возможности добавления условий.

Answer 1

[Дмитрий]

Такая строчка работать не может, strpos возвращает число
$sku == strpos($sku, 'https')

Я бы переписал так

$sku = isset($_GET['sku']) ? str_replace(" ", "", $_GET['sku']) : null;

if (is_numeric($sku)) { // если начало с цифры
    $filter = ['idProduct', $sku, true];  
} elseif (substr($sku, 0, 5) == 'https') { // если начало с https
    $beforeBlockMain = strstr($sku, '?block_main', true) || $sku;
    $filter = ['linkProduct', $sku . ($sku_new ? '/' : ''), false];
} elseif ($sku) {
    $filter = ['name', "%" . $sku . "%", true];
}

if ($filter) {
    $attrName = $filter[0];
    $value = $filter[1];
    $operator = $value[0] == "%" ? " LIKE " : " = ";
    $limit = $filter[2];

    $sql = "SELECT * FROM products WHERE " . $attrName . $operator . $value . ($limit ? " LIMIT 0,1" : "");
}

Comments

[arsenaljek]

Попробовал Ваш код. Вводил и цифры и урл и имя. И всегда возвращает
SELECT * FROM products WHERE nameLIKE %1%

[Дмитрий]

arsenaljek, Прошу прощения, не проверил сразу. Отредактировал ответ, проверьте.

[arsenaljek]

Дмитрий, да, этот вариант работает! Спасибо) Только вот с кавычками там проблема, они отсутствуют

SELECT * FROM products WHERE linkProduct = https://products/elektrogril

Ну и если в артикуле содержится буква, то тоже уже некорректное условие вызывает)

[Дмитрий]

arsenaljek, Насчет кавычек просто поменяйте составление $sql

$sql = "SELECT * FROM products WHERE " . $attrName . $operator ."'". $value ."'". ($limit ? " LIMIT 0,1" : "");

Насчет артикула - Возможно если в артикуле буква, то is_numeric возвращает false, соответственно проверка должна быть

is_numeric($sku[0])

видимо

[arsenaljek]

Дмитрий, ну тут буква не обязательно первая идёт, она так же может быть в середине и конце. Поэтому проверка по $sku[0] тут врятли поможет

[Дмитрий]

arsenaljek, это как раз проверка на цифру, а не на букву

[arsenaljek]

Дмитрий, А можете объяснить эти строки?

$beforeBlockMain = strstr($sku, '?block_main', true) || $sku;
    $filter = ['linkProduct', $sku . ($sku_new ? '/' : ''), false];

По идеи, когда урл вводиться, то он должен корректно отработать, если на конце есть / а также если на конце ?block_main то в запрос эти вещи не должны подставиться. Т.к будет != значению из базы, и строка не выведется

[Дмитрий]

arsenaljek, Тут я не в курсе, эти строки просто более краткая запись вашего кода. Первой строкой мы отсекаем ?blockMain и все что дальше него, второй устанавливаем данные для фильтра

[arsenaljek]

Дмитрий,
Вот как сейчас выглядит код. я добавил туда одно условие, которое убирает проблему со слешем

<? 
$sku = isset($_GET['sku']) ? str_replace(" ", "", $_GET['sku']) : null;

if (is_numeric($sku[0])) { // если начало с цифры
    $filter = ['idProduct', $sku, true];  
} elseif (substr($sku, 0, 5) == 'https') { // если начало с https
    if (substr($sku, -1) == '/') { // добавил это условие, чтобы не учитывать слэш
        $sku = substr($sku, 0, -1);
    }
    $beforeBlockMain = strstr($sku, '?block_main', true) || $sku;
    $filter = ['linkProduct', $sku . ($sku_new ? '/' : ''), false];
} elseif ($sku) {
    $filter = ['name', "%" . $sku . "%", true];
}
if ($filter) {
    $attrName = $filter[0];
    $value = $filter[1];
    $operator = $value[0] == "%" ? " LIKE " : " = ";
    $limit = $filter[2];

    $sql = "SELECT * FROM products WHERE " . $attrName . $operator ."'". $value ."'". ($limit ? " LIMIT 0,1" : "");
}
?>

Вот эта переменная нигде не используется. Соответвенно, она не отсекает, то что нужно отсечь

$beforeBlockMain = strstr($sku, '?block_main', true) || $sku;

И здесь, после вашей правки

$filter = ['linkProduct', $sku . ($sku_new ? '/' : ''), false];

Переменная $sku_new так же не используется. Её просто нету. Весь код я выложил