Как отменить повторную отправку формы? Знаю, что задавали уже вопрос, но они не подошли. Хелп?

Question

[Илхомжон Матазимов]

Доброго времени суток.
У меня на сайте есть функция добавления комментариев, но после обновления страницы пользователем, выходит повторная отправка формы. И данные комментария дублируются в БД, а так же выводится дублежом. И так бесконечно, пока будешь обновлять страницу.

Вот сама форма:

<label for="username">Ваше имя</label>
<input type="text" readonly name="username" value="<?=$_COOKIE['login']?>" id="username" class="form-control">
<label for="mess">Сообщение</label>
<textarea name="mess" id="mess" class="form-control"></textarea>
<button type="submit" id="mess_send" class="btn btn-success mt-3 mb-5">Добавить комментарий</button>

Вот код:

if($_POST['username'] != '' && $_POST['mess'] != '') {
$username = trim(filter_var($_POST['username'], FILTER_SANITIZE_STRING));
$mess = trim(filter_var($_POST['mess'], FILTER_SANITIZE_STRING));

$sql = 'INSERT INTO comments(name, mess, article_id) VALUES(?, ?, ?)';
$query = $pdo->prepare($sql);
$query->execute([$username, $mess, $_GET['id']]);
}

Comments

[Ankhena]

Очистка формы после отправки сообщения? и т.д.

Знаю, что задавали уже вопрос, но они не подошли.

Вы рассчитываете, что мы будем угадывать что вы уже пробовали и почему оно не подошло?

[Илхомжон Матазимов]

Ankhena, header('Location ') и тд не подошло. Так же с isset. Все варианты, которые были там - не подошли

[Ankhena]

jabrik,

которые были там - не подошли

Там? Там-там-та-да-да-там..

Нормально: это использовать ajax для таких дел.
Как при этом очистить форму есть в ссылке выше (там JQ, но сути это не меняет).

[AUser0]

Варианты, которые заведомо ДОЛЖНЫ были сработать, вам не подошли?
Окей, тогда переходите к заведомо неработающим...

[Роман Сарваров]

header('Location: **); подходит к вашей ситуации. Ищите проблему, почему этот вариант не сработал.

[Илхомжон Матазимов]

Роман Сарваров, Cannot modify header information - headers already sent by (output started at

[Роман Сарваров]

jabrik, если погуглить, то можно понять, что ваш header() выполняется после того, как был выведен какой-то текст на страницу (в вашем случая я предпологаю что это html с формой). Заголовки всегда должны выводиться вначале.

[Илхомжон Матазимов]

Роман Сарваров, иначе куда вписывать? Тут только после того, как сработает запрос на БД надо перенаправлять, иначе запрос в БД не будет сработан

[Илхомжон Матазимов]

AUser0, Вы такой интересный... Если не сработали, что я могу поделать? Я бы посмотрел на Вас, когда вы только только начинали кодить

[Роман Сарваров]

jabrik, выводите HTML код формы после кода, где записываете данные в БД и перенаправляете пользователя. Тогда ошибка исчезнет.

[AUser0]

Не думайте про то, что вы сделали запрос к MySQL, и поэтому нельзя сделать header("Location: ...").
Ищите, где у вас делается echo() (или аналогичное) ДО ВЫЗОВА функции header("Location: ...").

Хотя, если у вас уже готовы все скрипты, но в них ошибка, а вы не знаете её решения, и только-только начали кодить - то вы просто не поймёте и найдёте место вывода контента.
И тогда вам лучше искать учителя/профессионала, который посмотрит весь код, и покажет пальцем, в каком файле что делается. И всё-таки подскажет решение на основе вашей структуры скрпитов.

А так, без всех исходников перед глазами - получается тупое гадание на кофейной гуще. И в этой гуще вам указали, какие крупинки являются ответом, но вы их всё равно не найдёте по причине отсутствия опыта, ведь для вас все крупинки одинаковые.

Answer 1

[Илхомжон Матазимов]

Спасибо всем за ответы. Ошибка редиректа заключалась в том, что я перенаправляю не на самом начале php кода.
Решил вопрос с таким кодом, вставив его в начало кода:

<?php // on love
require_once 'mysql_connect.php';
$sql = 'SELECT * FROM `articles` ORDER BY `date` DESC';
$query = $pdo->query($sql);
while($row = $query->fetch(PDO::FETCH_OBJ)) {
  if($_POST['username'] != '' && $_POST['mess'] != '') {
    header("Location: /news.php?id=$row->id");
  } 
}
?>

Comments

[Abdula Magomedov]

что за шляпа?

[John Didact]

Я хоть и нубище дырявое и позорное, но этот код хуже даже меня! Заголовки в цикле!!! И без завершения работы!!! То есть, ты отправил несколько одинаковых заголовков (PHP скорее "починит" эту хрень, но всё же…), потом продолжаешь выполнять скрипт. Как по мне, может и ошибаюсь, но лучше так:
В начале проверять метод запроса, если POST и все данные корректны, если вставил запись в БД, то после всех махинаций необходимых (типа в сессию инфу записать и т.п.), отправлять ОДИН заголовок Location на активную страницу и выходить из скрипта, типа:

Header('Location: https://myass.oh/bidlo.php?id='. $id);
Exit();

Ну может и не так, хз, у дядинек умных спроси, они тебя пошлют тебе подскажут.

[Илхомжон Матазимов]

John Didact, нет. Там никак без этого. Если делать выход, то все по п&зде идет. Я знаю о чем ты, но в данном случае не надо дописывать выход

[John Didact]

jabrik, сомневаюсь… но ладно. Тогда, хотя бы, из цикла вытащи заголовок.

[AUser0]

jabrik, у вас выполняется header("Location...");, вам НАДО после него делать exit();!

Второй момент, вам незачем проходить по всей таблице `acticles` в базе данных, тратя на это время и ресурсы базы данных. Вот поверьте - тупо НЕ НАДО так делать!

Короче, просто поменяйте весь этот ваш неудачный код в начале сприпта на:

require_once 'mysql_connect.php';
if(!empty($_GET['id']) && isset($_POST['username']) && isset($_POST['mess']))
{
    $username = trim(filter_var($_POST['username'], FILTER_SANITIZE_STRING));
    $mess = trim(filter_var($_POST['mess'], FILTER_SANITIZE_STRING));

    if (!empty($username) && !empty($mess))
    {
        $sql = 'INSERT INTO comments(name, mess, article_id) VALUES(?, ?, ?)';
        $query = $pdo->prepare($sql);
        $query->execute([$username, $mess, $_GET['id']]);
    }
    header("Location: /news.php?id={$_GET['id']}");
    exit();
}

И внутри скрипта не забудьте удалить старый код, который дублирует функционал этого нового кода.

[Илхомжон Матазимов]

AUser0, Да-да, спасибова Вам. Я в тот момент только-только осваивал БД

Answer 2

[Danil Gaisin]

Самый простой способ.
После отправки формы сделайте редирект на ту же страницу.
А так много вариантов для решение этой проблемы) воспользуйтесь поиском.

Comments

[Илхомжон Матазимов]

Cannot modify header information - headers already sent by (output started at

[FanatPHP]

jabrik, Google