Что не так с запросом php?

Question

[chopik25]

Как правильно выстроить этот запрос.
Сам еще учусь....если что...

$masters = \R::getAll("SELECT * FROM `bel_master`INNER JOIN `beluser` WHERE category_id IN $ids AND bel_master.user_id = beluser.id LIMIT $start, $perpage");

Comments

[FanatPHP]

это не запрос, а какой-то урод.
Надо сначала научиться писать чистый SQL

[chopik25]

FanatPHP, есть способ тогда из урода сделать красавца?

[Slava Rozhnev]

Что вы хотите достичь Вашим запросом?

[chopik25]

Slava Rozhnev, если убрать category_id IN ($ids)
вот этот кусок, запрос работает..он связует 2 таблицы, user и article допустим...
Но именно этот кусок category_id IN ($ids)
отвечает за то что выводит название категории к которому он относиться

[chopik25]

chopik25, статья

[Akina]

Первое, что надо сделать, не собирать запрос прямо в операторе вызова. Сделай это в дополнительной переменной. Потом возьми то, что получилось в переменной, и, как посоветовал FanatPHP, попробуй превратить этого уродца во вменяемый SQL.

А когда получится - перейдёшь к пункту 2. Но он пока тебе не нужен...

Answer 1

[Эмиль]

Нужно для начала экранировать строки фунцией mysqli_escape_string.
Как это делать в redbean(точное название не помню, но помню что фреймворк начинается на R) я не знаю.

Comments

[Эмиль]

А во вторых строки в SQL нужно писать в одинарных кавычках.
В третьих, имена столбиков пишутся в наклонённых кавычках. Этот символ расположен на кнопке "Ё".
SELECT * FROM `collumn` = 'string'
И для PHP

$mysql=mysqli_connect();//подключение к MySQL
$string="Если мы напишем can't, let's,  isn't ain't и т.д то из-за одинарной кавычки(спецсимвол) MySQL запутается. Не говоря о иньекциях.";
$string=$mysql->escape_string($string);//А вот так уже нет! Иньекции и прочее не страшны.
$query = "SELECT * FROM `collumn` = '$string'";
$res =$mysql->query($query);//С экранированной строкой SQL отдаст нормальный запрос
var_dump($res->fetch_assoc());//результат