Модифицированный PDO и lastInsertId()

Это действительно интересный случай.

Проблема lastInsertId() является прямым следствием неправильного дизайна (и частично скопированного кода, созданного с помощью культа карго).

Эта «новая форма PDO» просто отображает распространенные ошибки (настолько распространенные, что я даже написал статью об этом). И одна из таких проблем:

Вы должны понимать, что каждый экземпляр PDO создает отдельное соединение с сервером БД. Таким образом, вы никогда не должны открывать и закрывать новое соединение в каждой функции. Потому что это значительно замедлит ваш PHP и не позволит вам использовать некоторые функции БД, которые можно использовать только в рамках одного и того же соединения, например, транзакции или получение идентификатора вставки.

Итак, теперь вы можете сказать, что проблема возникает из-за того, что в методе lastInsertId() создается новое соединение. И решение состоит в том, чтобы всегда поддерживать одно и то же соединение.

Есть два способа решить эту проблему: более простой на данный момент, который усложнит задачу в будущем, или тот, который немного сложнее реализовать, но сделает ваш код менее связанным и более простым в обслуживании. Оба они объяснялись в моей другой статье о создании такой «гладкой и лаконичной системы PDO на основе классов» (но на основе большого опыта и вопросов, увиденных здесь, в Stack Overflow).

Самым правильным решением было бы получить этот статический материал, поскольку он трудно поддерживать код. Таким образом, лучшим решением было бы создать обычный класс, а затем создать один экземпляр, который должен передаваться по всему вашему коду. В этом случае все вызовы $this->dbh будут иметь смысл и указывать на один и тот же экземпляр PDO. В этом случае вам придется изменить обозначение DB:query() на $db->query().

Однако код в моей статье использует другой подход к расширению PDO. Ваш подход лучше, так что давайте перепишем ваш код в обычный класс

class DB{

    public function __construct()
    {
        $host = '127.0.0.1';
        $db   = 'poetionpics';
        $user = 'root';
        $pass = '';
        $charset = 'utf8mb4';

        $options = [
            \PDO::ATTR_ERRMODE            => \PDO::ERRMODE_EXCEPTION,
            \PDO::ATTR_DEFAULT_FETCH_MODE => \PDO::FETCH_ASSOC,
            \PDO::ATTR_EMULATE_PREPARES   => false,
        ];
        $dsn = "mysql:host=$host;dbname=$db;charset=$charset";
        try {
             $pdo = new \PDO($dsn, $user, $pass, $options);
        } catch (\PDOException $e) {
             throw new \PDOException($e->getMessage(), (int)$e->getCode());
        }
    }
    public function query($query, $params = array())
    {
        $stmt = $this->pdo->prepare($query);
        $stmt->execute($params);
        return $stmt;
    }
    public function lastInsertId()
    {
        return $this->pdo->lastInsertId();
    }
}

Но помните, поскольку вы не расширяете PDO, вы должны реплицировать все методы PDO в своем классе.

Если на данный момент идея отказаться от такого гладкого подхода для вас невыносима, вы можете использовать статический метод, но такой, который сохраняет для вас один экземпляр PDO. Опять же, есть два способа, которые объясняются в статье, указанной выше.

После получения надлежащего класса БД мы можем переписать ваши запросы

$db = new DB();

$sql = 'INSERT INTO cloudbook_posts VALUES (null, :post_title,  :post_desc)';
$db->query($sql, ['post_title'=>$post_title, 'post_desc'=>$post_desc]);

$postid = $db->lastInsertId();

$sql = 'INSERT INTO vid_info VALUES (null, :newvidurl, :postid)';
$db->query($sql, ['newvidurl'=>$newvidurl, 'postid'=>$postid]);