Правильное написание sql запроса?

Question

[lemonlimelike]

Такой sql запрос будет правильный?

$res = mysqli_query($connection, "SELECT * FROM `video_category` WHERE `category_id` AND `video_id` = " . (int)$_GET['id']);

Хотя в бд у category_id и video_id 'ID' разные

Comments

[Сергей Горностаев]

Это не SQL-запрос, это кусок PHP'шного кода.

[lemonlimelike]

Сергей Горностаев: Да блин, в там же есть запрос... Вы мне скажите, я правильно его написал? или все же так
SELECT * FROM `video_category` WHERE `category_id` = " . (int)$_GET['id'] . " AND `video_id` = " . (int)$_GET['id'] будет правильнее

[Сергей Горностаев]

lemonlimelike: первый вариант не будет работать, второй будет.

[lemonlimelike]

Сергей Горностаев: Так бы сразу, спасибо

Answer 1

[Saboteur]

$res = mysqli_query($connection, "SELECT * FROM `video_category` WHERE `category_id` AND `video_id` = " . (int)$_GET['id']);

У вас в этом запросе вообще ID для category_id не указан - это раз.

Во-вторых, если ID разные, то значит разные нужно подставлять - в форме, где вы запрашиваете ID, назовите их по-разному, например category_id и video_id и будет тогда $_GET['category_id'], $_GET['video_id'] - вам же не переменную нужно назвать id, а значение конкретное подставить.