Почему не работает данный запрос?

Question

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

Почему-то не работает функция которая добавляет статью в таблицу. Почему так?

spoiler

Код

<?php
    include_once("model/articlesFunctions.php");

    $fields = ['titel' => '', 'content' => ''];
    $err = '';

    if($_SERVER['REQUEST_METHOD'] === 'POST')   {
        $fields['titel'] = trim($_POST['titel']);
        $fields['content'] = trim($_POST['content']);

        if($fields['titel'] == '' || $fields['content'] == ''){
            $err = 'Заполните все поля!';
        }
        
        else{
            articlesAdd($fields);
            header('Location: index.php');
            exit();
        }
    }
?>

<div class="form">
	<form method="post">
		Name:<br>
		<input type="text" name="titel" value="<?=$fields['titel']?>"><br>
		Text:<br>
		<textarea name="text"><?=$fields['content']?></textarea><br>
		<button>Send</button>
		<p><?=$err?></p>
	</form>
</div>



    function articlesAdd(array $fields) : bool {
        $sql = "INSERT INTO articles ('titel', 'content') VALUES (:titel, :content)";
        dbQuery($sql, $fields);

        return true;
    }

Comments

[ThunderCat]

ошибка какая?

и что за слово такое titel?

[Adamos]

ThunderCat, ну, это как nmae, только titel.
И ЧСХ, очень последовательный titel - везде строго соблюден...

[Геннадий]

В ходе выполнения не возникает никаких ошибок? Поле в таблице точно называется titel (см. п.3)? Похоже на копипасту с ошибкой...
1. У вас форма и php-код находятся в одном файле? (Если в разных, то нужно указать атрибут action для формы).
2. Проверьте, что файл articlesFunctions.php запрашивается корректно (путь к файлу и т.д.). Разница между require и include в том, что require, в случае если не найдет указанный файл, выкинет фатальную ошибку, include же выведет предупреждение и продолжит выполнение кода.
3. Проверьте, что возвращает sql запрос, если до него дело, в принципе доходит. (Возможно, что выполнение кода заходит в вашу функцию, но проблема в самом sql-запросе). Попробуйте скопировать ваш запрос, подставить значения и выполнить его в mysql.
4. Если у вас настроен отладчик, пройдетесь им и посмотрите как выполняется ваш код.

[ThunderCat]

Adamos, Просто в голове сразу какие-то похабные ассоциации крутятся... сильно отвлекая от кода...

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

ThunderCat, у меня есть БД, где titel это заголовок, возможно глупо сделал БД.

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

Adamos, ну не совсем)))

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

Геннадий, просто не видел ваш комментарий, а тут так много написанного, что Я просто в шоке.

[mayton2019]

Что это за функция?
dbQuery($sql, $fields);
В API MySQL/Mariadb таких не видно. Кроме того, не видно как отрабатываются ошибки.

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

Это PDO

function dbQuery(string $sql, array $params = []) : PDOStatement{
		$db = dbInstance();
		$query = $db->prepare($sql);
		$query->execute($params);
		dbCheckError($query);
		return $query;
	}

[mayton2019]

Insert обычно возвращает число вставленных строк. Попробуй проверить. Предположительно у тебя конфликт ключа. Ты вставляешь titel который уже есть, но поскольку игнорируешь ошибки то ничего не видно.

Вот проверь примерно как тут https://stackoverflow.com/questions/20221220/php-h...

Я не спец в PHP но везде при работе с любыми API надо чекать код ошибки всех функций которые что-то делают. Но если лень чекать - то обрабатывай какой-то общий exception. Честно для меня - весь код - это ужас-ужас и разбираться в нем тяжело и писать так неряшливо нельзя.

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

mayton2019, просто я вводил через интерфейс БД и сделал по аналогии как в запросе интерфейса, так и в этом, ошибка есть, но почему-то ошибку понять не могу.

Answer 1

[Siverius]

Возможно, вот тут <form method="post"> не хватает action, куда шлются данные

Comments

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

Ну код взят и у человека работает нормально. Просо проблема вот в чём ещё, у него в $sql наименование подсвечиваются, а у меня они как текст, то есть, что-то не так.

[Геннадий]

Евгений Залецкий, Подсвечивает или нет, это зависит от того как настроен ваш редактор кода.

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

Геннадий, проблема в том, что код который Я беру за пример, он нормально работает, мой же не работает, хотя всё сделано под копирку и сейчас стараюсь понять в чём проблема.

[Дмитрий]

Структуру таблицы ещё бы глянуть, может там ещё есть поле, которое надо заполнить

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

Дмитрий, да, мне нужно было "id_user" вносить, уже исправил, но вылетает ошибка.

[Дмитрий]

Евгений Залецкий, что за ошибка?

Answer 2

[ThunderCat]

function articlesAdd(array $fields) : bool {
        $sql = "INSERT INTO articles ('titel', 'content') VALUES (:titel, :content)";
        var_dump([$fields,$sql]);exit();
        dbQuery($sql, $fields);
        return true;
    }

Подставляете поля в место плейсхолдеров, идете в консоль мускуля/пхпмайадмин, выполняете, смотрите что получилось.

Comments

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

spoiler

Вот код

function dbQuery(string $sql, array $params = []) : PDOStatement{
		$db = dbInstance();
		$query = $db->prepare($sql);
		$query->execute($params);
		dbCheckError($query);
		return $query;
	}

Вылетала ошибка на 24 строку, а именно на $query -> execute($params);

Вот

"

[ThunderCat]

Евгений Залецкий, ну так там написано что беда в запросе, проверьте что у вас действительно есть такие поля в бд, возможно там все таки title? И имена полей обычно заключаются в обратные кавычки, `titel` или пишутся без кавычек.

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

ThunderCat,

INSERT INTO `articles` (`id_article`, `titel`, `content`, `dt_add`, `id_user`, `id_categorie`) VALUES (NULL, 'Тестовая запись', 'Тестовая запись', current_timestamp(), '2', NULL);

Вот добавил запрос вручную.

[Геннадий]

Евгений Залецкий, попробуйте как вам выше написали, сделать запрос с обратными кавычками возле полей: $sql = "INSERT INTO articles (`titel`, `content`) VALUES (:titel, :content)";

[Дмитрий]

Евгений Залецкий, смотрите структуру, видать мускул настроен на strict и поля без default value не пропускает, на дату можно сразу автомат добавить в структуре

[ThunderCat]

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

Вот добавил запрос вручную.

ну и? Вставилось? А то сообщение не несет информации...

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

Дмитрий, дата в БД ставится автоматически, основные поля 2(которые нужно заполнить)