Как перенаправлять на 404?

Question

[DragonPro]

Добрый день, не могу решить проблему с перенаправлением людей на 404, если такого url нет, имеется динамический php под названием cars.php, из базы данных берутся все столбцы и столбец name_url, если в этом столбце данные совпадают с url, на котором находится клиент, то выводятся данные, например, name, title, price и т.д., но, если клиент ввел рандомный url, то выводится попросту пустой шаблон cars.php без каких-либо данных и ошибок, как выловить этот момент, чтобы этот рандомный url переадресовывал на my_404.php?

С таким кодом в любом случае я попадаю на my_404.php,:

function get_car_url($url) {
  global $dbh;
  if ($dbh->prepare("SELECT * FROM `cars` WHERE `name_url` != '$url'")) {
    http_response_code(404);
    include('my_404.php');
    die();
  }
  else {
    $cars = $dbh->prepare("SELECT * FROM `cars` WHERE `name_url` = '$url'");
    $cars->execute();
    foreach ($cars as $car) {
      return $car;
    }
  }
}

Comments

[Константин Б.]

О боже. Запросы не возвращают true/false они возвращают массив. А если учесть что вы ищите все автомобили в которых url не соответствует вашему то и найти ничего не сможете. Но так как массив есть значит и условие true. Первый запрос на помойку.
Далее нужно сначала выполнить запрос, а уже потом по данным проверять что в них содержится
Также PDO используется, но без плейсхолдеров, это тоже жуть
execute у вас почему-то вызывается где-то не там

[Ипатьев]

и никакого перенаправления на 404 не бывает
статус может быть либо только либо 404 либо 302 (перенаправление)

Answer 1

[Константин Б.]

function get_car_url($url) {
  global $dbh;
  $query = $dbh->prepare("SELECT * FROM `cars` WHERE `name_url` = ?");
  $query->execute([$url]);
$car = $query->fetch(PDO::FETCH_ASSOC);
  if (!isset($car['id'])) {
    http_response_code(404);
    include('my_404.php');
    die();
  }
  return $car;
}

Answer 2

[toxa82]

Если вам нужно сделать редирект то вы должны отправить заголовок "Location: /url"

Answer 3

[DragonPro]

Константин Б. помог с вопросом, если кому нужно будет, то данный код - решение:

function get_car_url($url) {
  global $dbh;
  $cars = $dbh->prepare("SELECT * FROM `cars` WHERE `name_url` = '$url'");
  $cars->execute();
  foreach ($cars as $car) {
    return $car;
  }
  if ($url <> $car["name_url"]) {
    http_response_code(404);
    include('my_404.php');
    die();
  }
}

Comments

[Константин Б.]

Это троллинг такой да?) ваш код неверен
Вы ищите запись по url, а потом проверяете ее на равенство с ним же

[Ипатьев]

Вы бы не позорились, а взяли решение Константина как есть.
Ваш текущий код такой же бессмысленный и небезопасный, как раньше.

[DragonPro]

Константин Б., $url выдает url после 'site.ru/', например, 'site.ru/kakashka', то $url выдаст 'kakashka'

[Константин Б.]

DragonPro, мой пример возьмите проверьте на корректность, так как я лет 7 назад писал на голом PHP и работайте с ним

[Владислав Лысков]

А мне интересно, зачем этот вброс, когда код Константига верный, и вы же отметили его решением

[DragonPro]

Константин Б., Ваш работает точно также, как и тот, который я указал

[Константин Б.]

DragonPro, ))) да конечно. Мы же тут все дураки сидим в первый раз все пишем вместе с вами

[Ипатьев]

Владислав Лысков, заслуженный ответ отметил решением я. А автор скромненько отметил свой.

DragonPro, объясняю последний раз:
Ваш код работает СОВСЕМ не так же, как код Константина.
Если на вашем цыплячьем примере код "работает", это не значит, что он будет работать всегда. Надо смотреть чуть дальше своего носа.
Если в $url будет ';DROP TABLE cars; -- то за такую "работу" будет очень больно.

У вас в коде есть бессмысленное условие, которое дублирует условие в запросе. То есть проблема тут даже не в коде, а в том что вы не понимаете, что пишете. А это важно для программиста.
Ваш код опасный. Через $url вам напихают полную панамку инъекций.

И если уж вам самому всё равно на это, то хотя бы не отмечайте свой ответ решением. "Если кому нужно будет" и он возьмёт ваш код, то поимеет все те же проблемы. То есть это не решение, а западло.