Fatal Error: Using $this when not in object context

Question

I'm receiving this fatal error message: Using $this when not in object context. This class is setted up as a library in the CodeIgniter.

This is my class:

class My_class {

    function __construct()
    {
            $this->app = base_url('application') . '/cache/';
            if ($this->expire_after == '')
            {
                $this->expire_after = 300;
            }
    }

    static function store($key, $value)
    {
        $key = sha1($key);
        $value = serialize($value);
        file_put_contents( $this->app . $key.'.cache', $value);
    }
}

I'm initializing it via autoload.php. The line it is throwing the error at:

file_put_contents( $this->app . $key.'.cache', $value);

Where is my problem?

you're using $this in your static function, you can't do that. — thetaiko
– thetaiko, Commented Apr 3, 2012 at 17:17
I don't know why you are using a static method! I only use them in utility classes :) — Songo
– Songo, Commented Apr 3, 2012 at 17:23

Simeon Visser · Accepted Answer · 2012-04-03 17:18:08Z

5

You can't use $this in a static method. The variable $this is only available to class methods as these receive the object on which the method is called.

That's what "when not in object context" means: there's no object passed to that static method because it's static. A static method is part of the class, not part of the objects that are instantiated using that class.

answered Apr 3, 2012 at 17:18

Simeon Visser

123k19 gold badges192 silver badges185 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

rjz · Accepted Answer · 2012-04-03 17:21:18Z

3

$this will not be available in a static function. You'll probably want to re-create $app within the static function:

static function store($key, $value)
{
    $app = base_url('application') . '/cache/';
    $key = sha1($key);
    $value = serialize($value);
    file_put_contents( $app . $key.'.cache', $value);
}

I'm not quite sure what you're trying to do in the grand context of your application, but you may not even need a static method in the first place.

answered Apr 3, 2012 at 17:21

rjz

16.5k3 gold badges39 silver badges35 bronze badges

Comments

amphetamachine · Accepted Answer · 2012-04-03 17:21:53Z

1

To be honest, the store function should be an instance function (remove static keyword), otherwise using $this within it will have no idea what object it's referring to.

Alternatively, you could have objects pass in references to themselves so that the static function would know what object to act on: static function store($obj, $key, $value) [...] $obj->app [...]

Or, just pass in the contents of $obj->app since the static function only needs that piece of information and not access to the entire object:

static function store($app, $key, $value)
{
    $key = sha1($key);
    $value = serialize($value);
    file_put_contents( $app . $key.'.cache', $value);
}

answered Apr 3, 2012 at 17:21

amphetamachine

30.9k12 gold badges68 silver badges74 bronze badges

Collectives™ on Stack Overflow

Fatal Error: Using $this when not in object context

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related