Как выводить данные из базы с id 1 при запросе ?id=1?

Question

[Александр Коновалов]

Здравствуйте! Делаю каталог товаров, нужно, чтобы по ссылке site.ru/catalog?id=1 выводилась карточка товара с id 1. Я так понимаю, что это делается при подключении к базе:

<?php
	$dbh = new PDO('mysql:dbname=root;host=localhost', 'bd', 'password');
	 
	$sth = $dbh->prepare("SELECT * FROM `catalog` ORDER BY `date_add` DESC");
	$sth->execute();
	$items = $sth->fetchAll(PDO::FETCH_ASSOC);
	 
	if (!empty($items)) {
	?>

Пробовал так, но не работает:

$sth = $dbh->prepare("SELECT * FROM `catalog` ORDER BY `date_add` DESC WHERE `id`=".intval($_GET['id'])."");

Answer 1

[SagePtr]

$sth = $dbh->prepare("SELECT * FROM `catalog` WHERE `id` = ? ORDER BY `date_add` DESC");
$sth->execute([intval($_GET['id'])]);

Или для читабельности кода, лучше так:

$id = intval($_GET['id']);

$sth = $dbh->prepare("SELECT * FROM `catalog` WHERE `id` = :id ORDER BY `date_add` DESC");
$sth->bindParam('id', $id, PDO::PARAM_INT);
$sth->execute();

Comments

[Александр Коновалов]

SagePtr спасибо! Но в таком случае при запросе site.ru/catalog ничего не выводится, нужно, чтобы в каталоге выводились все карточки

[Василий Банников]

Александр Коновалов, добавь if, который проверит, указан ли параметр id, и если не указан - выполнял бы запрос без WHERE id =

[Александр Коновалов]

Василий Банников,

if (isset($_GET['id'])) {
	$id = intval($_GET['id']);  
	$sth = $dbh->prepare("SELECT * FROM `catalog` WHERE `id` = :id ORDER BY `date_add` DESC");
	$sth->bindParam('id', $id, PDO::PARAM_INT);
	$sth->execute();
	$items = $sth->fetchAll(PDO::FETCH_ASSOC);
	}
	else { 
	$sth = $dbh->prepare("SELECT * FROM `catalog` ORDER BY `date_add` DESC");
	$sth->execute();
	$items = $sth->fetchAll(PDO::FETCH_ASSOC);
	} 
if (!empty($items)) {

Верно?

[Василий Банников]

Александр Коновалов, а оно работает?