[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

[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

[sajegib]

O to mi chodziło, dzięki!

[q3trm]

1. Załóżmy, że mam do pobrania z formularza plik($_FILES) oraz jakąś opcję($_POST), czy odebrać te dane w jednej klasie?

[PHP] pobierz, plaintext 
  class DataForm 
 {
   protected $file;
   protected $option;
 
     public function __construct()
    {
        $this ->file = $_FILES['file'];
        $this ->option = $_POST['option'];
     } 
  }
[PHP] pobierz, plaintext 

2. Atrybuty $file i $option zrobić statycznymi, czy lepiej przekazywać je za pomocą delegacji?

[sajegib]

[PHP] pobierz, plaintext 
public function __construct($_FILES['file'], $_POST['option'])
{
$this->file = $_FILES['file']'
$this->option = $_POST['option'];
}
[PHP] pobierz, plaintext 

Ten post edytował sajegib 28.04.2013, 19:20:32

[q3trm]

[PHP] pobierz, plaintext 
public function __construct($_FILES['file'], $_POST['option'])
{
$this->file = $_FILES['file']'
$this->option = $_POST['option'];
}
[PHP] pobierz, plaintext 

Super globalnych nie można deklarować jako parametrów metod.

[sajegib]

przy tworzeniu obiektu:

[PHP] pobierz, plaintext 
$post = $_POST.....
$files =$_FILES...
$x = new DataForm($post, $files);
[PHP] pobierz, plaintext 

[q3trm]

Ciągle mam uczucie, że powinno się to rozdzielić na dwa kontrolery. Niby dane z jednego formularza, ale POST i FILES niewiele mają ze sobą wspólnego, każda pójdzie odrębną strukturą klas(patrząc na UML).