$this->db->Query($query);, Czy tak jest poprawnie? :| Opcje

[Fluke]

Witam, kolejny problem z którym się spotykam, tzn nie problem ale właściwy sposób.

Mam taki kod:

[PHP] pobierz, plaintext 
class Controller
{
	public function db()
	{
		try
		{
			if(!include_once('config/config.inc.php'))
				throw new Exception('Błąd ładowania pliku.');
 
			switch(DATABASE)
			{
				case 'mysql' :
					if(!include_once('config/db.inc.php'))
						throw new Exception('Nie można załadować pliku konfiguracyjnego do bazy MySQL');
 
 
					return new appMySQL($mysql);
 
				break;
			}
		} catch(Exception $e)
		{
			echo $e->getMessage();
		}
 
	}
}
[PHP] pobierz, plaintext 

Oraz:

[PHP] pobierz, plaintext 
<?php
class mojController extends Controller
{
	function __construct()
	{		
		if($this->db()->Query('SELECT * FROM users'))
			echo 'ok';
	}
}
?>
[PHP] pobierz, plaintext 

Działa więc się cieszę, ale czy to jest dobry sposób? Często widzę w niektórych frameworkach czy CMS`ach że tak się stosuje, ale czy w takich przypadkach jak mój.

Pozdrawiam.

[Atomic_Cookie`]

Za każdym zapytaniem od nową łączysz się z bazą danych, ja łączenie się z bazą danych zrobiłbym w konstruktorze.