Как вывести сообщение об ошибке?

Question

[Night386]

Добрый день всем, подскажите имеется данный код

<form class="subscribe" id="subscribe" method="POST" action="subscribe.php"> 
                            <p>Подписка на информационную рассылку</p>
							<div class="inputs">
                            <div class="d-flex">
                                <input class="form-control" type="email" name="email" id="email" required placeholder="Email">
                                <button class="btn" type="submit" name="submit_form" value="submit">
                                    <img src="/newdev/images/arrow-submit.svg" alt="">
                                </button>
                            </div>
							</div>
										 <div class="success" style="display: none;">Вы успешно подписаны</div>
                        </form>

if(isset($_POST['email'])) 
{
		 $email = $_POST["email"];
		 $query_result = db::i()->query("INSERT INTO s_mailer_list (added,email,cats) VALUES (NOW(),'$email',',С сайта,')");
		

}

вот сообщение выводится для успешной подписки так

$('#subscribe').on("submit", function (e) {
                e.preventDefault();

                var $target = $(e.currentTarget);
                var $inputs = $($target.find('.inputs'));
                var $success = $($target.find('.success'));

                $.post(e.currentTarget.action, $target.serialize(), function (data, status) {
                    if (status === 'success') {
                        $inputs.hide();
                        $success.show();
                    }
                })
            });

как можно вывести сообщение " пользователь с таким емейлом уже есть в базе"

колонка email unique

выдает при дубликате в браузере -

array(3) {
  ["message"]=>
  string(94) "SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '[e-mail]' for key 'email'"
  ["function"]=>
  string(5) "query"
  ["sql"]=>
  string(86) "INSERT INTO s_mailer_list (added,email,cats) VALUES (NOW(),'[e-mail]',',С сайта,')"
}

Comments

[Akina]

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

Надо создать уникальный индекс по этому полю в структуре таблицы. Тогда выполнение INSERT приведёт к ошибке, которая отлавливается на стороне PHP.

[Night386]

Akina, да есть такое , ошибка такая

array(3) {
  ["message"]=>
  string(94) "SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '[e-mail]' for key 'email'"
  ["function"]=>
  string(5) "query"
  ["sql"]=>
  string(86) "INSERT INTO s_mailer_list (added,email,cats) VALUES (NOW(),'[e-mail]',',С сайта,')"
}

а как ее можно вывести для людей ? не могу разобраться, это я только в браузере вижу в консоле

[Akina]

Night386, а вот это уже за пределами моих знаний - я только по части СУБД. Откорректируй вопрос (или создай новый) - типа "я получаю ошибку от MySQL, как вывести в браузер кастомное сообщение об этом".. и не забудь учесть, что это не единственная ошибка, которую может вернуть выполнение этого INSERT.

Answer 1

[Максим Кручинин]

ChatGPT на что?

Необходимо изменить запрос на добавление данных в базу, чтобы он не вызывал ошибку при дубликате, а вместо этого возвращал информацию о том, что пользователь с таким емейлом уже существует. Для этого можно использовать конструкцию INSERT IGNORE или INSERT ... ON DUPLICATE KEY UPDATE.

Пример с использованием INSERT IGNORE:

if (isset($_POST['email'])) {
    $email = $_POST["email"];
    $query_result = db::i()->query("INSERT IGNORE INTO s_mailer_list (added, email, cats) VALUES (NOW(), '$email', ',С сайта,')");
    if ($query_result) {
        // Пользователь успешно добавлен
        echo "success";
    } else {
        // Пользователь с таким емейлом уже существует
        echo "duplicate";
    }
}

или просто try...catch:

if (isset($_POST['email'])) {
    $email = $_POST["email"];

    try {
        $query_result = db::i()->query("INSERT INTO s_mailer_list (added, email, cats) VALUES (NOW(), '$email', ',С сайта,')");
        // The query was successful, so the user is subscribed.
        echo "success";
    } catch (Exception $e) {
        // Check if the exception is related to a duplicate entry for the 'email' column.
        if ($e->getCode() === 23000) {
            // This code represents a MySQL duplicate key error (Integrity constraint violation).
            // Handle the case when the email is already in the database.
            echo "duplicate";
        } else {
            // Handle other exceptions if needed.
            echo "error";
        }
    }
}

