Как производить проверку параметров GET?

Question

[Ivan Ivanovich]

Здравствуйте.

Есть файл news.php в который нужно передавать имя новости, которое берётся из БД.

Например: site.com/news?a=howtomakeit

Т.е. мы передаём всё в GET параметрах.

Я сделал такую проверку:

if (!empty($_GET['a'])){
require __DIR__ . '/php/db.php';
$stmt = $pdo->prepare('SELECT * FROM news WHERE url = ?');
$stmt->execute([$_GET['a']]);
} else {
  header('Location: 404.php');
}

Но понимаю, что этого не достаточно, потому что если мы допустим введём не существующий url site.com/news?a=fwuds, то редирект на страницу с ошибкой не будет работать.

Подскажите пожалуйста, как сделать такую проверку и вообще в целом какие нужно внести изменения.

Заранее спасибо.

Comments

[Михаил]

У тебя проверка стоит не на существование новости в бд, а на то, что передается ли что-либо в параметре "a", т.е. надо еще одно условие добавить, на присутствие новости в бд

[Valeron Sergeev]

if (empty($_GET['a'])) header('Location: 404.php');

require __DIR__ . '/php/db.php';
$stmt = $pdo->prepare('SELECT * FROM news WHERE url = ?');
$stmt->execute([$_GET['a']]);

if (!$stmt) header('Location: 404.php');

[Ivan Ivanovich]

Valeron Sergeev, Спасибо, а можете в ответы написать?

[Valeron Sergeev]

Ivan Ivanovich, секунду

[Rsa97]

Valeron Sergeev, execute вернёт false только если будет ошибка в самом запросе. Если же запрос правильный, но ни одной строки не найдено, то вернётся true.

[Valeron Sergeev]

Rsa97, в целом же суть понятна, сначала проверить наличие параметра, а потом после выборки уже существование записи

Answer 1

[Valeron Sergeev]

<?php
if (empty($_GET['a'])) header('Location: 404.php');

require __DIR__ . '/php/db.php';
$stmt = $pdo->prepare('SELECT * FROM news WHERE url = ?');
$stmt->execute([$_GET['a']]);

if (!$stmt) header('Location: 404.php');

Comments

[Ivan Ivanovich]

Valeron Sergeev Спасибо!

[Антон Р.]

Так а если запрос в базу ничего не вернул то переменная $stmt всё равно же будет существовать и перенаправление на 404 не сработает же? Тут еще нужна проверка что запрос в базу вывел больше 0 записей, по моему.

[Valeron Sergeev]

Антон Р., суть же понятна, сначала проверить наличие параметра, а уже потом проверить запись

Answer 2

[FanatPHP]

$row = [];
if (!empty($_GET['a'])){
    require __DIR__ . '/php/db.php';
    $stmt = $pdo->prepare('SELECT * FROM news WHERE url = ?');
    $stmt->execute([$_GET['a']]);
    $row = $stmt->fetch();
}
if (!$row);
    header('Location: 404.php');
    exit;
}