Witam zaczynam bawic sie z obiektowka, napisalem takie cos:

[PHP] pobierz, plaintext 
<?php
 
class mysql {
 
	function connect(){
 
		define("HOST", "localhost");
		define("USERNAME", "user");
		define("PASSWORD", "password");
		define("DB", "database");
 
		$conn = new PDO(HOST:DB, USERNAME, PASSWORD);
	}
 
}
 
?>
[PHP] pobierz, plaintext 

Czy jest to dobrze sformulowane?? Prosze o porady itd. Z gory dziekuje

Masz zamiar definiować stałe za każdym razem gdy będziesz nawiązywał połączenie z bazą danych? Poza tym by się przydało żeby funkcja zwracała uchwyt do bazy.

[PHP] pobierz, plaintext 
return $conn;
[PHP] pobierz, plaintext 

A właściwie to takie masło maślane, opakowanie obiektu klasy w inny obiekt klasy. Tak wywołujesz PDO:

[PHP] pobierz, plaintext 
$conn = new PDO(HOST:DB, USERNAME, PASSWORD);
[PHP] pobierz, plaintext 

a tak swoją klasę:

[PHP] pobierz, plaintext 
$mysql = new mysql;
$conn = $mysql->connect();
[PHP] pobierz, plaintext 

co i tak nie zadziała, bo nie zwraca uchwytu.

Jeśli już to tak:

[PHP] pobierz, plaintext 
<?php
define("HOST", "localhost");
define("USERNAME", "user");
define("PASSWORD", "password");
define("DB", "database");
 
class mysql {
 
        public $host;
        public $user;
        public $pass;
        public $db;
 
        function __construct() {
          $this->host = HOST;
          $this->user = USERNAME;
          $this->pass = PASSWORD;
          $this->db = DB;
        }
 
	function connect(){
 
		$conn = new PDO($this->host:$this->db, $this->user,$this->pass);
                return $conn;
	}
 
}
 
?>
[PHP] pobierz, plaintext 

czyli jezeli dam

[PHP] pobierz, plaintext 
$conn = new PDO($host:$db, $username, $password);
[PHP] pobierz, plaintext 

to na dol musze jesczze zwrocic uchwyt jako :

[PHP] pobierz, plaintext 
$conn = new PDO($host:$db, $username, $password);
return $conn;
[PHP] pobierz, plaintext 

tak?

i wtedy jak chce wydac jakies polecenia typu

[PHP] pobierz, plaintext 
$conn->exec("SQL")
[PHP] pobierz, plaintext 

to juz bedzie wszystko dzialalo?

[PHP] pobierz, plaintext 
$conn->exec("SQL")
[PHP] pobierz, plaintext 

tak będzie działało, jeśli masz na myśli, że po takim wywołaniu:

[PHP] pobierz, plaintext 
$mysql = new mysql;
$conn = $mysql->connect();
[PHP] pobierz, plaintext 

ale zaznaczam, że definiowanie stałych przy każdym połączeniu mija się z celem zupełnie.

a mozecie mi pokazac jaki nawiazywanie takiego polaczenia wyglada profesjonalnie? czyli taki wzorzec ktorym moglbym sie kierowac

Ja tam nie wiem jak wygląda profesjonalnie, ale przykładowo:

config.php:

[PHP] pobierz, plaintext 
define('DB_TYPE', 'mysql');
define('DB_HOST', 'localhost');
[PHP] pobierz, plaintext 

itd.

Database.php

[PHP] pobierz, plaintext 
class Database extends PDO {
public function __construct($DB_TYPE, $DB_HOST, $DB_NAME, $DB_USER, $DB_PASS) {
        parent::__construct($DB_TYPE . ':host=' . $DB_HOST . ';dbname=' . $DB_NAME, $DB_USER, $DB_PASS);
    }
public function selectAll($sql, $array = array(), $fetchMode = PDO::FETCH_ASSOC) {
 
        $sth = $this->prepare($sql);
 
        foreach ($array as $key => $value) {
            $sth->bindValue("$key", $value);
        }
 
        $sth->execute();
        return $sth->fetchAll($fetchMode);
    }
}
[PHP] pobierz, plaintext 

Model.php

[PHP] pobierz, plaintext 
class Model {
 
	public function __construct() {
		$this->db = new Database(DB_TYPE, DB_HOST, DB_NAME, DB_USER, DB_PASS);
	}
 
}
[PHP] pobierz, plaintext 

i w jakimś tam przykładowym pliku gdzie będę korzystał z bazy danych:

[PHP] pobierz, plaintext 
class User_Model extends Model {
public function __construct() {
        parent::__construct();
    }
public function userSelectAll() {
        return $this->db->selectAll("SELECT id, login, role FROM user");
    }
}
[PHP] pobierz, plaintext 

Przy okazji, ktoś kto wie jak to powinno być profesjonalnie, może zweryfikować, czy to jest ok, sam się może czegoś nauczę nowego