$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.

[Crozin]

a nie chcę źle się uczyć.

No to:

1. Nazwa Controller sugeruje jakiś związek z architekturą MVC. Jeśli tak jest to: kontroler nie ma nic wspólnego z bazą danych i nie powinien mieć nawet do niej dostępu.
2. Klasa Exception powinna być w PHP abstrakcyjna, bo... jej obiektami nie powinno się rzucać (powinieneś skorzystać z bardziej wyspecjalizowanego typu).
3. Wyrzucenie wyjątku tylko po to by za chwilę go złapać i kompletnie nic z nim nie zrobić, poza durnym wyświetleniem wiadomości mija się z celem - już lepiej użyć exit w takim przypadku.
4. Do obsługi bazy danych masz w miarę przyzwoite rozszerzenie PDO.
5. Korzystaj z mechanizmów automatycznego ładowania klas (ang. autoloading) bo się pochlastasz z ręcznym wczytywaniem.
6. Zdajesz sobie sprawę, że Controller:(IMG:style_emoticons/default/biggrin.gif) b() każdorazowo tworzy nowy obiekt co jest w tym przypadku raczej niekorzystne.

EDIT: I daruj sobie póki co lekturę wzorca Singleton, bo użycie jakie sugeruje @Wicepsik jest tutaj złe pod każdym względem, a de facto nie potrzebujesz tutaj nigdzie prawdziwych cech tego wzorca.

Ten post edytował Crozin 11.01.2011, 21:30:31