constructor + objects

[xwin]

Hi, I have a constructor which loads two lib files and creates two new objects, $Debug and $Skin.

in another function in the same class I try to reference these but can't... is there a way I can do this while still initializing the objects in the constructor?

Thanks.

[thorpe]

you need to use the $this-> method. eg:

class myclass
{
    private $obj;
    function __construct() {
        $this->obj = new obj();
    }
    function foo() {
        $this->obj->an_obj_method();
    }
}

[xwin]

Yeah, but I'm trying to access a different object.

<?php

class System {

    // Constructor

    function System() {

        // Load core libraries

        require_once CORE_PATH."lib/debug.php" ;
        require_once CORE_PATH."lib/skin.php" ;
        require_once CORE_PATH."lib/error.php" ;
        require_once CORE_PATH."lib/lang.php" ;
        require_once CORE_PATH."lib/data.php" ;

        // Create core objects

        $Debug = new Debug ;
        $Skin  = new Skin ;
        $Error = new Error ;
        $Lang  = new Lang ;
        $Data  = new Data ;

        // Load configuration files

        require_once CONFIG_PATH."onepowered_global.php" ;
        require_once CONFIG_PATH."settings.php" ;

    }

    function Go() {

        // Start debug if enabled

        $Debug->Go() ;

$Debug->Go etc. won't run

Thanks for the help!

[thorpe]

because it needs to be...

$this->Debug->Go();

[xwin]

thorpe wrote:

because it needs to be...

$this->Debug->Go();

Ahh, thanks a lot!

[xwin]

Actually, that doesn't work either....

Call to member function on a non-object again.

[thorpe]

That does work. Your problem is you are decalring the $Debug variable local to the construct only. You need to decalre it outside. Eg:

class system {

  var $debug;

  function system() {
    $this->debug = new debug();
  }

  function go() {
    $this->debug->go();
  }

}

[xwin]

class System {

    var $Debug ;
    var $Skin ;

    // Constructor

    function System() {

        // Load core libraries

        //require_once CORE_PATH."lib/cache.php" ;
        require_once CORE_PATH."lib/debug.php" ;
        //require_once CORE_PATH."lib/error.php" ;
        //require_once CORE_PATH."lib/lang.php" ;
        //require_once CORE_PATH."lib/parser.php" ;
        require_once CORE_PATH."lib/skin.php" ;

        // Create core objects

        $Debug = new Debug ;
        $Skin  = new Skin ;
        //$Error = new Error ;
        //$Lang  = new Lang ;
        //$Data  = new Data ;

        // Load configuration files

        require_once CONFIG_PATH."onepowered_global.php" ;
        require_once CONFIG_PATH."settings.php" ;

    }

    function Go() {

        // Start debug if enabled

        $this->Debug->Go() ;

Reports Fatal error: Call to a member function on a non-object in dev/core/system.php on line 40

Line 40 is $this->Debug->Go() ;

[thorpe]

Man... I have shown you this like 4 times. You need to read the docs on how objects work. In your construct you will need to reference Debug using $this->

So this...

$Debug = new Debug;

becomes...

$this->Debug = new Debug ;