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 ...
Tak czy siak - liczę na opinie i dyskusję.

Pozdro

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 ;]


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.

Oni wszyscy kłamią?

http://www.google.pl/#hl=pl&source=hp&...708fd10123afbf1

Powiem tyle, że strzeliłeś sobie w kolano Przeczytaj głębiej np. wpis w pierwszym wyniku w google w linku który podałeś...

Być może było to kiedyś prawdą, być może odnosi się to do wywołania metody niestatycznej (obiektowej) w sposób statyczny - wtedy faktycznie widać różnicę że wywołanie statyczne metody obiektowej jest wolniejsze od wywołania na obiekcie i wtedy taką metodę należy zamienić na statyczną - wzrost wydajności o ok 1/4.

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 ?

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

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

(czyli gdyby metoda była statyczna).