Дальше JS сам, думаю, допишешь.

Comments

[Night386]

Доброй ночи! спасибо! а почему может быть такое когда вписываю email любой который есть в бд или нет , выдает success? попробовал оба метода

[AUser0]

Night386, потому что перед добавлением лучше проверять существование такой записи. И выводить ошибку, если запись уже существует. А INSERT IGNORE - плохая практика, положительный результат при отсутствии результата. Тем более INSERT ... ON DUPLICATE KEY UPDATE - черезвычайно плохая практика, можно поменять/поломать подписки всем пользователям, почтовые ящики которых известны.

P.S. А полное отсутствие защиты от SQL Injections - за такое розгами надо пороть. Прилюдно.

[Night386]

AUser0, я понял, только как это все сделать ?)

[Night386]

AUser0, попробовал вот так опять пишет запись существует и не добавляет другие записи теперь

if(isset($_POST['email'])) {
    $email = $_POST["email"];

    $check = db::i()->query("SELECT * FROM s_mailer_list WHERE email = '".$email."'");
    if($check != null)  {echo "Запись уже существует";}
    else {
        $sql = db::i()->query("INSERT INTO s_mailer_list (added,email,cats) VALUES (NOW(),'$email',',С сайта,')");
		echo "OK";
    }
	}

[AUser0]

Night386, а вы проверяли, чему равно $check при наличии/отсутствии записи в БД? Там точно null либо не null?

Не добавляет другие записи? Вам нужно добавит несколько записей?

[Night386]

AUser0, вот так я еще раз попробовал , если запись имеется то check выдает true , если нет записи то пишется в базу, но уведомления и что есть запись что нет - success

if (isset($_POST['email'])) {
    $email = $_POST["email"];
	$check = db::i()->query("SELECT EXISTS(SELECT `email`  FROM `s_mailer_list` WHERE `email` = '".$email."' LIMIT 1) AS exist");
    try {
        $query_result = db::i()->query("INSERT INTO s_mailer_list (added, email, cats) VALUES (NOW(), '$email', ',С сайта,')");
        // The query was successful, so the user is subscribed.
        echo "success";
    } catch (Exception $e) {
        // Check if the exception is related to a duplicate entry for the 'email' column.
        if ($e->getCode() === 23000) {
            // This code represents a MySQL duplicate key error (Integrity constraint violation).
            // Handle the case when the email is already in the database.
            echo "duplicate";
        } else {
            // Handle other exceptions if needed.
            echo "error";
        }
    }
}

[AUser0]

Night386, поздравляю, у вас нет проверки на наличие записи. Вне зависимости от результата SELECT у вас выполняется код в try{}, и если там произошла ошибка и db::i()-&query() выкинуло Exception - выполнится catch{}. $check ни где не проверяется, можете удалить SELECT - ничего не поменяется, гы-гы-гы!

[Night386]

AUser0, а как можно сделать такую проверку? не подскажете? уже столько вариантов перепробовал(

[AUser0]

Night386, так напрягитесь и узнайте точное содержимое $check при наличии и отсутствии записи в Базе Данных. От этого и пляшите.

[Night386]

AUser0, понял , пока не получилось конечно, может не те условия пишу как то

[AUser0]

Night386, как-то напрягитесь и узнайте точное содержимое $check при наличии и отсутствии записи в Базе Данных. Это не просто, но это надо сделать, как-то собраться и как-то напрячься.

[Night386]

AUser0, когда запись есть $check выдает true , когда нету тоже самое тру

[AUser0]

Night386, ну вот, уже конкретный результат. Теперь смотрите другие запросы, из которых нормально вытаскиваются данные - и повторяйте весь алгоритм здесь, проверяйте возвращаемое значение `email` на не пустоту, и т.д.

[Night386]

AUser0, спасибо! буду пробовать