Wróciłem po jakiś dwóch miesiącach do pisania w PHP i natrafiłem na jakąś swoją ociężałość umysłową która nie pozwala mi uporać się z problemem

[PHP] pobierz, plaintext 
<?php
 
class MySQL
{
	private $hostname;
	private $username;
	private $password;
	private $database;
 
 
	// METODY
 
	protected function __construct() {}
 
	// Singleton
	//
	// Wymagana argument to nazwa pliku konfiguracyjnego
 
	public function singleton() // ($filename)
	{		
		static $instance;
 
		if(!isset($instance))
		{
			$instance = new MySQL;
 
			$config_data = parse_ini_file(func_get_arg(0))
				or die('plik');
 
			$this->parse_config($config_data);
		}		
 
		return $instance;
	}
 
	private function parse_config($array)
	{
		$this->hostname = $array[MySQL][hostname];
		$this->username = $array[MySQL][username];
		$this->password = $array[MySQL][password];
		$this->database = $array[MySQL][database];
	}	
 
}
 
?>
[PHP] pobierz, plaintext 

Podany błąd:



Odnosi się do tego fragmentu



Korzystam z KRASNAL-Serv-2.7, w składzie:
- Apache 1.3.31
- PHP 5.0.2

Oczywiście WinXP

Możliwe że problem leży w ustawieniach ale trochę czasu już spędziłem na kombinowaniu dlatego z góry dzięki za pomoc

singleton to chyba jest funkcja statyczna, a zamiast

[PHP] pobierz, plaintext 
<?php
$this->parse_config($config_data);
?>
[PHP] pobierz, plaintext 

chciałeś oczywiście dać

[PHP] pobierz, plaintext 
<?php
$instance->parse_config($config_data);
?>
[PHP] pobierz, plaintext 

?

Zgadza się, właśnie tu leżała moja ociężałość - dzięki wielkie, zabieram sie w końcu dalej do pisania ;]

Kolego jak zabierasz sie do pisania klas to rob to porzadnie. Wzorzec singleton ma wygladac jak singleton, a nie surówka kodu. No i zawsze pisz tak, zeby kod był skalowalny. Następnym razem, gdy bedziesz pisal jakis projekt, wystarczyc bedzie, ze skopiujesz sobie plik, a on dalej bedzie śmigał:

[PHP] pobierz, plaintext 
<?php
class mySQL {
 
	private $host, $user, $pass, $db;
	private $handle, $dbhandle;
	private static $instance;
 
	/**
	* Prywatny konstruktor
	*/	
	private function __construct() {}
 
	/**
	* Destruktor, np zamykanie polaczenia
	*/
	public function __destruct()
	{
		@mysql_close($this->handle);
	}
 
	/**
	* Jak przystało na Singleton
	* metoda getInstance()
	*/
	public static function getInstance()
	{
		if (empty(self::$instance))
			self::$instance = new mySQL();
 
		return self::$instance;
	}
 
	/**
	* łączenie z bazą danych
	*
	* @throws Exception
	*/
	public function connect()
	{
		$this->handle = @mysql_connect($this->host, $this->user, $this->pass);
		$this->dbhandle = @mysql_select_db($this->db);
 
		if (mysql_errno())
		{
			throw new Exception('Blad mysql (' . mysql_error() . ')');
		}
	}
 
	/**
	* Parsowanie pliku konfiguracyjnego.
	*
	* @throws Exception
	*/
	public function parse_config($file)
	{
		   $array = @parse_ini_file($file);
		   
		if (!count($array))
		{
			throw new Exception('Blad czytania pliku konfiguracyjnego.');
			return;
		}
		   
		$this->host = $array[MySQL][hostname];
		$this->user = $array[MySQL][username];
		$this->pass = $array[MySQL][password];
		$this->db   = $array[MySQL][database];
	}  
}
?>
[PHP] pobierz, plaintext 

dzieki takiemu kodowi, w kazdej aplikacji masz bezposrednią kontrolę nad działaniem klasy, obsługą błędów, itp:

[PHP] pobierz, plaintext 
<?php
$sql = mySQL::getInstance();
 
try {
   $sql->parse_config('config.ini');
}
catch (Exception $e) {
   // obsługa błędu w razie
   // błędnej konfiguracji
}
 
try {
   $sql->connect();
}
catch (Exception $e) {
   // obsługa błędu
   // w razie błędu połączenia
}
 
// dalsze działanie skryptu
// mozesz dalej rozszerzyc tą klasę o metody wywoływania zapytań
// sql, filtrowanie danych.. mozliwosci są nieskończone, a zabawa wspaniała
?>
[PHP] pobierz, plaintext 

powodzenia