Filtry i kolejność wykonywania Opcje

[sf]

Posiadam klasę HFilters:

[PHP] pobierz, plaintext 
<?php
class HFilters
{
	private $aFilterList = 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ł?

Ten post edytował sf 7.03.2007, 13:30:49

[mike]

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.

[sf]

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

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

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

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

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.

Ten post edytował sf 7.03.2007, 13:53:37