ugrás a tartalomhoz

Method chaining és a konstruktorok

breakline · 2010. Feb. 17. (Sze), 00.34
Hi

Bocs a rossz fogalmazásért, nem tudom mi a magyar szakszó erre (összefűzés?).
Nagyon szeretem ezt a típusú írásmódot használni:

$a = new valami();
$a -> foo() -> bar() -> stb();
Viszont mégsem működik így:

$a = new valami() -> foo() -> bar();
A php oldalon fent van mint "bug", de igazából nem értem, mert logikus lenne, hogy működjön, vajon javítani fogják ezt, vagy valami számomra ismeretlen okból ez így marad (5.2.5 alatt még nem megy)?

http://bugs.php.net/bug.php?id=34502

Nem az lenne értelmes hogy a new valami() már használható objektumra mutat, ha a konstruktor elvileg (és azért van végülis) az objektumra kéne hogy mutasson (mintha return $this lenne)?
‹ Adatbázistábla tartalmának listázása Egy intranetes alkalmazás ›
 ■
1

factory

Drawain · 2010. Feb. 17. (Sze), 02.51
Írhatsz rá egy statikus factory függvényt. Nem muszáj olyannak lennie, mint amilyen a factory tervezési minta, le lehet egyszerűsíteni.


class valami
{
    public static function factory()
    {
        return new valami;
    }
}

$a = valami::factory()->foo()->bar();
2

5.3 -al sem akaródzik

h31st · 2010. Feb. 18. (Cs), 16.50
5.3 -al sem akaródzik működni...
a factory is jó megoldás, vagy egy pici funkció is megteszi:
function chain (&$c) {
	return $c;
	}

chain (new MyClass ()) ->ehlo ();
megjegyzem az eredeti két példa lényegesen különbözik, az első esetben $a-ban az eredeti obj marad, a másodikban meg a bar() visszatérési értéke...

a new myClass() "visszatérési értékének" nem sok köze van egyébként a __construct() függvényhez, legalábbis hiába return-ölsz bármit belőle, nem fog a példád szerinti $a -ba kerülni.

ha meg mindenképp magyarítani akarod, akkor megtenné a 'metódus láncolás' is :)

egyébként én is szeretem :)

vannak vele problémák és aggályok, az egyik: a hibák kezelése, nem dobhatsz vissza pl false-t mert akkor ugye megtörik a lánc... lehet mondjuk exception-özni, ami tönkrevágja az egésznek a legnagyobb előnyét, a rövidítést és átláthatóságot. vagy lehet használni egy 'chainbreaker' classt, ami valahogy így néz ki:
class ChainBreaker {

	/**	@var public error message string */
	public $cbError = '';

	/**	record some message on create
	 *	@param mixed error message or anything
	 */
	public function __construct ($p_error=NULL) {
		if (!is_null ($p_error))
			$this->cbError = $p_error;
		}

	/**	the meaning of life - catch every property access */
	public function __get ($key) {
		return $this;
		}

	/**	another part - catch every method access */
	public function __call ($key, $params) {
		return $this;
		}
	}
amit így lehet használni:
class myClass {

	function ehlo ($p=NULL) {
		if (empty ($p))
			return new ChainBreaker ('param is empty');
		else echo 'hello' . $p;
		return $this;
		}
	}
ez mindenre önmagát adja vissza (változókra is), így gyakorlatilag bármilyen chain végig tud futni, és a végén meg lehet nézni, hogy mi volt a hiba...

egy másik baj, hogy okosan kell megtervezni a class-eket, mert ha a methodok egyik fele chainelhető a másik meg nem, a harmadik fele meg nem önmagát dobja vissza, hanem valami tökmás objektumot (neadjisten skalárt), akkor átláthatatlan lesz az egész... plusz, sokszor nem egyértelmű, hogy az adott objecten kellene csinálni valamit, vagy egy új objektum kell, megváltozott tulajdonságokkal. pl: string obj (ami ugyan nincs, de lehet írni ;) ) esetén a $myString->substr() hívásnak a legritkább esetben kell saját magát átírnia, viszont ha mindig új string objektet dobsz vissza, akkor többszörös futásidővel lehet számolni, mintha saját magát módosítgatná...