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?

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?

Stanowczo w złym kierunku idziesz. Zajmij się logiką aplikacji, a nie tym, co ma ją obsługiwać - to zostaw profesjonalistom, którzy pracują nad tym latami i wiedzą w jaki sposób ma to wszystko działać.

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.

to tylko przykład był :/

Crozin, możesz jaśniej o pkt 3?

http://pl2.php.net/__autoload
http://pl2.php.net/manual/en/function.spl-...ad-register.php

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.




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.

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 (sory wookieb)

Lepiej zajmij się nauką jakiegoś frameworka, np. Symfony 2. Bez doświadczenia nie napiszesz dobrego systemu, a to co robisz to zwyczajna strata czasu.