[PHP] OOP - poprawnie? Opcje

[emajl22]

piszę sobie coś obiektowo i mój index wygląda mniej więcej tak:

[PHP] pobierz, plaintext 
// gora
	<?php
		require_once('./silnik.php');
		$api = new API;
 
		$api->doContent($cos);
	?>
	//dol
[PHP] pobierz, plaintext 

i teraz pytanie - dobrze to jest napisane? chodzi mi o samą logikę aplikacji...

klasa silnik.php wygląda tak:

[PHP] pobierz, plaintext 
class API {
 
			public function __construct() {
				// laczenie z baza
				// doklejanie innych klas i ich inicjowanie
			}
 
			public function doContent($param) {
				switch($this->protect($param))) {
					case 'rejestracja':
						include('./class/rejestracja.class.php');
						break;
					case 'kontakt':
						include('./views/kontakt.php');
						break;
					default:
						if (isset($_SESSION['cos']))
							// zalogowany
						else
							include('./views/default.php');
						break;
				}
			}
 
			public function protect($co) {
 
				// zabezpieczanie danych z formularzy, getow itd
				return $co;
 
			}	
		}
[PHP] pobierz, plaintext 

w dobrym ide kierunku?

[Crozin]

W złym.

1. Nie trzymasz się konwencji nazewnictwa. Co to za dziwny pomysł by w pliku silnik.php (już mieszanie polskiego i angielskiego pomińmy) znajdowała się klasa API? - jej nazwa swoją drogą też powala.
2. http://en.wikipedia.org/wiki/Single_responsibility_principle
3. Korzystaj z mechanizmów automatycznego ładowania.