Почему не работает insert?

Question

[lexstile]

Запрос:

public function query($sql, $params = []) {
		$stmt = $this->db->prepare($sql);
		if (!empty($params)) {
			foreach ($params as $key => $val) {
				if (is_int($val)) {
					$type = PDO::PARAM_INT;
				} else {
					$type = PDO::PARAM_STR;
				}
				$stmt->bindValue(':'.$key, $val, $type);
			}
		}
		$stmt->execute();
		return $stmt;
	}

Вызов:

$params = [
			'id' => '',
			'name' => $post['name'],
			'description' => $post['description'],
		];
		$this->db->query('INSERT INTO events VALUES (:id, :name, :description)', $params); // print_r
		// PDOStatement Object
		// (
		//     [queryString] => INSERT INTO events VALUES (:id, :name, :description)
		// )
		return $this->db->lastInsertId();

Этот код работает:

public function row($sql, $params = []) {
		$result = $this->query($sql, $params); // query тот же
		return $result->fetchAll(PDO::FETCH_ASSOC);
	}
	return $this->db->row('SELECT * FROM events ORDER BY id DESC LIMIT :start, :max', $params);

Версия php: PHP/7.1.11

Answer 1

[Rsa97]

Вангую, id у вас в базе имеет тип INT AUTO_INCREMENT, а вы пытаетесь записать в него пустую строку. Уберите id из запроса или используйте NULL вместо пустой строки.

Comments

[lexstile]

Попробовал и с NULL, и убрал id вовсе - результат одинаковый (не помогло).
По идее с пустой строкой тоже должно работать.
А так вы правы - id - int, auto_increment.

[Дмитрий]

lexstile, с null должно работать

[Rsa97]

lexstile, В логах что пишет?

[lexstile]

Дмитрий, не робит, может причина в другом?!

[Дмитрий]

lexstile, попробуйте не имёнованные плэйсхолдеры
И можно чуть красивее сделать

stmt->bindValue(':'.$key, $val, (is_int($val) ? PDO::PARAM_INT : PDO::PARAM_STR));

[Дмитрий]

lexstile, может константы не видит, пробросить в use

[lexstile]

Rsa97, включил вывод ошибок в PDO (все равно не доходит, почему не соотвествует):

Fatal error: Uncaught PDOException: SQLSTATE[21S01]: Insert value list does not match column list: 1136 Column count doesn't match value count at row 1 in C:\Users\Acer\Desktop\usbwebserver\root\application\lib\Db.php:28 Stack trace: #0 C:\Users\Acer\Desktop\usbwebserver\root\application\lib\Db.php(28): PDOStatement->execute() #1 C:\Users\Acer\Desktop\usbwebserver\root\application\models\Main.php(29): application\lib\Db->query('INSERT INTO eve...', Array) #2 C:\Users\Acer\Desktop\usbwebserver\root\application\controllers\MainController.php(32): application\models\Main->addEvent(Array) #3 C:\Users\Acer\Desktop\usbwebserver\root\application\core\Router.php(52): application\controllers\MainController->addAction() #4 C:\Users\Acer\Desktop\usbwebserver\root\index.php(13): application\core\Router->run() #5 {main} thrown in C:\Users\Acer\Desktop\usbwebserver\root\application\lib\Db.php on line 28

[Rsa97]

lexstile, А в таблице `events` какие-то ещё колонки кроме `id`, `name` и `description` есть? Если да, то в запросе надо указать, какие именно колонки заполняются при вставке.

[lexstile]

Rsa97, так, да - колонки есть:

[Rsa97]

lexstile, Ну вот. Вы не указали в запросе, какие именно колонки заполняются, поэтому MySQL и ждёт значения для всех колонок.

[lexstile]

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

[Rsa97]

lexstile, Никакого. Указывайте в запросе только заполняемые поля.

INSERT INTO `events` (`name`, `description`)
  VALUES (:name, :description)

[lexstile]

Rsa97, точно - у меня, получается, все поля накладывались - забыл, огромное человеческое спасибо!

[lexstile]

Дмитрий, да, согласен с вами, с тернарниками красивее смотрится, благодарю.