Tell me more ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 5 down vote favorite

I'm trying out a recipe on how to store string resources in PHP, but I can't seem to get it to work. I am a little unsure on how the __get function works in relation to arrays and objects.

Error Message: "Fatal error: Cannot use object of type stdClass as array in /var/www/html/workspace/srclistv2/Resource.php on line 34"

What am I doing wrong?

/**
 * Stores the res file-array to be used as a partt of the resource object.
 */
class Resource
{
    var $resource;
    var $storage = array();

    public function __construct($resource)
    {
        $this->resource = $resource;
        $this->load();
    }

    private function load()
    {
        $location = $this->resource . '.php';

        if(file_exists($location))
        {
             require_once $location;
             if(isset($res))
             {
                 $this->storage = (object)$res;
                 unset($res);
             }
        }
    }

    public function __get($root)
    {
        return isset($this->storage[$root]) ? $this->storage[$root] : null;
    }
}

Here is the resource file named QueryGenerator.res.php:

$res = array(
    'query' => array(
        'print' => 'select * from source prints',
        'web'  => 'select * from source web',
    )
);

And here is the place I'm trying to call it:

    $resource = new Resource("QueryGenerator.res");

    $query = $resource->query->print;
share|improve this question
add comment (requires an account with 50 reputation)

1 Answer

up vote 2 down vote accepted

It's true that you define $storage as an array in your class but then you assigne object to it in load method ($this->storage = (object)$res;).

Fields of class can be accessed with following syntax: $object->fieldName. So in your __get method you should do:

public function __get($root)
{
    if (is_array($this->storage)) //You re-assign $storage in a condition so it may be array.
        return isset($this->storage[$root]) ? $this->storage[$root] : null;
    else
        return isset($this->storage->{$root}) ? $this->storage->{$root} : null;
}
share|improve this answer
I think this works directly $this->storage->$root – Elzo Valugi Nov 22 '12 at 12:45
@ElzoValugi Sure, it does. I use this because I think it's more understandable for "non-php" programmers. – PLB Nov 22 '12 at 12:45
@PLB : Using this function returns NULL (from the "else"-part of the check). Still getting with "$resource->query->print" as if it where a scalar with a string. – Jonas Ballestad Nov 22 '12 at 13:32
@JonasBallestad You can access it like: $resource->query['print']. – PLB Nov 22 '12 at 13:51
@PLB So this means that going (object)$res only "objectifies" the top level, but keeps the lower ones at array behaviour? – Jonas Ballestad Nov 22 '12 at 13:53
show 1 more commentadd comment (requires an account with 50 reputation)

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.