[PHP] return object in constructor werkt niet

[Verwijderd]

Ik wil een object in een constructor returnen, maar dit werkt niet.

PHP:

class quickie {

    public function quickie ($obj) {

        $strPathQuickies = PATH_LIB . '/quickie/';
        
        if (is_file($f = $strPathQuickies . $obj . '.php')) {
            require_once $f;
            return new $obj;
        }
    }
}

Ergens doe ik dus:

PHP:

$quickie = new quickie('form');

Dan wil ik dus gereturned krijgen de class in $obj.php zeg maar, maar ipv daarvan hou ik gewoon de quickie class.

Weet iemand iets dat ik over het hoofd zie?

edit: beveiliging zit verder wel goed, heb het voorbeeld beetje ingekort.
edit2: heb die '$this hack' al geprobeerd

[ Voor 12% gewijzigd door Verwijderd op 20-05-2007 01:22 ]

[Snake]

Je moet niet quickie pakken als constructor gelijk bij java, maar __construct

[Verwijderd]

MOET niet maar kan wel Ja, ik weet php5 enzo maar vind het zo lelijk staan

Maar annyhowz dat maakt natuurlijk geen verschil he

[Grijze Vos]

Een constructor zal -altijd- het net aangemaakte object (in dit geval quickie) returnen. Krijg je hier trouwens geen notice van ofzo? Een constructor hoort namelijk niets te returnen.

[ Voor 6% gewijzigd door Grijze Vos op 20-05-2007 01:07 ]

[Verwijderd]

Nope, geen notice. Hoe stel je voor dat ik dit dan doe, kan wel in de constructor een var aanmaken en die dan extern oppakken, maar heb liever dat het gaat werken zoals boven beschreven

^ zo maar opgelost.

[ Voor 8% gewijzigd door Verwijderd op 20-05-2007 02:24 ]

[PowerSp00n]

Ik denk dat jij meer zoekt naar de autoload mogelijkheid: http://nl3.php.net/autoload ?

[Verwijderd]

Nee, ook dat zou het object niet vanuit de constructor returnen.

Heb het alsvolgt maar opgelost:

PHP:

    $quickie = new quickie('form');
        
    $quickie->obj->new_element('input', array(
        'label' => 'Username',
        'type' => 'text',
        'name' => 'username'
    ));

PHP:

class quickie {

    ## xtemplate object
    private $xtpl;
    ## html object
    public $obj;

    public function quickie ($obj) {

        $strPathQuickies = PATH_LIB . '/quickie/';

        if (is_file($f = $strPathQuickies . $obj . '.php')) {
            require_once $f;
            $this->obj = new $obj;
        }
    }
}

[Infinitive]

Volgens mij is dit een typisch geval van de Factory Method

[BikkelZ]

Geen idee wat je wil bereiken. Een contstructor maakt altijd een nieuwe instantie van het object aan.

Ben jij niet gewoon heel erg op zoek naar een singleton methode voor PHP (een instantie maximaal van een object)?

[ACM]

BikkelZ schreef op zondag 20 mei 2007 @ 11:05:
Ben jij niet gewoon heel erg op zoek naar een singleton methode voor PHP (een instantie maximaal van een object)?

Nee, hij is op zoek naar een factory pattern/method/constructie, dat probeert ie iig te maken
Singleton is iets heel anders, hoewel je je factory wel zo kan maken die ie ook singleton's produceert.

[mithras]

Als je het dan zo oplost:

PHP:

class quickie {

    function __construct ($obj) {

        $this->obj = $obj;
    }

    public function get_object () {

        $strPathQuickies = PATH_LIB . '/quickie/';
        
        if (is_file($f = $strPathQuickies . $this->obj . '.php')) {
            require_once $f;
            return new $this->obj;
        }
    }
}

En dan:

PHP:

$quickie = new quickie('form');
$quickie = $quickie->get_object();

[Verwijderd]

Wanneer je een classe zeg maar wilt uitlezen, kort door de bocht, wordt uit de klasse de magic function __toString() aangeroepen (indien aanwezig). Ik denk dat je daar eens mee moet prutsen dmv overriden.

PHP:

class test{
    private $holder;

    public function __construct($value){
        $this->holder = $value;
    }
    public function __toString(){
       return $this->holder;
    }
}

$instance = new test('Hallo');
echo $instance;      //  'Hallo'

Edit:
Jammer, toString returned expliciet strings en geen objecten. __autoLoad() lijkt me de oplossing voor jouw probleem.

[ Voor 11% gewijzigd door Verwijderd op 20-05-2007 14:49 ]

[Verwijderd]

mithras schreef op zondag 20 mei 2007 @ 11:37:
Als je het dan zo oplost:

PHP:

code...

En dan:

PHP:

$quickie = new quickie('form');
$quickie = $quickie->get_object();

Dit maakt natuurlijk niet zo heel veel verschil met hoe ik het nu heb he

Verwijderd schreef op zondag 20 mei 2007 @ 14:21:
Wanneer je een classe zeg maar wilt uitlezen, kort door de bocht, wordt uit de klasse de magic function __toString() aangeroepen (indien aanwezig). Ik denk dat je daar eens mee moet prutsen dmv overriden.

PHP:

class test{
    private $holder;

    public function __construct($value){
        $this->holder = $value;
    }
    public function __toString(){
       return $this->holder;
    }
}

$instance = new test('Hallo');
echo $instance;      //  'Hallo'

En deze returned dan ook strings, objects doet ie niet

Infinitive schreef op zondag 20 mei 2007 @ 09:24:
Volgens mij is dit een typisch geval van de Factory Method

