Почему в таблицу базы данных добавляются сразу 2 новых записи, а не одна?

Question

[Вячеслав Лебедев]

При обновлении страницы, в таблицу добавляются 2 записи. В коде найти не могу, почему 2, а не одна.
Помогите, уже минут 30 тыкаю туда сюда... И query() выполняется 1 раз, но почему записи то 2 :(
Использую для подключения к базе PDO.

Выкладываю кодики:
index.php

$sth = $dataBase->insert('tasks', array('task_title' => 'Третья запись', 'task_desc' => 'Ура, появилась третья запись!'));
echo '<br>';
var_dump($sth);

Файл работы с базой:

public function insert($tableName, $row = array()) {
		if (!count($row)) return false;
		$tableName = $this->getTableName($tableName); // Вернет имя таблицы с учетом префикса
		$fields = '(';
		$values = 'VALUES (';
		$params = array();
		foreach ($row as $key => $value) {
			$fields .= "`$key`,";
			$values .= "?,";
			$params []= $value;
		}
		// Удаление последнего символа ','
		$fields = substr($fields, 0, -1).')';
		$values = substr($values, 0, -1).')';
		$query = "INSERT INTO `$tableName` $fields $values";

		echo $query.'<br>';
		var_dump($params);

		return $this->query($query, $params);
	}

        private function query($query, $params) {
		echo '<br>YO!';
		$STH = $this->_DBH->prepare($query);
		if (!$STH) return false;
		if (isset($params)) $STH->execute($params);
		else $STH->execute();
		// Если НЕ производилась вставка, вернет true, иначе вернет id вставленной записи
		if ($this->_DBH->lastInsertId() === 0) return true;
		else return $this->_DBH->lastInsertId();
	}

p.s.: кажется ничего не забыл добавить.

UPD: Ответ крылся в настройках .htaccess, а именно в этих строчках:

RewriteCond %{REQUST_FILENAME} !-f
RewriteCond %{REQUST_FILENAME} !-d
RewriteRule .* index.php [L]

Переход на index.php порождал перенаправление на index.php.
Что было абсолютно не предсказуемым поведением для меня :)

Comments

[Ростислав Григорьев]

$this->_DBH->prepare($query); // что здесь?

[Вячеслав Лебедев]

Зря обрадовался новому комменту...
habrahabr.ru/post/137664

[rafuck]

выводите все отладочные сообщения в файл c "a+", а потом вдумчиво его просмотрите :) По приведенному вами коду неясно по какому событию происходит INSERT и может ли это событие произойти два раза

[rafuck]

И еще. $params = '`'.implode('`,`', array_keys($row)).'`'; $values = str_repeat('?,', count($row) - 1). '?';

[rafuck]

ну и array_values(...) сократят ваш код раза в два

[Вячеслав Лебедев]

Вот оно, событие это:

$sth = $dataBase->insert('tasks', array('task_title' => 'Третья запись', 'task_desc' => 'Ура, появилась третья запись!'));

Как оно 2 раза происходит?

[Вячеслав Лебедев]

И в файл ничего выводить не нужно, вот эта стройка вполне справляется:
echo '<br>YO!';
Выводится ровно 1 раз :) а в таблице +2 записи, как такое возможно...

[rafuck]

Вы же не привели полный код вашего index.php Может, оно там рефрешится у вас на уровне HTTP Location или на уровне JS, или meta. В итоге в браузере вы видите результат лишь одного инсерта, а на деле их два. Это я включил телепатию. Без _полного_ кода мне не понять. А вам помогут понять логи.

[Ростислав Григорьев]

Раз такая "мистика", попробуй проставить нормально фигурные скобки в условиях. Метод execute() в коде прописан два раза, но там условие, однако вдруг зависит от версии php или модулей и еще учитывая эти так называемые "Безымянные placeholder’ы". То есть так:
if (isset($params)) { $STH->execute($params); echo 'EXECUTE1';}else { $STH->execute();echo 'EXECUTE 2';}

[rafuck]

Это точно не поможет.

[rafuck]

У вас же одинаковые записи. И вообще, это условие странное. Тогда уж так надо было: private function query($query, $params = NULL){.... if ($params !== NULL)

Answer 1

[Вячеслав Лебедев]

Ноль помощи :) Круто.
Для супер непонятливых, index.php:

<?php

require_once 'sys/config/config.class.php';
require_once 'sys/core/abstractdatabase.class.php';

$dataBase = new AbstractDataBase(Config::$dbHost, Config::$dbName, Config::$dbUser, Config::$dbPass, Config::$dbPrefix);

$sth = $dataBase->insert('tasks', array('task_title' => 'Четвертая запись!', 'task_desc' => 'Именно четвертая, не пятая и не шестая!'));

echo '<br><pre>';
var_dump($sth);
echo '</pre>';

execute() - вызывается 1 раз, будьте внимательнее, прежде, чем писать о таком...

Answer 2

[Евгений Елчев]

30 не срок, что бы вопросы создавать, откройте для себе xdebug, будет проще такие косяки искать. А вообще не ленитесь использовать фигурные скобки, тогда тоже будете замечать очевидные вещи.

if (!$STH) return false;
       if (isset($params)) $STH->execute($params);
else $STH->execute();

Два раза execute вызываете

Comments

[rafuck]

Нет же. 1) Без $params запрос будет синтаксически неверным;
2)

$str = '';
if(0) $str.=1;
if(1) $str.=2;
else $str.=3;
echo $str;

Печатает "2"..

[Евгений Елчев]

Ну тогда xdebug вам в помощь.

[rafuck]

не мне :)