PHP OOP Pomoc

Question

Witam! Co powinienem poprawić w tym kodzie, aby wyglądał dobrze? Czy są jakieś lepsze sposoby na klasę Database? Pozdrawiam!

class Database {

    private static $_instance = null;
    private $_pdo;

    function __construct() {
        try {
            $this->_pdo = new PDO();
            $this->_pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            $this->_pdo->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_ASSOC);
        } catch (PDOException $e) {
            echo $e->getMessage();
        }
    }

    public function getInstaceOfConnection() {
        if(!isset(self::$_instance)) {
            self::$_instance = new Database();
        }

        return self::$_instance;
    }
}

class Product {

    private $_query;

    public function getDetailsOfProductById($id) {

        $stmt = Database::getInstaceOfConnection()->prepare("SELECT * FROM products WHERE id = {$id}");

        $stmt->execute();

        return true;

    }
}

 

Answer 1

Tak, są lepsze sposoby. Polecam poczytać o Dependency Injection i wstrzykiwać klasę połączenia do innych miejsc w kodzie.