Najpierw podam kod:

Klasa String:

[PHP] pobierz, plaintext 
<?php
class String {
	private $sTxt;
 
	public function set( $sTxt ) {
		$this->sTxt= $sTxt;
	}
 
	public function get() {
		return $this->sTxt;
	}
 
 
	public function length() {
		return strlen( $this->sTxt );
	}
 
	public function __toString() {
		return $this->get();
	}
}
 
?>
[PHP] pobierz, plaintext 

Klasa Coment:

[PHP] pobierz, plaintext 
<?php
class Comment {
	private $title;
 
	public function __construct() {
		$this->title = new String();
	}
 
	public function __call( $sName, $aArguments ) {
		return $this->$sName;
	}
 
}
?>
[PHP] pobierz, plaintext 

Odwołanie:

[PHP] pobierz, plaintext 
<?php
$c1 = new Comment();
$c1->title()->set('jakis tytul');
echo'Dlugosc tekstu to ' . $c1->title()->length() . ' znakow';
 
?>
[PHP] pobierz, plaintext 

Zastosowałem tutaj IMHO brzydkie rozwiązanie z użyciem metody __call.

Innym rozwiązaniem jest zastosowanie public $title. Wówczas odwoływalibyśmy się w ten sposób:

[PHP] pobierz, plaintext 
<?php
$c1 = new Comment();
$c1->title->set('jakis tytul');
echo'Dlugosc tekstu to ' . $c1->title->length() . ' znakow';
 
?>
[PHP] pobierz, plaintext 

Czyli bez użycia title(). Jednak chyba nie jest to zbyt dobry pomysł ponieważ zmienna będzie publicznie dostępna, a takto mamy tylko dostęp do jej metod (title->set, title->length, etc.), więc nie można jej np. usunąć z zewnątrz.



Żeby nie zakładać nowego tematu napiszę, że brakuje mi bardzo przeładowywania metod w taki oto sposób:

[PHP] pobierz, plaintext 
<?php
class Comment {
public function __construct() {}
public function __construct( $sTitle ) {}
public function __construct( $sTitle, $sBody ) {}
}
?>
[PHP] pobierz, plaintext 

Jestem ciekaw jak sobie radzicie z tym problemem.

IMO klasa Comment powinna wyglądać tak:

[PHP] pobierz, plaintext 
<?php
class Comment {
	private $title;
 
	public function __construct() {
		$this->title = new String();
	}
 
	public function getTitle() {
		return $this->title;
	}
 
}
?>
[PHP] pobierz, plaintext 

I potem odwołujesz się tak:

[PHP] pobierz, plaintext 
<?php
$c1 = new Comment();
$c1->getTitle()->set('jakis tytul');
echo'Dlugosc tekstu to ' . $c1->getTitle()->length() . ' znakow';
 
?>
[PHP] pobierz, plaintext 

Oczywiście wiem o tym ;-) Zapomniałem dopisać to rozwiązanie na Forum. Obecnie korzystam właśnie z takiego

Trochę pokręciłem. To które podałeś matid wydaje się najrozsądniejsze i najbezpieczniejsze.

Jednak czy istnieje jeszcze inne rozwiązanie? Może jakieś wariacje z __get i __call? Chodzi mi o to, aby odwołać się poprzez

[PHP] pobierz, plaintext 
<?php
$c1->title->length()
?>
[PHP] pobierz, plaintext 

Jednak jeśli użyję __get to i tak nie sprawdzi się to, ponieważ zmienna $title istnieje i nie dojdzie do wykonania metody __get.

Innym rozwiązaniem może być dodanie jakiegoś przedrostka np. get_ (gdzieś widziałem klasę z wariacją __get lub _call (już nie pamiętam) i przedrostka get_ (wartość) lub Get (odwołanie do metody). Jednak to mija się z celem, już lepiej stworzyć metodę GetTitle() i zwrócić referencję do obiektu, która to jest domyślnie zwracana w php 5.

Zależy mi na tym by wartość była private ze względów bezpieczeństwa. Jestem ciekaw jak rozwiąże to Seth w swoim frameworku Carbon, który ma się odwoływać m.in. tak w swoim systemie szablonów

[PHP] pobierz, plaintext 
<?php
{$carbon->content->nowosci->cos_tam->tresc}
?>
[PHP] pobierz, plaintext 

Pozostaje chyba jedynie albo zastępować np. content na getConent(), nowosci na getNowosci() albo zmienić access z private na public.

Druga kwestia

Przeładowywanie metod.
Czy pozostaje jedynie rozwiązanie przez stworzenie setterów?

[PHP] pobierz, plaintext 
<?php
 
$c1 = new Comment();
$c1->setTitle();
$c1->setBody();
}
?>
[PHP] pobierz, plaintext 

