Отправляется две формы на mysql. Что делать?

Question

[KurwaTM]

У меня отправляется две формы на mysql. Не знаю в чём ошибка. Я пробовал использовать
musqli_multi_query но что-то пошло не так :). А вот сам код:

<?php
include_once 'database.php';
$sucess = "";

if(isset($_POST['save'])) {
    $name = $_POST['name'];
    $product = $_POST['product'];
    $price = $_POST['price'];
    $email = $_POST['email'];
    $uid = $_POST['uid'];

    $sql = "UPDATE shtuck SET sht = sht - 1
      WHERE id='$uid'";

    $sqlu = "INSERT INTO tovar (name,product,price,email)
    VALUES ('$name','$product','$price','$email')";

    if ($mysqli = mysqli_query($conn, $sql)) {
        if ($mysqli = mysqli_query($conn, $sqlu)){
            $sucess = "Good!";
        }
    } else {
        echo "Error: " . $mysqli . "
    " . mysqli_error($conn);
    }
    mysqli_close($conn);
}
?>

Comments

[KurwaTM]

Александр Бехоли, Простите точней не формы а два продукта за одно нажатие на кнопку "Отправить"

[KurwaTM]

Александр Бехоли, Я хачу чтобы отправлялись &sql и $sqlu но $sqlu отправляется 2 штуки за раз

Answer 1

[KurwaTM]

Спасибо всем я ришил проблему тупо но я $sql и $sqlu поменял местами

Answer 2

[Slava Rozhnev]

В данном случае следует использовать транзакции

if(isset($_POST['save'])) {
    $name = $_POST['name'];
    $product = $_POST['product'];
    $price = $_POST['price'];
    $email = $_POST['email'];
    $uid = $_POST['uid'];

	mysqli_begin_transaction($mysqli);
	
	try {
		/* Добавление значений */
		$stmt = mysqli_prepare($mysqli, "INSERT INTO tovar (name,product,price,email) VALUES (?, ?, ?, ?)");
		mysqli_stmt_bind_param($stmt, 'ssss', $name, $product, $price, $email);
		mysqli_stmt_execute($stmt);
		
		$stmt = mysqli_prepare($mysqli, "UPDATE shtuck SET sht = sht - 1 WHERE id=?");
		mysqli_stmt_bind_param($stmt, 's', $uid);
		mysqli_stmt_execute($stmt);

		/* Если код достигает этой точки без ошибок, фиксируем данные в базе данных. */
		mysqli_commit($mysqli);
	} catch (mysqli_sql_exception $exception) {
		mysqli_rollback($mysqli);

		throw $exception;
	}
	
    mysqli_close($mysqli);
}

Тестировать данный PHP код можно здесь

Comments

[KurwaTM]

Спасибо но у меня в коде не работает много ошибок на версия PHP 7.4 и MYSQL 10.3

[FanatPHP]

Очень, очень неплохо! Выше среднего уровня тостера.
Но один типично нубский косячок все-таки проскочил.
Новички всегда тестируют только успешные сценарии, но почти некогда не проверяют свой код, который написан на случай возникновения непредвиденных ситуаций, например ошибок.
А это тоже следует делать.
Ну и по результатам вносить правки в код соответствующие.

Кстати, ищу авторов phpize.online как раз на эту тему - чтобы настроили mysqli правильно.
Форма обратной связи не работает :(

[Slava Rozhnev]

FanatPHP, спасибо за комплимент. По-поводу PHPize.onlin / SQLize.online обращайтесь: [e-mail] (про форму знаю, починю при возможности :) С косячком - поясните - буду рад критике

[FanatPHP]

о, круто!
можно попросить добавить к коннекту mysqli строчку

mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);

чтобы она тоже кидала исключения по умолчанию, как и PDO?

Сайт гениальный, кстати.
Такой песочницы, которая позволяла бы работать с БД, реально не хватало.

[Slava Rozhnev]

FanatPHP, Пожалуйста напишите мне со своей почты, не хотелось бы обуждать свой проект в комментариях к чежому вопросу :)

[FanatPHP]

А собственно мне же больше нечего писать, я уже всё написал.
больше не буду ничего обсуждать

[Slava Rozhnev]

FanatPHP, добавил mysqli_report

[FanatPHP]

спасибо!

С косячком - поясните - буду рад критике

ну собственно это он и был: без выставления режима выброса исключений для mysqli, catch ничего не поймает, и транзакцию не откатит.