Почему запись в БД вставляется более одного раза?

Question

[Артем Липатов]

При записи данных в БД MySQL через PDO запись вместо одного раза вставляется 1-3 раза (чаще всего 2). Анализ кода (кода мало) не дал никакого понимания ситуации.

//index.php
$db = new \system\DB();
$query = "INSERT INTO `crafty` (`value`) VALUES (:value)";
$params = [":value" => 'value3'];
var_dump($db->insert($query, $params));

//DB.php
public function insert($query, array $params = [])
	{
		try {
			$sth = $this->connection->prepare($query);
			$sth->execute($params);
			return $this->connection->lastInsertId();
		} catch (PDOException $e) {
			throw new DBException("Wrong table or parameters");
		}
	}

В чем может скрываться ошибка?

Answer 1

[Melkij]

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

Comments

[Артем Липатов]

Спасибо) Как-то я сразу не догадался)

Answer 2

[FanatPHP]

throw new DBException("Wrong table or parameters");

Это такая шутка остроумная?

Comments

[Артем Липатов]

Иногда жаль, что дизлайки ставить нельзя. Скажите, чем Вам не нравится этот код?
Естественно, в будущем код изменится

[FanatPHP]

Мне тоже жаль, что нельзя ставить дизлайки, за такой вот код. Не нравится он мне своим наличием. Зачем он здесь вообще? Что за дурацкая идея подменять реальное сообщение о реальной ошибке каким-то невнятным бормотанием? Есть ли в этом хоть малейший смысл? В будущем, я так понимаю, код станет еще хуже

[Артем Липатов]

FanatPHP: Вы слышали про уровни абстракции? Этот код пишется для дальнейшего использования в моделях, где неважно знать детали того, почему не прошел запрос. Уменьшение числа классов исключений - вот причина его существования. Стандартным классом Exception можно кое-что лишнее поймать.

[FanatPHP]

Не надо пытаться прикрыть умными словами обыкновенную глупость. Моделям, может быть, и неважно, но программист должен знать, какая ошибка произошла. Если хочется изменить класс - ради бога. Но зачем просто брать и выкидывать жизненно важную информацию - загадка. Ты, видимо, просто из тех, кто не понимает, что такое отладка программы. А все проблемы решаешь способом "вот код он не работает исправьте его мне"

[Артем Липатов]

FanatPHP: глупая ложь. Вы скатились до оскорблений.

[Артем Липатов]

FanatPHP: о коде исключения я говорил: код будет изменяться, когда допилю класс целиком. Ибо это преждевременная оптимизация.

[Артем Липатов]

FanatPHP: глупы здесь только Ваши комментарии, потому что содержат неконструктивную критику и оскорбления.

[FanatPHP]

ОМГ какой ламер нервный пошел. Вместо того чтобы слушать да на ус мотать, только слюнями брызгает. Где здесь глупая ложь, мальчик? Вот я пишу, что ты берешь, и просто выбрасываешь жизненно важную информацию. Так оно и есть: было у тебя исключение в переменной $e - и ты его тупо затираешь, выдавая вместо него абсолютно бессмысленный, никому не нужный текст. Где здесь ложь? Я описываю ТВОЙ СОБСТВЕННЫЙ КОД. То что он глупый - это я тебе с самого начала говорил. А вранья здесь нет. Неча на зеркало пенять. Вот я пишу, что отлаживать ты не умеешь. Ну так это очевидно, раз ты не понимаешь важности сообщения об ошибке и затираешь его. Раз затираешь - значит не понимаешь принципиальной важности. Раз не понимаешь - значит не умеешь отлаживать. Программист будет зубами рвать за сообщение об ошибке. А ламер будет фантазировать об абстракциях.

[Артем Липатов]

FanatPHP: вы умеете читать? Я стопицотый раз Вам говорю: класс не дописан. А преждевременная оптимизация чаще всего нехорошо. Вы же не знаете, на какой стадии находится данный код и для каких целей он пишется, а кидаетесь резкими заявлениями оскорбительного характера. Могли бы хотя бы попробовать перечитать предыдущие сообщения, вместо того, чтобы пытаться самоутвердиться за счет того, что Вам не понравились промежуточные версии чужого кода.

[FanatPHP]

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

[Артем Липатов]

FanatPHP: с Вашей настойчивостью Вам бы в спорт, а не в программирование. Или в грузчики.

[FanatPHP]

и всё же я настоятельно рекомендовал бы убрать блок try..catch из функции совсем. Никакой пользы он не приносит, а вреда - более чем.