Как правильно отдавать ответ 404?

Question

[arsenaljek]

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

if (isset($_GET['shop'])) {
    $offer = isset($_GET['shop']) ? str_replace(" ", "", $_GET['shop']) : null;
    $sql = "SELECT * FROM shops WHERE link_shop = '".$offer."' ";
    $result = mysqli_query($linkMain,$sql);
}

Т.е в случае ввода пользователем урла /view/?shop=mvideo.ru страница будет найдена.
Но он так же найдет ее, если пользователь введет что-то типо /view/?shop=mvideo.ru1 и т.п

Я попробовал обернуть вывод страницы в условие, что если строка найдена, то показываем страницу, иначе или возвращаем на рабочую страницу, редиректим на 404 или тут же отдаем заголовок header("HTTP/1.0 404 Not Found");

if (mysqli_num_rows($result) > 0) {

} else {
    
    // header("Location: /view/?shop=mvideo.ru");
// header("HTTP/1.0 404 Not Found");
    header("Location: /404/");
}

Как правильно здесь поступить, чтобы поисковики адекватно отреагировали?

Answer 1

[alekssamos]

header для поисковиков будет достаточно.

Comments

[arsenaljek]

т.е это нормально, что сработает 302 редирект?
header("Location: /404/");
Вот, чтобы будет при этом хедере, если человек введет /view/?shop=mvideo.ru1

spoiler

[galaxy]

Редирект на страницу - это глупость какая-то. Чем не устроило header("HTTP/1.0 404 Not Found")?

[arsenaljek]

galaxy, ок, а как сделать, чтобы открывалась 404 страница при таком хедере? я выставил хедер header("HTTP/1.0 404 Not Found");
Теперь пишет
Страница website не найдена
Не найдена страница для веб-адреса website /view_shop/?shop=mvideo.ru1
HTTP ERROR 404
в htaccess пишу так

RewriteRule ^404\/$ template/404.php [L]
ErrorDocument 404 /404

[galaxy]

Я уже не очень помню заморочки апача, но что-то мне подсказывает, что rewrite не работает на путь от ErrorDocument

[arsenaljek]

galaxy, Ну почему же, если сделать header("Location: /404/"); то он успешно средиректит на 404

[galaxy]

Кто он? В этом случае браузер редиректит.
А в htaccess вы хотите, чтобы по статусу 404 еще и срабатывал rewrite /404 -> emplate/404.php. Вот это имхо не сработает.

[arsenaljek]

galaxy, есть вариант решения? учитывая, что запрос в бд не вернет результат и сработает

} else {
header("HTTP/1.0 404 Not Found");
}

нужно, чтобы он бросил пользователя на страницу 404. А не просто вывалил ошибку браузера...
Кроме редиректа ничего в голову не приходит, но Вы говорите, что это плохая идея...

[alekssamos]

Значит в одном файле как бы и делай эту 404, какую-нибудь отдельную функцию сделай и в ней уже целую страницу, как так или заранее сделай html страницу заготовь, потом просто подключай её include

[galaxy]

Смысл директивы ErrorDocument в том, чтобы отдавать красивую страничку вместо по умолчанию.
Вот и заверстайте ее, положите в папку и сделайте:
ErrorDocument 404 /papka/404.html

Если это не подходит, выводите красивый html в php после header("HTTP/1.0 404 Not Found").