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

[emajl22]

KKH, wreszcie coś do mnie trafiło jeśli chodzi o oop, za co wielkie dzięki.

Tylko teraz pytanie; przyczepiłeś się do:

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

nie rozumiem jak można to inaczej zapisać... jedynie co mi przychodzi na myśl to:

[PHP] pobierz, plaintext 
$user->showContent()
[PHP] pobierz, plaintext 

i w tym pliku dać sprawdzanie sesji i ew. opcje (jeśli zalogowany)

teraz z innej półki - czepiacie się html'a w kodzie i jak najbardziej wiem, że to racja, ale jak inaczej to zapisać? stworzyć plik np. error.php i:

[PHP] pobierz, plaintext 
$error['db_connect'] = '<div..>Brak połączenia z bazą danych.</div>';
$error['db_select'] = '<div..>Nie mogę wybrać bazy danych....</div>';
[PHP] pobierz, plaintext 

tak? i jeszcze jedno - jak zaimplementować nową klasę w klasie tylko raz?

[PHP] pobierz, plaintext 
$klasa = new KLASA;
[PHP] pobierz, plaintext 

w którym miejscu to dodać? jak dam nad klasą to jej nie wykrywa, jak dam w konstruktorze także... oczywiście includuje ją wcześniej. Teraz muszę w każdej metodzie dodawać to $klasa = new KLASA; a przecież to jest bezsens..

#edit;
__autoload mi pomoże (IMG:style_emoticons/default/biggrin.gif) (IMG:style_emoticons/default/biggrin.gif) (sory wookieb)

Ten post edytował emajl22 13.02.2011, 20:09:41