Czy to jeszcze jest Fabryka?, Oceńcie proszę czy wyszedłem już poza koncepcję Fabryki (Factory) Opcje

[djgrin]

Cześć wszystkim,
krótko mówiąc - miałem kod oparty o wzorzec fabryki i po licznych zmianach ewoluował on do czegoś takiego:

[PHP] pobierz, plaintext 
class TheFactory {
	// ...
 
	public function theFactoryMethod($what) {
		switch($what) {
			case 'one': return new ClassA();
				break;
			case 'two': return new ClassB();
				break;
			case 'three': return new ClassC();
				break;
 
			// ...
		}
	}
 
	// ...
}
 
class TheClass extends TheFactory {
	// ...
}
 
$obj = new TheClass();
$newObj = $obj->theFactoryMethod('one');
[PHP] pobierz, plaintext 

Chodzi o to, że metoda produkująca obiekty przestała być statyczna, a ja wywołuję ją z obiektu. Klasy ClassA, B i C implementują oczywiście wspólny interfejs, pominąłem tutaj wiele szczegółów, żeby nie zaciemniać, ale mam nadzieję, że wiecie, o co chodzi...
I moje pytanie: czy to jeszcze opiera się o koncepcję Fabryki, czy już nie? Jak sądzicie?

Być może pytanie jest czysto teoretyczne (bo kod oczywiście działa), ale może pokażecie mi błędy, których ja już nie zauważam (IMG:style_emoticons/default/winksmiley.jpg) ...
Tak czy siak - liczę na opinie i dyskusję.

Pozdro

[-=Peter=-]

Jest to nadal metoda fabryczna, jednak jaki masz cel w tym aby ta metoda była nie statyczna? O to chodzi w tym wzorcu aby utworzyć obiekt klasy wyspecjalizowanej wywołując metodę statyczną klasy bazowej, po to aby nie tworzyć obiektu w celu utworzenia obiektu właściwego ;]

robiąc to statycznie na pewno działało by to szybciej. Wywołanie statycznej funkcji działa około 2 razy szybciej. Tak mi się o uszy obiło.

A nie obiło ci się o uszy, że nie powinno się rozpowszechniać "faktów"? Przed napisaniem takiej bzdury lepiej uruchom prostego benchmarka (10 linijek kodu) i sprawdź, czy to prawda.

[djgrin]

jaki masz cel w tym aby ta metoda była nie statyczna?

Chodzi o to, że obiekt już i tak istnieje (kontynuując przykład, chodzi o obiekt klasy TheClass). A dodatkowo w metodzie fabryki ustawiane są pewne parametry zwracanego obiektu, które bazują na atrybutach właśnie obiektu TheClass (mówiąc w dużym skrócie). Stąd to niestatyczne wywołanie.
Czy to ma sens (IMG:style_emoticons/default/smile.gif) ?

Bo wolałem zrobić tak, niż np.:

[PHP] pobierz, plaintext 
TheClass::theFactoryMethod($obj, 'one');
[PHP] pobierz, plaintext 

(czyli gdyby metoda była statyczna).

Ten post edytował djgrin 31.05.2010, 12:12:27