Drukowana wersja tematu

Napisany przez: sf 7.03.2007, 13:29:42

Posiadam klasę HFilters:

[PHP] pobierz, plaintext 
<?php
class HFilters
{
	private $aFilterList = http://www.php.net/array();
 
 
	private $oContext;
 
 
	public function __construct(& $oContext)
	{
		$this->oContext = & $oContext;
	}
 
 
	public function add(iFilter $oFilter, $iPreWeight = 10, $iPostWeight = 10)
	{
		$this->aFilterList[] = $oFilter;
	}
 
 
	public function pre()
	{
		foreach($this->aFilterList as $oFilter) {
			$oFilter->pre($this->oContext);
		}
	}
 
 
	public function post()
	{
		foreach($this->aFilterList as $oFilter) {
			$oFilter->post($this->oContext);
		}
	}
}
?>
[PHP] pobierz, plaintext 

IFilter wygląda następująco:

[PHP] pobierz, plaintext 
<?php
interface iFilter
{
	public function pre($oContext);
 
 
	public function post($oContext);
}
?>
[PHP] pobierz, plaintext 

Mam problem z zaimplementowaniem kolejności wykonywania metod pre() i post() w HFilter wg ustalonwej wagi podanej w metodzie add(). Czy ktoś ma jakiś pomysł?

Napisany przez: mike_mech 7.03.2007, 13:50:41

Proste.
Zamiast HFilters::aFilterList daj HFilters::aPreFilterList i HFilters::aPostFilterList. Potem przy wrzucaniu daj:

[PHP] pobierz, plaintext 
<?php
	// ...
 
	public function add(iFilter $oFilter, $iPreWeight = 10, $iPostWeight = 10)
	{
		$this->aPreFilterList[$iPreWeight] = $oFilter;
		$this->aPostFilterList[$iPostWeight] = $oFilter;
 
		// Tutaj posortuj tak jak Ci wygodnie
	}
 
	// ...
?>
[PHP] pobierz, plaintext 

A na końcu wykonaj filtry, uwzględniając powyższe zmiany.

P.S.
Zamiast pętli po tablicach nie lepiej użyć iteratorów?

P.S.2
Po co Ci & w konstruktorze?

update (po przemyśleniu)
Tak w ogóle to moim zdaniem źle się za to zabrałeś.
Skoro (na to przynajmniej) metody pre() i post() są widocznie rozłączne to czemu nie dasz:

[PHP] pobierz, plaintext 
<?php
 
abstract class AbstractFilter implements Filter
class PreFilter extends AbstractFilter
class PostFilter extends AbstractFilter
class FilterChain implements Filter
 
?>
[PHP] pobierz, plaintext 

Miałbyś wtedy łańcuchy filtrów. Każdy taki łańcuch wywołałby wszystkie swoje filtry niezależnie i w dowolnej kolejności. Byłoby troszkę ładniej i przejrzyściej zorganizowane.

Napisany przez: sf 7.03.2007, 13:51:43

Nie sprawdzi się jeśli ustawimy dla dwóch filtrów taką samą wagę, jeden nadpisze drugi.

W tym przypadku nie robi to raczej różnicy.. bo implemetnacja i tak nie wychodzi nigdzie na zewnątrz. Chyba, że itereatory są szybsze.


Mój głupi błąd.

Dopisek:
To musi być jedna klasa ponieważ np. załóżmy, że mamy filtr benchmark.. nie chce tworzyć benchmarkStart i benchmarkEnd. Wole to umieścić w jednej klasie.

Napisany przez: mike_mech 7.03.2007, 13:55:07

To u góry to był tylko zarys pomysłu.
Już Twoja w tym działka jak stworzyć te tablice żeby się nie nadpisały. Przecież nie jest to nic strasznie trudnego

Napisany przez: sf 7.03.2007, 15:52:49

Implementacja wcale nie jest taka prosta. Ponieważ klucz musi być unikalny, czyli można by to rozwiązać poprzez dodanie iteracyjnego przyrostku, ale to powoduje, że musimy ustalić zakres. Jeśli to będzie liczba złożona z jednej cyfry to mamy ograniczenie do 10 pozycji tylko.

Przy okazji Twój przykład kopiuje dwukrotnie ten sam obiekt. Rozwiązałem całość w następujący sposób.

