Дублирование данных при insert, где ошибка?

Question

[Павел Клочков]

Здравствуйте!
Пытаюсь написать класс для работы с Mysql.
Когда записываю что-то в базу данных они дублируются и получается 2 записи вместо одной.

Класс простой

class DatabaseMysql
{
    
    private $pdo;
    private $dbc;

    public function __construct()
    {
        $this->dbc = require('./core/config/DbConfig.php');
        $this->pdo = new \PDO(
            'mysql:host=' . $this->dbc['db_host'] . ';dbname=' . $this->dbc['db_name'],
            $this->dbc['db_user'],
            $this->dbc['db_pass']
        );
    }
    public function insert(string $query, array $params)
    {

        $sth = $this->pdo->prepare($query);
        $res = $sth->execute($params);

        if (false === $res) {
            return null;
        }

        return $this->pdo->lastInsertId();
    }

}

Пытаюсь вставить данные

$db = new DatabaseMysql;

$res = $db->insert('INSERT INTO test_table (`name`, `age`) VALUES (?,?)', ['Ivan',43]);

В результате в базе 2 одинаковые строки.
Подскажите где я ошибаюсь?

Answer 1

[FanatPHP]

про проблему тебе все уже рассказали, все изменения в БД должны быть только методом POST.

Но поскольку вопрос изначально про класс, качестве шефской помощи перепишем его на более осмысленный, учитывая что

- доступ к инстансу ПДО должен быть не только изнутри
- а вот конфиг держать в переменной класса нужно как рыбке зонтик
- кодировку при соеднинеии с базой надо задавать, чтобы потом не плодить на тостере очередной пост, "у меня в базе вопросики записались"
- причем сразу правильную
- самое главное - вежливо попросить РНР сообщать обо всех ошибках в запросах
- ну и заодно установить разные удобные опции
- пути всегда должны быть абсолютными
- сначала надо написать универсальный метод для выполнения запросов, а потом уже использовать его во всех остальных методах
- ненужный код не нужен

class DatabaseMysql
{
    public $pdo;

    public function __construct()
    {
        $options = [
            \PDO::ATTR_ERRMODE            => \PDO::ERRMODE_EXCEPTION,
            \PDO::ATTR_DEFAULT_FETCH_MODE => \PDO::FETCH_ASSOC,
            \PDO::ATTR_EMULATE_PREPARES   => false,
        ];
        $dbc = require(__DIR__.'/core/config/DbConfig.php');
        $this->pdo = new \PDO(
            "mysql:host={$dbc['db_host']};dbname={$dbc['db_name']};charset=utf8mb4",
            $dbc['db_user'],
            $dbc['db_pass'],
            $options,
        );
    }
    public function query(string $query, array $params = [])
    {
        $sth = $this->pdo->prepare($query);
        $sth->execute($params);
        return $sth;
    }

    public function insert(string $query, array $params)
    {
        $this->query($query, $params);
        return $this->pdo->lastInsertId();
    }
}

Хотя если честно, то я большого смысла в этом классе не вижу.
Меня бы устроило простое расширение класса PDO типа такого

class MyPDO extends PDO
{
    public function run($sql, $bind = NULL)
    {
        $stmt = $this->prepare($sql);
        $stmt->execute($bind);
        return $stmt;
    }
}

Да, вставка будет выполняться за две строчки, а не одну,

$pdo->insert('INSERT INTO test_table (`name`, `age`) VALUES (?,?)', ['Ivan',43]);
$id = $pdo->lastInsertId();

но я не уверен что стоит городить целый класс из-за одной строчки.

Comments

[Павел Клочков]

Благодарю!

Answer 2

[galaxy]

В этом коде на первый взгляд все ок. Ошибка где-то еще, вероятно. Попробуйте логировать пошагово.

Comments

[Сергей delphinpro]

Да тут скорее всего классика. Отправка GET запроса, по которому и происходит добавление. А там уже или браузерные расширения или сам браузер или что то еще повторяют запрос.
Решение очевидное — для изменения данных использовать только POST. GET для получения, что следует из его названия.

[Павел Клочков]

Странно получается, что когда просто в файле вызываю

$db = new DatabaseMysql;

$res = $db->insert('INSERT INTO test_table (`name`, `age`) VALUES (?,?)', ['Ivan',43]);

запись дублируется.
Написал по быстрому форму

if(isset($_POST['name']) and isset($_POST['age'])) {
    var_dump($_POST);
    $db = new DatabaseMysql();
    $res = $db->insert('INSERT INTO test_table (`id`, `name`, `age`) VALUES (?,?,?)', [null, $_POST['name'], $_POST['age']]);
    var_dump($res);
    header('location:/');
}

?>
<form action="" method="post">
    <input type="text" name="name">
    <input type="number" name="age">
    <input type="submit">
</form>

Всё вставляется нормально. Такое чуство что браузер 2 раза перезагружается или xj-nj с настройками apache

[galaxy]

Т.е. Сергей delphinpro был, видимо, прав. Вы посмотрите через dev tools в браузере, какие запросы шлются (только поставьте галочку preserve log между запросами). Если вы в index.php прямо insert делаете, возможно от браузера прилетает также запрос на favicon, который тоже туда перенаправляется

[FanatPHP]

Павел Клочков, это дома на уютненьком ноутбучике у тебя перегружается 2 раза. На реальном сервере будет перегружаться пицот раз. Поэтому тебе и говорят что обновление информации должно быть только методом пост.

[FanatPHP]

Сергей delphinpro, вообще-то это должно быть ответом, а не комментарием, чтобы автор вопроса потом не помечал решением супер-полезную реплику "низнаю, наверное проблема где-то ищо"