Почему выдает ошибку: Сайт выполнил переадресацию слишком много раз, когда ввожу header(Location: /)?

Question

[gazes12]

<?php
require 'config.php';
    $name = $_POST['name'];
    $surname = $_POST['surname'];
    $message = $_POST['message'];

    $errors = [];
    if($name == '' || $surname == '' || $message == ''){
        $errors[] = 'Не все поля заполнены!';
    }else if(!preg_match('/[a-z]/', $name || !preg_match('/[a-z]/', $surname) || !preg_match('/[a-z]/', $message))){
        $errors[] = 'В полях должны содержатся буквы!';
    }

    if(empty($errors)){
        $sql = "INSERT INTO `comments` (`name`, `surname`, `message`) VALUES(:name, :surname, :message)";
        $query = $conn->prepare($sql);
        $query->execute(['name' => $name, 'surname' => $surname, 'message' => $message]);
    }else{
        header('location: /');  /*После этой строки, выдает ошибку: Сайт выполнил переадресацию слишком много раз.*/
    }
?>

Главный файл

require 'php/config.php';
require 'php/comments.php';
    <div class="errors">
        <?php
            if(!empty($errors)){
                array_shift($errors);
            }
        ?>
    </div>

Comments

[Евгений Залецкий]

А что за странный редирект? У тебя одна страница? Почему не просто Location: имя главной страницы.

[SagePtr]

Дайте угадаю, код взяли у рандомного клоуна с Ютуба? В чём вообще логика при наличии ошибки выполнять переадресацию? Не логичнее ли текст ошибки вывести пользователю на экран? Ну а так как при переадресации все поля станут вновь пустыми, то сработает это: $errors[] = 'Не все поля заполнены!'; и переадресация выполнится снова, и снова, и снова, пока браузеру не надоест через 15 попыток.

[gazes12]

SagePtr, Ошибка должна выводится в другом php файле, в который я делаю переадресацию. Я передаю через get ошибку, и вывожу ее уже файле на который идет переадресация.

comments.php

if(empty($errors)){
        $sql = "INSERT INTO `comments` (`name`, `surname`, `message`) VALUES(:name, :surname, :message)";
        $query = $conn->prepare($sql);
        $query->execute(['name' => $name, 'surname' => $surname, 'message' => $message]);

        header('location: /');
    }else{
        header('location: /index.php?error='.array_shift($errors).' ');
    }

index.php

<div class="comments__errors">
                            <?php
                            if(isset($_GET['error'])){
                                echo $_GET['error'];
                            }
                            ?>
                        </div>

Как думаете нормальное решение проблемы? Или можно по лучше решить?

Answer 1

[ThunderCat]

1) зачем require 'config.php'; в обоих файлах?
2) судя по всему, код первого файла должен выполняться только при субмите формы, соответственно весь код должен быть обернут в проверку метода запроса, смотрите $_SERVER['REQUEST_METHOD'].

Comments

[gazes12]

ThunderCat, Да, первый файл выполняется при субмите кнопки. А что будет, если я оберну код первого файла в $_SERVER['REQUEST_METHOD']. ?

[ThunderCat]

gazes12,

А что будет, если я оберну код первого файла в $_SERVER['REQUEST_METHOD']. ?

Во первых не в $_SERVER['REQUEST_METHOD'], а в проверку. Во вторых - не помешает немного самостоятельной работы для изучения алгоритмов и логики работы приложений.

Хинт -

header('location: /'); /*После этой строки, выдает ошибку: Сайт выполнил переадресацию слишком много раз.*/

подумайте, почему здесь будет выполняться переадресация почти всегда?

[gazes12]

<?php
require 'config.php';
if($_SERVER['REQUEST_METHOD'] == 'POST'){
    $name = $_POST['name'];
    $surname = $_POST['surname'];
    $message = $_POST['message'];

    $errors = [];
    if($name == '' || $surname == '' || $message == ''){
        $errors[] = 'Не все поля заполнены!';
    }else if(!preg_match('/[a-z]/', $name || !preg_match('/[a-z]/', $surname) || !preg_match('/[a-z]/', $message))){
        $errors[] = 'В полях должны содержатся буквы!';
    }
    if(empty($errors)){
        $sql = "INSERT INTO `comments` (`name`, `surname`, `message`) VALUES(:name, :surname, :message)";
        $query = $conn->prepare($sql);
        $query->execute(['name' => $name, 'surname' => $surname, 'message' => $message]);
    }else{
        header('location: /');
    }
}
?>

Действительно, теперь не выдает ошибку, интересно почему?

[ThunderCat]

gazes12,

$name = $_POST['name'];
$surname = $_POST['surname'];
$message = $_POST['message'];
if($name == '' || $surname == '' || $message == ''){
$errors[] = 'Не все поля заполнены!';
} else...

что будет если я зайду сюда методом гет?

[gazes12]

ThunderCat, В переменные ничего не запишется. Как это повлияло на header location? Ошибка появлялась, когда я отправлял пост запрос.

[ThunderCat]

gazes12,

В переменные ничего не запишется.

Зато в ошибки запишется. И далее по коду смотрите что в этом случае будет происходить.