Accessing database class

[ejsexton82]

You could turn your database object into a "singleton":

Code: Select all

class DatabaseObject {
	protected $id;
	
	protected static $instance;
	
	protected function __construct() {
		$this->id = uniqid();
	}
	
	public function getId() {
		return $this->id;
	}
	
	public static function &getInstance() {
		if(!isset(self::$instance)) {
			$class = get_class();
			self::$instance = new $class();
		}
		return self::$instance;
	}
}

Then to access your database object, all you would have to do is call getInstance():

Code: Select all

class One {
	public function __construct() {
		$db = DatabaseObject::getInstance();
		echo "<p>Database Instance #{$db->getId()}</p>";
		$db->one = "Accessed by Class One";
	}
}

class Two {
	public function __construct() {
		$db = DatabaseObject::getInstance();
		echo "<p>Database Instance #{$db->getId()}</p>";
		echo "<p>{$db->one}</p>";
	}
}

$one = new One();
$two = new Two();

[NigelRen]

'A better way' is almost always a matter of opinion. If you use a singleton then you remove a lot of the passing the database connection around, but you also limit the flexibility of your code. What happens if you want to use another connection for some reason?
If you consider something like mysql_query - in it's simplest form it uses the 'currently active database' which is fine till someone else changes the sequence of the code and suddenly code that has worked for ages falls over.
If you always specify the database/connection you want to use then you know what it should be accessing and you can see how it's being used.