Centralizacja konfiguracji a klasy Opcje

[Lewiatan]

Witam

Chciałbym się zorientować jakie są możliwości jeśli chodzi o centralizację konfiguracji i wykorzystywanie jej zmiennych bezpośrednio w klasach. Otóż stworzyłem sobie pliczek, załóżmy config.php i zrobiłem tam tablicę, np:

[PHP] pobierz, plaintext 
$config['db']['host'] = 'localhost';
$config['db']['user'] = 'user';
$config['db']['pswd'] = 'hasieł';
$config['db']['baza'] = 'baza';
[PHP] pobierz, plaintext 

dalej mam klasę dostępową do bazy danych, taki tam banał, ale problem jest z przekazaniem zmiennych bezpośrednio do klasy np. w taki sposób:

[PHP] pobierz, plaintext 
 
require_once('./config.php');
 
class baza
	{
		private $host; //host
		private $user; // użytkownik
		private $pswd; // hasło
		private $baza; //baza danych
		private $db;
		private $affected_rows = 0;
 
		public function __construct()
			{
				$this->host = $config['db']['host'];
                                $this->user = $config['db']['user'];
                                $this->pswd = $config['db']['pswd'];
                                $this->baza = $config['db']['baza'];
                         }
}                
[PHP] pobierz, plaintext 

Do tej pory zmienne konfiguracyjne ustawiałem w każdej klasie z osobna.

Pomyślałem, że można zrobić klasę, tam zdefiniować tablice z danymi konfiguracyjnymi i utworzyć metody dostępowe żeby zwracały tablicę, coby dostęp wyglądał np. tak:

[PHP] pobierz, plaintext 
class baza
	{
		private config = array();
		private $db;
		private $affected_rows = 0;
 
		public function __construct()
			{
				$conf = new Config();
                                $this->config = $conf->getDbConfig();
                         }
}                
[PHP] pobierz, plaintext 

Pytanie jest następujące: jakie są dobre możliwości rozwiazania tego problemu? Bo czytałem, że define raczej powinno się unikać.

[Crozin]

Podstawy OOP - obiekt nie powinien się w jakiś automagiczny sposób samoistnie konfigurować. Konfiguracja musi być przekazana mu z zewnątrz, czyli:

[PHP] pobierz, plaintext 
public function __construct($host, $user, $password, $database) {
  ...
}
[PHP] pobierz, plaintext 

Jeżeli zmiennych konfiguracyjnych jest za dużo (konstruktor z 10-oma argumentami to zła rzecz) możesz utworzyć jakiś obiekt reprezentujący konfigurację dla danego obiektu, czyli:

[PHP] pobierz, plaintext 
public function __construct(DatabaseConfiguration $config) {
  ...
}
[PHP] pobierz, plaintext 

[Lewiatan]

Podstawy OOP - obiekt nie powinien się w jakiś automagiczny sposób samoistnie konfigurować. Konfiguracja musi być przekazana mu z zewnątrz, czyli:

[PHP] pobierz, plaintext 
public function __construct($host, $user, $password, $database) {
  ...
}
[PHP] pobierz, plaintext 

Jeżeli zmiennych konfiguracyjnych jest za dużo (konstruktor z 10-oma argumentami to zła rzecz) możesz utworzyć jakiś obiekt reprezentujący konfigurację dla danego obiektu, czyli:

[PHP] pobierz, plaintext 
public function __construct(DatabaseConfiguration $config) {
  ...
}
[PHP] pobierz, plaintext 

No rozumiem. Tak myślałem, że jest możliwość przekazania tego przez konstruktor (w zasadzie to pierwsze przyszło mi do głowy) ale znowu średnio mi się jakoś widzi za każdym wywołaniem przekazywać zmienne bądź obiekt w konstruktorze. Liczyłem na to że jest jakiś bardziej "elegancki" sposób

[Zyx]

To jest właśnie elegancki sposób. Zacznij pisać testy automatyczne lub używać kodu wielokrotnie, to się o tym sam przekonasz .

[Lewiatan]

OK, a ten sposób który napisałem na początku, żeby obiekt był tworzony przez konstruktor a nie przesyłany do konstruktora też się nada?

[Crozin]

Wtedy równie dobrze możesz na stałe wpisać sobie konfigurację w kod klasy, bo przecież nie będzie możliwości żadnego sensownego zmieniania tych ustawień.

[...] ale znowu średnio mi się jakoś widzi za każdym wywołaniem przekazywać zmienne bądź obiekt w konstruktorze [...]

A ile to masz obiektów odpowiedzialnych za połączenie z bazą danych na stronie? Zapewne jeden, może dwa.