O ile łatwiej by było gdyby:

[PHP] pobierz, plaintext 
<?php
$c1 = new Comment();
$c2 = new Comment( 123 ); //id
$c3 = new Comment( 456, 'Tytul', 'Tresc' ); //id, tytul, tresc
?>
[PHP] pobierz, plaintext 

Można oczywiście skorzystać z funkcji func_get_args" title="Zobacz w manualu PHP" target="_manual

Czyżby ? Sprawdzałeś? Przeczytaj to -> http://www.php.net/manual/en/language.oop5...ading.php#53539
( notka by Dasko )

Czytałem czytałem i to wielokrotnie ;-)

Dla leniwych:

[PHP] pobierz, plaintext 
<?php
class MyClass {
 
   private $foo;
 
   public function __get($name) {
	   echo $name . ' doesn't exist...<br />';
   }
 
   public function __set($name, $value) {
	   echo 'Cannot set ' . $name . '. The requested property doesn't exist...<br />';
   }
 
}
 
// class' instance
$obj = new MyClass;
// trying to access foo - __get() is called
echo $obj->foo;
// trying to set foo - __set() is called
$obj->foo = 'newValue';
?>
[PHP] pobierz, plaintext 

Ja mam php Version 5.0.3 i wyskakuje mi

Pozostaje mi polecić php 5.1

Skoro chcesz się odwoływać do tego w ten sposób:
$cm->title->lenght();
No to jakby na to nie patrzeć 'title' jest własnością publiczną i żadna kwestia bezpieczeństwa nie jest naruszona.

Jest to wprawdzie przeciw zasadzie hermetyzacji, no ale skoro koniecznie tego potrzebujesz to chyba innego wyjścia nie ma.

Właśnie dlatego nie chcę aby $title było public. Zaciekawiło mnie to ponieważ Seth podawał na swoim blogu koncepcję swojego frameworka i odwoływał się w przykładzie właśnie w ten sposób.

Zostaję przy private i stworzeniu gettera.

Ciekawi mnie jeszcze druga kwestia, którą opisałem mianowicie jak sobie radzicie z przeładowywaniem metod. Czy korzystacie z funkcji func_get_args i tym podobnych czy raczej jesteście za tworzeniem dodatkowych setterów?

[PHP] pobierz, plaintext 
<?php
 
class String {
	private $sTxt;
 
	public function set( $sTxt ) {
		$this->sTxt = $sTxt;
	}
 
	public function get() {
		return $this->sTxt;
	}
 
	public function length() {
		return strlen( $this->sTxt );
	}
 
	public function __toString() {
		return $this->get();
	}
}
 
class Comment {
	private $title;
	private $body;
 
	public function __construct() {
		$this->title = new String();
		$this->body = new String();
	}
 
	public function getTitle() {
		return $this->title->__toString();
	}
 
	public function setTitle( $sTxt ) {
		$this->title->set( $sTxt );
	}
 
	public function getBody() {
		return $this->body->__toString();
	}
 
	public function setBody( $sTxt ) {
		$this->body->set( $sTxt );
	}
 
}
 
 
$cb = new Comment();
$cb->setTitle('sdafas');
$cb->setBody('body');
 
echo $cb->getTitle();
echo $cb->getBody();
 
?>
[PHP] pobierz, plaintext 

Myślę, że to będzie najrozsądniejsze rozwiązanie. Jeśli będę chciał odwołać się do jakiejś metody String przesłonię ją (Facade) w klasie Comment.

Ja osobiście nie lubię dodawać kolejnych setterów do containera. U mnie container ma zwracać poszczególne instancje, dodawać je lub usuwać. Czyli robie np. tak:

[PHP] pobierz, plaintext 
<?php
$Container->addObject( new Parameter( 'Name', 'Value' ) );
$Container->getObject( 'Name' )->setValue( 'New Value' );
echo $Container->getObject( 'Name' )->getValue();
?>
[PHP] pobierz, plaintext 

Ale jak to mówią - każdy robi jak mu się podoba

Masz rację ;-) Coś dzisiaj ciężko idzie mi myślenie ;-) Co będzie jeśli rozbuduję klasę String? Aż strach pomyśleć ;-) Dodawanie nowej funkcjonalności (czyli nowych metod) w innych klasach ;/ a takto jedynie zostanie mi korzystanie z nowych features ;-)

