[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?

[KKH]

Jak dla mnie trochę popsuta hermetyzacja.

Metody doContent() i protect() klasy API odpowiadają dość różnym zadaniom. Nie powinno się projektować klasy, która robi zbyt wiele rzeczy na raz. Generalnie 1 rzecz = 1 klasa. Zbyt obszerna klasa przestaje mieć coś wspólnego z obiektówką, a staje się bardziej "modułem", tzn. czymś w rodzaju "przestrzeni nazw" dla strukturalnego zbioru funkcji. Czemu to złe? Chcesz np. coś zmienić w protect(). Być może protect() działa w prozumieniu z jakimś polem prywatnym, powiedzmy $prot. Ale czy $prot nie jest używana gdzieś w doContent()? Tego już nie jesteś pewien. Aby zmodyfikować jedną funkcjonalność muisz analizować kod, który nie ma z nią związku. Nie temu służy idea programzowania obiektowego.

Te Twoje API robi za kontroler, walidator (unieszkodlwiajacy niebezpieczne stringi - ale na co: HTML, SQL?) i za model, np. zarządza użytkownikami tak:

[PHP] pobierz, plaintext 
if (isset($_SESSION['cos']))
    // zalogowany
[PHP] pobierz, plaintext 

A przecież gdy myślimy obiektowo, to aż się prosi by użytkownikowi odpowiadała osobna klasa.

W złym. Od tego jest framework - czyli Symfony czy też Zend (wiem, wiem są jeszcze inne równie dobrze, ale nie o to tutaj chodzi).
Po co chcesz wynajdować koło od nowa?

Kolega pisał: "dobrze to jest napisane? chodzi mi o samą logikę aplikacji...". Przecież coś nie może być napisane źle i nielogicznie tylko dlatego, że nie używa framewroka. Co innego, gdy rozważymy cel tego pisania. Mały, duży system. Nauka, praca, hobby? Dojdzie ktoś do zespołu itd.