Piszę aplikacje obiektowych formularzy i natknąłem się na mały problem. OF moga miec klasy Rule które walidują pola formularza i pluginy które wykonują na nich operacje np md5 na polu po stronie klienta.

Na początku chciałem zrobic te dwie rzeczy oddzielnie ale pomyslałem sobie, że zastosuje takie samo API dla Rules i Pluginów i zrobie to za pomocą dekoratora np

[PHP] pobierz, plaintext 
<?php
new MD5hash( new IsNotNullRule( new isNumericRule( new Form() ) ) );
?>
[PHP] pobierz, plaintext 

Co Wy o tym myslicie?

Ja też ostatnio miałem taki problem. Postanowiłem połączyć mapper z Rules Api i tak oto powstał jakiś dziwny stwór który dzięki Bogu działa

Do mnie nie przemawia łączenie reguł z pluginami. Reguły służą do sprawdzenia poprawności danych, a pluginy mają na tych danych operować. Poza tym dekorator nie jest najlepszym wyjściem w tym przypadku, jeżeli już musisz tak zrobić. Wzorzec ma to do siebie, że nie zapewnia odpowiedniej kolejności wykonania. Więc możesz przez przypadek złe dane hashować. Ja bym to osobiście rozłączył w jakiś sposób.

Wszystko zależy do tego w jakiej kolejności utworzy się instancje (patrz kod wyżej).

Poza tym może zajśc potrzeba kiedy najpierw należy jakies dane "zoperowac" a dopiero potem je zwalidowac. W przypadku rozdzielenia nie da się tego zrobic.

To nie jest do końca prawdą. Nie mówię o wywołaniu metod, tylko wykonaniu czynności filtrów, a to jest duża różnica. Oto prosty przykład.

[PHP] pobierz, plaintext 
<?php
abstract class Decorator {
	public function __construct($nextDecorator = null) {
  if (!empty($nextDecorator) && $nextDecorator instanceof Decorator) {
	  $this->next = $nextDecorator;
  }
	}
 
	abstract public function dosmth($param);
 
	protected $next;
}
 
class Decorator1 extends Decorator {
	public function __construct($nextDecorator = null) {
  parent::__construct($nextDecorator);
	}
 
	public function dosmth($param) {
		echo '<1>';
		if (!empty($this->next)) {
			$this->next->dosmth($param);
		}
	}
}
 
class Decorator2 extends Decorator {
	public function __construct($nextDecorator = null) {
  parent::__construct($nextDecorator);
	}
 
	public function dosmth($param) {
		if (!empty($this->next)) {
			$this->next->dosmth($param);
		}
		echo '<2>';
	}
}
 
class Decorator3 extends Decorator {
	public function __construct($nextDecorator = null) {
  parent::__construct($nextDecorator);
	}
 
	public function dosmth($param) {
		echo '<3>';
		if (!empty($this->next)) {
			$this->next->dosmth($param);
		}
	}
}
 
$d = new Decorator1(new Decorator2(new Decorator3()));
$d->dosmth('');
?>
[PHP] pobierz, plaintext 

Niestety dekorator wymaga trzymania porządku w kodzie i pozwala na takie przekłamania...