Z metodami __get() czy też może i __call() trzeba bardzo uważać. Ja kiedyś sobie przez __get() pobierałem moduły jądra i jakież moje zdziwienie było, kiedy pojawiały mi się błędy... Powód? Pobrany moduł przez __get() chciał pobrać inny moduł. Myślałem, że tak się da, a jednak PHP5(1) ma blokade na takie odwołania (ochrona przed zapętleniem).

(1) Było to kilka miesięcy temu, dopisałem się do innego zgłoszenia na bugtracku, nie wiem jak jest obecnie - warto wpierw sprawdzić to, zanim będzie się używać.

@dag: Rozwiązanie jest proste:

[PHP] pobierz, plaintext 
<?php
class Comment {
	private $aAttributes = array(
		'title' => null,
	);
 
	public function __construct() {
		$this->aAttributes['title'] = new String();
	}
 
	public function __get($sName) {
		return array_key_exists($sName, $this->aAttributes) ? $this->aAttribute[$sName] : null;
	}
}
?>
[PHP] pobierz, plaintext 

Co do przeładowania konstruktora: robienie dodatkowych setterów nie wchodzi w grę, bo to zupełnie inna funkcjonalność niż konstruktor z wieloma parametrami. Nie widzę przeszkód w dodaniu wartości domyślnej i func_get_args.

@matid: A czy ten Container nie przypomina za bardzo hashmapy ?

[PHP] pobierz, plaintext 
<?php
class Type {
	private $mType;
 
	public function set( $mType ) {
		$this->mType = $mType;
	}
 
	public function get() {
		return $this->mType;
	}	
 
	public function __toString() {
		return $this->get();
	}
}
 
class String extends Type {
		public function length() {
			return strlen( $this->get() );
		}
}
 
class Input extends String {
 
	public function toForm( $sName ) {
		$sForm = '<input type=\"text\" name=\"' . $sName . '\"';
 
		if( $this->get() != '' ) {
			$sForm .= ' value=\"' . $this->get() . '\"';
		}
 
		return $sForm . '/>';
	}
 
}
 
abstract class Bean {
	protected $aAttributes = array();
 
	public function __get( $sName ) {
		return array_key_exists( $sName, $this->aAttributes ) ? $this->aAttributes[ $sName ] : null;
	}	
 
	public function toForm() {
		$aForms = array();
 
		foreach( $this->aAttributes as $key=>$value ) {
			$aForms[ $key ] = $this->$key->toForm( $key );
		}
 
		return $aForms;
	}
}
 
class CommentBean extends Bean {
 
	public function __construct() {
		$this->aAttributes[ 'title' ] = new Input();
		$this->aAttributes[ 'body' ] = new Input();
	}
}
 
 
 
$cb = new CommentBean();
$cb->title->set('tytul1');
$cb->body->set('tresc1');
$aForms = $cb->toForm();
echo $aForms['title'];
 
$dlugosc = new Input();
$dlugosc->set( $cb->title->length() );
echo '<br/><br/>';
echo $dlugosc->toForm( 'dlugosc' );
 
echo '<hr/><hr/>';
 
//normalne wykonanie metody __toString()
echo $cb->title;
 
echo '<hr/>';
 
//przecinek zamiast kropki
echo $cb->title,'<hr/>';
 
//kropka zamiast przecinka
echo $cb->title.'<hr/>';
?>
[PHP] pobierz, plaintext 

Ciekawi mnie kwestia kropki i przecinka. Gdy jest kropka php nie wykonuje metody __toString(), a gdy przecinek wykonuje. Gdy zastosujemy przecinek to czy

[PHP] pobierz, plaintext 
<?php
echo $cb->title, '<hr/>';
?>
[PHP] pobierz, plaintext 

jest równoznaczne z:

[PHP] pobierz, plaintext 
<?php
echo $cb->title;
echo '<hr/>';
?>
[PHP] pobierz, plaintext 

na to wygląda.


Najlpeszmy i "najczystszym" rozwiązaniem było by chyba wprowadzenie czwartego typu access ;-), który by miał większość właściwości private jednak pozwalał by na zwrócenie obiektu. ;-)

Problem z kropką i __toString() to znany bug php.

To jest raczej feature, ale nadal bardzo uciazliwy




Przy echo, gdy rozdzielasz argumenty (echo to nie funkcja) przecinkiem, kazdy z argumentow jest wypisywany osobno -- __toString() zadziala, a przy laczeniu stringow kropka __toStrint() nie jest uruchamiane.

Taaaaak... this is not a bug, this is a feature. To nic, że kompletnie bez sensu, ale przecież "feature" brzmi dumnie .