[oop] Rozbijanie klas Opcje

[sajegib]

Witam!

Mam taki pytanko do szanownego grona.

Załóżmy, że mam malutki czat oparty na OOP, do którego napisałem następujące klasy

[PHP] pobierz, plaintext 
class database
{
//operacje na bazie, select, insert, update, same zapytania i zwracanie wyniku
// dane docierające tutaj są  już wyfiltrowane pod kątem SQLI itd.
}
 
class chat
{
   public function get_messages()
   {
 
        //pobieram wiadomosci z bazy, zwracam tablice z wynikiem z bazy
   }
 
   public function add_message()
   {
       // $this->validate();
       //dodaje wiadomosci po uprzednim wyescapowaniu
    }
 
   public function validate()
   {
       //filtruje dane pod kątem sqli itd. 
   }
 
}
[PHP] pobierz, plaintext 

to taki ogólny szkic, nie patrzcie na błędy, moje pytanie dotyczy raczej architektury całości (IMG:style_emoticons/default/smile.gif)

Teraz moje pytanie:
1. czy rzeczą pożądaną jest, by metoda validate była umieszczona w klasie chat, czy też powinienem stworzyć osobną klasę z metodami filtrującymi dane?
A może nie ma to żadnego znaczenia?
Gdyby było to coś większego, z pewnością zrobiłbym całą klasę walidującą (zawierałaby wtedy wiele metod)

2. Czy ma sens robić klasę zawierającą tylko jedną metodę?
Jak 'powinno się robić' i dlaczego?

Ten post edytował sajegib 23.04.2013, 23:08:17

[skowron-line]

Single responsibility principle
Ja bym w tą stronę szedł.

[PHP] pobierz, plaintext 
class database
{
	public function __construct()
}
 
class message
{
	public function write($message)
	public function read()
}
 
class validate
{
	public function not_empty()
	public function email()
	public function is_valid()
}
 
class chat
{
	public function __construct(Database $db, Validate $validate)
	public function send($message)
	{
		// validacja
               // zapis
	}
	public function receive()
}
[PHP] pobierz, plaintext 

Ten post edytował skowron-line 24.04.2013, 08:27:43