PHP - Fatal error: Using $this when not in object context

Question

I'm really sorry to be asking this question because I know it has been asked many times before (and I have read the threads) but I just can't puzzle this out.

I have a class:

class Fivehundredpx {
    private $_user;
    public $fivehundredpx;

    public function __construct(User $user){ 
        $this->_user = $user->data();
        $this->fivehundredpx = $user->data()->fivehundredpx;
    }

    public function fhpxEndpoint(){ //truncated - this function actually has a number of switch statements
        return $apistring = "https://api.500px.com/v1/photos?feature=user_favorites&username=". $this->fivehundredpx."&sort=rating&image_size=3&include_store=store_download&include_states=voted&consumer_key=I9CDYnaxrFxLTEvYxTmsDKZQlgStJG868GKb"; //this is the line that causes the error
    }

}

when fhpxEndpoint() is called I get the Fatal error: Using $this when not in object context message.

However if I add another method into the class:

public function userData(){
    print 'here is some text '. $this->fivehundredpx;
}

and call this from outside the class it prints out here is some text jinky32 as expected.

Any help much appreciated!

edit as per request: method is called via

$fivehundredpx = new Fivehundredpx;
$obj = $fivehundredpx->fhpxApiConnect($fivehundredpx->fhpxEndpoint());

fhpxApiConnect() is:

public function fhpxApiConnect($apiString){
    print $apiString;
    $ch = curl_init();
    curl_setopt($ch, CURLOPT_URL,$apiString);
    curl_setopt($ch , CURLOPT_HEADER, 0);
    curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1); 
etc etc

printing out $apiString as I do in fhpxApiConnect() shows that the url has been constructed correctly and the name jinky32 has been inserted correctly but i still get the fatal error

EDIT: OK, I've played around and found a way to get it to work. If I change the class variable to

public static $fivehundredpx;

and then set it

public function __construct(User $user){
    self::$fivehundredpx=$user->data()->fivehundredpx;
}

I can then access it in fhpxEndpoint() using self::$fivehundredpx rather than $this->fivehundredpx

Please post the code where you actually call fhpxEndpoint(). — colburton
– colburton, Commented Jun 12, 2014 at 10:48
@b4rt3kk do you mean within the constructor? if so no it doesn't — Stuart Brown
– Stuart Brown, Commented Jun 12, 2014 at 11:02
You should start by passing in an instance of User when instantiating Fivehundredpx. Right now it should fatal even before you call fhpxEndpoint — tlenss
– tlenss, Commented Jun 12, 2014 at 11:06
@tlenss yes you are right. The Fivehundredpx object is actuallt instantiated elsewhere and does pass in the instance of User. I just added it above in closer proximity to where I am using the instance but forgot to include the User param. Sorry — Stuart Brown
– Stuart Brown, Commented Jun 12, 2014 at 11:21

tlenss · Accepted Answer · 2014-06-12 11:02:36Z

0

Try this:

public function userData(){
    print 'here is some text '. Fivehundredpx::fivehundredpx();
}

or can try with:

public function userData(){
    $obj = new Fivehundredpx();
    print 'here is some text '. $obj->fivehundredpx();
}

edited Jun 12, 2014 at 11:02

tlenss

2,6172 gold badges23 silver badges26 bronze badges

answered Jun 12, 2014 at 10:48

prava

3,9762 gold badges26 silver badges35 bronze badges

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

1 Comment

Stuart Brown Over a year ago

doing this means that no longer prints out here is some text jinky32 and generates Fatal error: Call to undefined method Fivehundredpx::fivehundredpx()

b4rt3kk · Accepted Answer · 2014-06-12 11:07:18Z

0

Based on your comment, you can't access to a property of User like this:

$user->data()->fivehundredpx;

While method data of class User doesn't return your object. Add to method data:

return $this;

at the end.

answered Jun 12, 2014 at 11:07

b4rt3kk

1,5773 gold badges17 silver badges31 bronze badges

1 Comment

Stuart Brown Over a year ago

I can access the value using $user->data()->fivehundredpx; though. If I couldn't then surely my userData() method (which I am only using to try and debug this issue) wouldn't print out the value since $this->fivehundredpx is set in the constructor by $user->data()->fivehundredpx

Collectives™ on Stack Overflow

PHP - Fatal error: Using $this when not in object context

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related