[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

[PrinceOfPersia]

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?

to pewnie zależy od konkretnej sytuacji, ale jakbyś stworzył osobną klasę (albo choćby zestaw funkcji - nie bądźmy zbyt oop na siłę, bo to niezdrowe (IMG:style_emoticons/default/wink.gif) , to byś mógł ponownie użyć kodu walidacji do innych projektów/modułów niż tylko ten jeden czat.

2. Czy ma sens robić klasę zawierającą tylko jedną metodę?

to zależy.

Ten post edytował PrinceOfPersia 24.04.2013, 00:53:45