MySQLi gebruiken van een andere klasse in PHP

Er zijn verschillende slechte praktijken die u tot deze fout hebben geleid.

Het is duidelijk dat het uitbreiden van de gebruiker vanuit een database een verkeerde zet is. Ook is de hele Database-klasse nogal nutteloos omdat het niets nuttigs doet.

Daarom zou ik willen voorstellen om

• verwijder de nutteloze Database-klasse.
• maak een enkele $db instantie van vanilla mysqli.
• geef het als een constructorparameter door aan elke klasse die een databaseverbinding nodig heeft

database.php:

<?php
mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
$db = new mysqli("localhost", "DBUserName", "UserPassword", "SelectedDB");
$db->set_charset('utf8mb4');

myapi.php

<?php
class MyAPI
{
    protected $db;

    public function __construct($db, $request_uri, $postData, $origin)
    {
        $this->db = $db;
    }

    public function getUser($id)
    {
        $sql = "SELECT * FROM users where id=?";
        $stmt = $this->db->prepate($sql);
        $stmt->bind_param("s", $id);
        $stmt->execute();
        $result = $stmt->get_result();
        return $result->fetch_assoc();
    }
}

app.php

<?php
# require_once 'Database.php';
# require_once 'myapi.php';
require 'vendor/autoload.php'; // autoloading is a must

$api = new MyAPI($db, $request_uri, $postData, $origin);
$user = $api->getUser($_POST['id']);