Voor zover ik er uit kan halen: jep dat klopt, maar hoe op te lossen?


Ik lees alwel op meerdere sites dat het gewoon niet mogelijk is om een object te returnen in een constructor. Manier die ik hier boven opgeef is dan denk ik kortste klap.

Tnx, voor alle replies alvast

[ Voor 21% gewijzigd door Verwijderd op 20-05-2007 14:56 ]

[_Sunnyboy_]

Zoals al eerder gezegd: Factory Method

PHP:

<?php 
class quickie { 

    public function createObject($objname) { 

        $strPathQuickies = PATH_LIB . '/quickie/'; 
         
        if (is_file($f = $strPathQuickies . $objname . '.php')) { 
            require_once $f; 
            return new $obj; 
        } 
    } 
} 
?>

En dan:

PHP:

<?
$quickie = Quickie::createObject('form');
?>

[Verwijderd]

Als ik dat doe krijg ik een E_STRICT error

code:

Non-fatal error: Non-static method quickie::create() should not be called statically, assuming $this from incompatible context (# 2048)

Uiteraard is hij niet critical dus kan em wel supressen, maar liever niet.

[Morax]

Probeer dan de functie eens hierin te wijzigen:

PHP:

public static function createObject($objname){

}

(Het gaat hier vooral om de toevoeging van het woord "static" in de functie declaratie )

[ Voor 5% gewijzigd door Morax op 20-05-2007 15:57 ]

[Verwijderd]

Oh, en het werkte in the first place al niet
Even voor de duidelijkheid, dit is wat ik nu heb:

PHP:

class quickie {

    public static function create ($obj) {
        $strPathQuickies = PATH_LIB . '/quickie/';
        if (is_file($f = $strPathQuickies . $obj . '.php')) {
            require_once $f;
            return new $obj;
        }
    }
}

$quickie = quickie::create('form');

[ Voor 14% gewijzigd door Verwijderd op 20-05-2007 16:18 ]

[Verwijderd]

wat werkt er precies niet?

Het zou moeten werken, weet je zeker dat het bestand bestaat etc ?

[mithras]

Verwijderd schreef op zondag 20 mei 2007 @ 14:54:
[...]


Dit maakt natuurlijk niet zo heel veel verschil met hoe ik het nu heb he

Maar jij moet nu werken met $quickie->obj, in plaats van (wat je eigenlijk wil) $quickie. Als je nu al lappen code hebt gebaseerd op $quickie, zou je dat allemaal moeten veranderen. Met mijn "methode" niet

Ik lees alwel op meerdere sites dat het gewoon niet mogelijk is om een object te returnen in een constructor. Manier die ik hier boven opgeef is dan denk ik kortste klap.

Hoewel die warning niet expliciet op php.net vermeld staat heb je afaik wel gelijk

[Verwijderd]

Verwijderd schreef op zondag 20 mei 2007 @ 16:31:
wat werkt er precies niet?

Het zou moeten werken, weet je zeker dat het bestand bestaat etc ?

Zegt gewoon dat ie de method niet kan vinden.

mithras schreef op zondag 20 mei 2007 @ 16:37:
[...]
Maar jij moet nu werken met $quickie->obj, in plaats van (wat je eigenlijk wil) $quickie. Als je nu al lappen code hebt gebaseerd op $quickie, zou je dat allemaal moeten veranderen. Met mijn "methode" niet

Kan ik natuurlijk ook gewoon er onder zetten:

PHP:

$quickie = $quickie->obj;

[ Voor 45% gewijzigd door Verwijderd op 20-05-2007 17:48 ]

[Verwijderd]

Dit topic is alweer een maandje oud, maar in de tussentijd had ik het gevoel dat het wel moet kunnen wat de topicstarter wilt bereiken, wat voor ons alsnog onbekend is of dit al is bereikt.. Vandaar dat ik mijn oplossing toch nog even post.

Het idee is om enkele magic functions te overloaden. Ik heb in dit voorbeeld hiervoor een klasse genaamd Proxy, welke als tussenlaag dient tussen de klasse en de scope waarvandaan je call-ed. Bij dat laatste dient de proxy genegeerd te worden. Zie tweede code voorbeeld:

PHP:

class Proxy{

    private $obj_class = NULL;
    
    public function __construct($className){
        $this->obj_class = new $className;
    }
    
    public function __call($funcName, $args){
        for($num = 0, $args_vars = array(); $num < count($args); $num++)
            $args_vars[] = '$args['.$num.']';

        eval('return $this->obj_class->$funcName(' . implode(", ", $args_vars) . ');');
    }

    public function __get($var){
        return $this->obj_class->$var;
    }
    
    public function __set($var, $val){
        $this->obj_class->$var = $val;
    }
    
    public function __isset($var){
        return isset($this->obj_class->$var);
    }
    
    public function __unset($var){
        unset($this->obj_class->$var);
    }

    public function __destruct(){
        unset($this->obj_class);
    }

    
}

Gebruik:

PHP:

    $obj = new Proxy("MijnTestKlasse");   
    $obj->test(1, 2, 3);     // functie uit MijnTestKLasse

    $obj->varA = 1;          // var uit MijnTestKlasse,  etc
    echo $obj->varA."<br>";
    $obj->varA = 2;
    echo $obj->varA."<br>";

[ Voor 6% gewijzigd door Verwijderd op 25-06-2007 22:30 ]

[Shadowman]

$obj->test("test 'enzo"); < je wil niet dat er userinput bij die functies wordt meegegeven.

Het Factory-pattern (in dit topic al een aantal keer genoemd: wikipedia: Factory Method) is geschikt voor wat je nu op een (mijns insziens) nog al aparte manier oplost.