[PHP] pobierz, plaintext 
<?php
class HFilters
{
	/** Filters objects list. */
	private $aFilters = http://www.php.net/array();
 
 
	/** Pre methods order. */
	private $aPreOrder = http://www.php.net/array();
 
 
	/** Post methods order. */
	private $aPostOrder = http://www.php.net/array();
 
 
	/** Context objects. */
	private $oContext;
 
 
	public function __construct($oContext)
	{
		$this->oContext = & $oContext;
	}
 
 
	public function add(iFilter $oFilter, $iPreWeight = 10, $iPostWeight = 10)
	{
		$this->aFilters[] = $oFilter;
		$this->aPreOrder[] = $iPreWeight;
		$this->aPostOrder[] = $iPostWeight;
	}
 
 
	public function pre()
	{
		http://www.php.net/asort($this->aPreOrder);
		foreach($this->aPreOrder as $iKey => $iWeight) {
			$this->aFilters[$iKey]->pre($this->oContext);
		}
	}
 
 
	public function post()
	{
		http://www.php.net/asort($this->aPostOrder);
		foreach($this->aPostOrder as $iKey => $iWeight) {
			$this->aFilters[$iKey]->post($this->oContext);
		}
	}
}
 
?>
[PHP] pobierz, plaintext 

Napisany przez: mike_mech 7.03.2007, 19:20:47

Niekoniecznie.
Jeśli taki indeks jest już zajęty to można z niego zrobić tablicę i zagnieździć się głębiej.
Ogólnie.
1. Sprawdzam czy podana waga istnieje.
2. Jeśli nie to dodaję do tablicy.
3. Jeśli tak to sprawdzam czy pod indeksem jest tablica. Jeśli tak to dopisuję się do niej (z kolejnym indeksem)
4. Jeśli indeks istnieje i nie jest tablicą. To wyjmuję obiekt znajdujący się pod tym indeksem a wkładam tablicę gdzie ten obiekt jest jedynym elementem.

To bardzo prosty algorytm.

Nieprawda.
Przeanalizuj poniższy przykład:

[PHP] pobierz, plaintext 
<?php
 
class testClass {
	public $property = 10;
}
 
$testObject = new testClass();
 
$array1 = http://www.php.net/array();
$array1[] = $testObject;
$array2 = http://www.php.net/array();
$array2[] = $testObject;
 
$array1[0]->property = 20;
 
http://www.php.net/echo $array2[0]->property;
 
?>
[PHP] pobierz, plaintext 

(PHP5 naprawdę już nie jest takie ułomne jeśli chodzi o obiekty )

Napisany przez: Strzałek 8.03.2007, 12:22:45

Rozwiązanie takich filtrów jak dla mnie jest trochę mało wygodne. Ja rozwiązuję to o wiele wygodniej i filtry mają większe możliwości. Mianowicie filtr implemetuje interface z jedną metodą - execute która przyjmuje jako argument $filterChain'a z publiczną metodą next() która wykonuje następny filtr. Kod zapisany przed wykonaniem metody next to pre, a po metodzie to post. Proste w działaniu, banalne w implementacji.

[PHP] pobierz, plaintext 
<?php
 
interface IFilter {
 public function execute(IFilterChain $filterChain);
}
 
interface IFilterChain {
 public function addFilter(IFilter $filter);
 public function http://www.php.net/next();
}
 
 class FilterChain implements IFilterChain {
 
   public $filters = http://www.php.net/array();
 
 
	public function addFilter(IFilter $filter){
	  $this -> filters[] = $filter;
	}
 
	public function http://www.php.net/next(){
	  if($filter = http://www.php.net/array_shift($this -> filters)){
	   $filter -> execute($this);
	 }
	}
 
	public function execute(){
	 $this -> http://www.php.net/next();
	}
 
 }
 
 
 class AuthFilter implements IFilter {
 
	public function execute(IFilterChain $filterChain){
 
	  if($auth -> isLoged()){
	   $this -> http://www.php.net/next();
	  }else{
		http://www.php.net/echo 'zaloguj się';
	  }
 
	}
 
 }
 
 class ExecutionFilter implements IFilter {
 
   public function execute(IFilterChain $filterChain){
 
   
	//odpalamy aplikacje
   $filterChain -> http://www.php.net/next();
   }
 
 }
 
 
$filterChain = new FilterChain();
 
$filterChain -> addFilter(new AuthFilter());
$filterChain -> addFilter(new ExecutionFilter());
 
$filterChain -> execute();
 
?>
[PHP] pobierz, plaintext 

Ufff, pisane z palca na lekcji historii (mamy w sali od inf.)

Mam nadzieję że nie ma błędów i ogólny koncept zrozumiany.
Pozdrawiam