[PHP] Singleton jako klasa ustawien, jak przy tworzeniu prawidlowo ustawic dane Opcje

[karmelik]

Mam wzór klasy z uzyciem wzorca Singleton:

[PHP] pobierz, plaintext 
class Config{
 
	private static $instance;
 
	private $config = array();
 
	private function __construct() {}
	private function __clone() {}
 
	public static function getInstance() {
		if(self::$instance === null) {
			self::$instance = new Config();
		}
		return self::$instance;
	}
 
}
[PHP] pobierz, plaintext 

Teraz chcialbym zeby przechowywala ona ustawienia aplikacji ktore pobieram z bazy danych (czyli inna klasa to pobiera). Ze wzgledu na poczatki z wzorcami projektowymi, pytanie jak przerobic ta klase zeby mozna bylo ja wywolac razem z danymi wejsciowymi. W konstruktorze, w getInstance ? czy moze utworzyc metode ktora bedzie zapisywala dane wejsciowe juz po utworzeniu instancji. Mysle ze wszystkimi sposobami daloby rade, ale ktory jest prawidlowy ? Wolalbym nie robic profanacji juz od poczatku

Edit: chodzi mi tylko o podpowiedz "któredy" powinno sie wgrywac dane do klasy. Sam sobie napisze wszystko tylko któredy zrobic to zeby bylo prawidlowo.

Ten post edytował karmelik 10.03.2015, 08:56:10

[Pyton_000]

__construct

[karmelik]

Sporo mi z tym zeszło ale w koncu doszedlem, moze sie komus przyda:

[PHP] pobierz, plaintext 
class Config{
 
	private static $instance;
 
	private $config = array();
 
	private function __construct(array $config) {
			$this->config = $config;
	}
	private function __clone() {}
 
	public static function getInstance(array $config) {
		if(self::$instance === null) {
			self::$instance = new Config($config);
		}
		return self::$instance;
	}
 
}
[PHP] pobierz, plaintext 

[Pyton_000]

Nie do końca.

Jeżeli to jest klasa stricte do configa to w samym konstruktorze powineneś zadbać o pobieranie danych.

Co jak będziesz w 1000 miejscach używam Config::getinstance();
To co za każdym razem masz tam przekazywać konfiguracje? Bez sensu.
Możesz co najwyżej przekazać obiekt który zajmie się importem konfiguracji do Singletonu, albo nawet nic nie przekazywać. Niech Singletn sam sobie pobierze dane jak chce.

[karmelik]

Właśnie nie wiem jak to zrobić. Musiałbym zrobić tak:

[PHP] pobierz, plaintext 
public static function getInstance() {
		if(self::$instance === null) {
			$db = new Database();
			$config = $db->pobierz_konfiguracje();
			self::$instance = new Config($config);			
		}
		return self::$instance;
	}
[PHP] pobierz, plaintext 

czyli drugi raz wykorzystac klase do polaczenia i obslugi bazy danych, drugi raz ja zainicjowac. Ewentualnie stworzyc nowa klase tylko do pobierania ustawien ktora rozszerza klase do polaczenia bazy danych. Ewentualnie zrobic klase Database jako singleton i z niej korzystac ? Albo moze ja klonowac ? Nie wiem kurcze jak do tego podejsc zeby klasy same sobie radzily i nie inicjowac pare razy tych samych klas w innych klasach :/ dopiero poczatkuje w klasach i wzorcach projektowych :/ Podrzuc prosze jakies pomysly.

[com]

To już Ci napisał Pyton_000
Więc jeszcze raz zacytuje

w samym konstruktorze powineneś zadbać o pobieranie danych

A nie w getInstance()