Implementacja klas, Czy potrzebna? Opcje

[kezard]

Witam!

Załóżmy że mamy przykladowa funkcje doSomething() :

[PHP] pobierz, plaintext 
function doSomething($obj)
{
    $obj->imie   = 'Adam';
    $obj->nazw  = 'Kowalski';
 
   return $obj;
}
 
doSomething($test);
echo $test->imie.' '.$test->nazw;
[PHP] pobierz, plaintext 

Ten kod oczywiscie zadziała, jednak pytanie brzmi czy warto implementowac klase dla obiektow wykorzystanych w przykladzie. Czy jest sens, nawet jesli zwracaja wiekszoa ilosc danych niz tylko imie i nazwisko? Czy warto tworzyc dodatkowy kod aby byc OOP czy nie marnowac czasu"skoro dziala" ? Jakie sa roznice techniczne?

Dla porownania :

[PHP] pobierz, plaintext 
class Test
{
	public $imie;
	public $nazw;
}
 
function doSomething($obj)
{
    $obj->imie   = 'Adam';
    $obj->nazw  = 'Kowalski';
 
   return $obj;
}
 
$test = new Test;
doSomething($test);
echo $test->imie.' '.$test->nazw
?>
[PHP] pobierz, plaintext 

Dodam że wykorzystuje to przy przekazywaniu danych do widokow (MVC). Czy warto tworzyc implementacje?

pozdrawiam.

Ten post edytował kezard 20.09.2010, 08:29:50

[Crozin]

Dlaczego? Uwazam ze to marnowanie czasu na pisanie funkcji get i set tylko po to zeby przepisywac wartosci do zmiennych. Pozatym niepotrzebnie sie kod powieksza. Dlaczego nie upraszczac sobie bezposrednim dostepem do zmiennych?

Dzięki temu zyskujesz większą kontrolę nad dostępem do danych zawartych w obiekcie. Masz dodatkowy "punkt dostępu". Nawet jeżeli w tej chwili jest to tylko przypisanie/zwrócenie pola, w przyszłości może dość potrzeba rozbudowy tego (np. dodania jakieś walidacji danych). Co prawda w przypadku PHP takie gettery/settery to dodatkowe obciążenie, ale w praktyce jest ono zerowe, a elastyczność/przyszłościowość kodu na tym zyskuje.

I dobrze ze PHP jest malo struktur bo one sa przeznaczone dla jezykow nieobiektowych

PHP to nie jest język obiektowy - nawet udawanie obiektowego słabo mu wychodzi. (IMG:style_emoticons/default/smile.gif)

W tej chwili wyglada to tak ze mam w kodzie takie "kwiatki" :

Wygląda mi na to, że używasz pętli for, zamiast foreach, która by prawdopodobnie uprościła zapis. Na początek możesz pokombinować w tę stronę.