Rozšíření Object o metodu factory()

Bylo. I ObjectFactory je zavádějící, ale nemůžu přijít na názvosloví, jak tu třídu výstižněji pojmenovat.

Myslim, že v Nette\Objectu by mělo být co nejméně funkčnosti. Stačilo by mi, kdyby dělal jen takové ty věci jako vyhazování výjimek při přístupu k neinicializované property a podobně. Pokud někdo potřebuje tuhle metodu (a nepopírám její jistou užitečnost), nechť si Nette\Object podědí a používá ten.

S čistotou Nette\Objectu souhlasím. Pokud by se statická metoda přidala přímo jako Object::construct() (varianta newInstance() taky pěkná), nelze ji v potomkovi předefinovat jako nestatickou. Pokud by ji někdo v potomkovi předefinoval jako statickou s jiným počtem povinných parametrů, dostane E_STRICT.

extensionMethod() použít nejde, metoda se pak nedá volat staticky. Navíc get_called_class() lze volat jen v určitých případech.

Měl jsem spíš na mysli přidat novou třídu, která by tuto funkčnost poskytovala a rozšiřovala Nette\Object. Ostatně tak to dělám teď já.

namespace Nette;

abstract class ObjectFactory extends Object // název třídy berte s rezervou, to mi fakt nejde
{
        public static function construct()
        {
                $class = get_called_class();

                $ref = new \ReflectionClass($class);
                if (!$ref->hasMethod('__construct')) {
                        return new $class;
                }

                $args = func_get_args();
                return $ref->newInstanceArgs($args);
        }
}

Editoval Milo (2. 11. 2011 18:02)

To jsem blbě napsal. Funkci get_called_class() nelze volat z extension metody tak, aby vrátila název třídy ke které je extension metoda přidaná. Extension metoda by musela nějak zjišťovat, ke které třídě je přidaná. Příklad:

class MyClass extends Nette\Object
{
}

function construct()
{
	// Jak tady zjistit, že jsem na třídě MyClass abych mohl vrátit instanci MyClass?
}

MyClass::extensionMethod('construct', 'construct');

// Ale extension metodu teď nelze volat staticky, takže to je stejně jedno
$o = MyClass::construct()->fluent();	// nelze

Stačí počkat pár let na PHP 5.5 a nebude to potřeba ;)

Nebo pokud ti jde o stručnost, tak si můžeš udělat takovou tu zvrácenou funkci

function vratSe($object)
{
	return $object;
}

vratSe(new Vec())->fluent();

@Honza Marek: já myslel, že to má být už v 5.4: new Cls()->doSomething()...

Honza Marek napsal(a):

Stačí počkat pár let na PHP 5.5 a nebude to potřeba ;)

Nette\Object je vlastně jen fixem na to čekání ;-) Proto mi metoda newInstance příp newInstanceArgs připadá fajn, protože řeší něco, co se musí v PHP zatím obcházet.

HosipLan napsal(a):

@Milo: Znáš __callStatic() ?

Asi mi docvalko. Máš na mysli… ?

abstract class Nette\Object
{
	public static function __callStatic($name, $args)
	{
		if($name === 'construct') {
			$class = get_called_class();

			$ref = new \ReflectionClass($class);
			if (!$ref->hasMethod('__construct')) {
				return new $class;
			}

			return $ref->newInstanceArgs($args);
		}

		return ObjectMixin::callStatic(get_called_class(), $name, $args);
	}
}

Tak co třeba takhle? Má cenu psál pull a testy?

public static function __callStatic($name, $args)
{
	$methodName = strtolower($name);
	if ($methodName === 'newinstance' || $methodName === 'newinstanceargs') {
		$class = get_called_class();

		$ref = new \ReflectionClass($class);
		if (!$ref->hasMethod('__construct')) {
			return new $class;
		}

		if ($methodName === 'newinstanceargs') {
			if (!array_key_exists(0, $args)) {
				$args = array();

			} elseif ($args[0] instanceof \Traversable) {
				$args = iterator_to_array($args[0], false);

			} elseif (!is_array($args[0])) {
				throw new InvalidArgumentException("First parameter of $class::newInstanceArgs() must be an array, " . gettype($args[0]) . " given.");

			} else {
				$args = & $args[0];
			}
		}

		return $ref->newInstanceArgs($args);
	}

	return ObjectMixin::callStatic(get_called_class(), $name, $args);
}

FYI PHP 5.4 již patch pro „instance method call“ obsahuje, http://svn.php.net/…PHP_5_4/NEWS?…, takže bych si dal pohov a věnoval se raději nějakým těm bugům a pull requestům, na githubu jich vidím požehnaně

Tenhle návrh (jak už tu padlo) přichází s křížkem po funuse. Totiž v době PHP 5.4 chtít něco výhradně pro PHP 5.2 je k zamyšlení (5.2 je už skoro něco jako PHP 4) – v 5.3 jde snadno workaroundovat a v 5.4 je to podporované nativně:

(new Something)->foo();
(new Something(1, 'arg2'))->foo();

Milo napsal(a):

Člověče, Tebe to nějak rozčílilo…

Ale má pravdu. :)

Milo napsal(a):

Jaký snadný workaround pro 5.3 máš na mysli? Na nic univerzálnějšího jsem nepřišel.

Vpodstatě to s get_called_class (nepřijde mi to tak problematické), příp. třeba:

use Nette\Reflection\ClassType;
ClassType::from(get_called_class())->newInstance(1, 'arg');

>

Za jak dlouho myslíte, že 5.4 bude stable?

Do konce roku, maximálně v lednu. Už je RC1.