database.php

<?php

// класс БД создаёт приватные переменные с данными для входа в бд "users_db"
// в конструктор попадают переменные с данными для входа и инициализируются
// также проверяется ошибка подключения 
// метод getConnection() возвращает переменную с объектом подключения, чтобы далее использовать её в классе пользователя

class Database
{
    private $host = "localhost";
    private $user = "root";
    private $password = "";
    private $database = "users_db";
    protected $connection;

    public function __construct()
    {
        $this->connection = new mysqli($this->host, $this->user, $this->password, $this->database);

        if ($this->connection->connect_error) {
            die("Connection failed: " . $this->connection->connect_error);
        }
    }

    public function getConnection()
    {
        return $this->connection;
    }
}

class User extends Database
{

    // наследует класс БД, чтобы вытащить переменную с объектом подключения сюда
    // private??? ну а есть ли смысл public тут

    private $db;
    public function __construct($database)
    {
        $this->db = $database->getConnection();
    }

    // то как добавляются в БД пользователи
    // создается выражение $stmt (в процессе защищается от инъекций с помощью prepare() и bind_param())
    // и проверяется на ошибку создания через execute()

    public function createUsers($name, $email)
    {
        $stmt = $this->db->prepare("INSERT INTO Users (name, email) VALUES (?, ?)");
        $stmt->bind_param("ss", $name, $email);

        if ($stmt->execute()) {
            return true;
        } else {
            return false;
        }
    }

    // выборка всех пользователей вообще
    // думаю не надо никаких prepare(), т.к. сам запрос не получает никаких параметров

    public function readUsers()
    {
        $result = $this->db->query("SELECT * FROM users");

        // вариант как пробежаться по массиву с созданными пользователями
        // вернёт ассоциативный массив с массивами {пользователями}, которые потом будем вставлять в index.php

        if ($result->num_rows > 0) {
            return $result->fetch_all(MYSQLI_ASSOC); // Возврат массива ассоциативных массивов
        } else {
            return []; // Возвращаем пустой массив, если нет пользователей, ошибок тут нет (хотя наверное надо бы проверку сделать какую-то)
        }
    }
    public function updateUsers($id, $name, $email)
    {
        // подготовка запроса, ничего нового, делается как раньше
        $stmt = $this->db->prepare("UPDATE users SET name = ?, email = ? WHERE id = ?");

        // если вдруг что-то пойдёт не так, пусть всё оборвёт
        if ($stmt === false) {
            die("Preparing statement error: " . $this->db->error);
        }

        // а если всё ок, продолжаем формировать запрос
        $stmt->bind_param("ssi", $name, $email, $id);

        // выполнение подготовленного запроса
        if ($stmt->execute()) {
            return true;
        } else {
            echo "Executing query error: " . $stmt->error;
            return false;
        }

        // хорошо бы закрыть, чтобы сохранить ресурсы, но вс код ругается
        // пока что закомментирую
        // $stmt->close();
    }

    // по идее не должно крашнуть всю таблицу, есть слово FROM
    public function deleteUsers($id)
    {
        $stmt = $this->db->prepare("DELETE FROM users WHERE id = ?");
        $stmt->bind_param("i", $id);

        if ($stmt->execute()) {
            return true;
        } else {
            return false;
        }
    }
}