Using $this when not in object context. ERROR

Question

<?

class Flip 
{

    private $db;

    public function __construct()
    {
        $this->db = new Database();
    }

    public function flips()
    {
        $flips = array();

        $result = $this->db->Select()->Get('flips');
        if($result)
        {
            foreach ($result as $value) 
            {
                $flips[] = $value;
            }
            return $flips;
        }
        return false;
    }

    public function flipComment($id)
    {
        return $this->db->Select()->Where('id', $id)->Get('comments');
    }

}

Fatal error: Using $this when not in object context.

Any ideas how to fix? I don't want to recall class Database for each function. I need to be able to use $this->db

Can you post a full class declaration? $this is used inside the class declaration. Otherwise you need to use a instance name. — gene
– gene, Commented Jul 9, 2015 at 3:01
Can you please show the full error message , which line does this error occurs ? — Kanishka Panamaldeniya
– Kanishka Panamaldeniya, Commented Jul 9, 2015 at 3:31
The code you have pasted thus far is valid. You need to show the full error message including line number, and make sure you are copy/pasting exactly your code. — longofest
– longofest, Commented Jul 9, 2015 at 3:43

bnahin · Accepted Answer · 2015-07-09 02:59:34Z

1

You cannot use $this because the functions are not part of a class.

    Class MyClass {
        private $db;

        public function __construct()
        {
            $this->db = new Database();
        }

        public function comment()
        {
            $this->__construct();

            return $this->db->Select()->Get('comments');
        }
}

answered Jul 9, 2015 at 2:59

bnahin

8161 gold badge8 silver badges20 bronze badges

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

Comments

Cohan · Accepted Answer · 2015-07-09 03:25:21Z

0

Make your variable static and in the constructor you can see if it's already set. Then you will only have to open the connection once and every time you call Flip it will be able to use that connection.

class Flip
{
    private static $db;

    function __construct()
    {
        if (!isset(self::$db))
            self::$db = new Database();
    }

    function comment()
    {
        return self::$db->Select()->Get('comments');
    }
}

5 Comments

Rylee Harrison Over a year ago

I don't wanna recall the class Database.

Cohan Over a year ago

what do you mean by recall? do you mean you don't want to make a new connection to the database?

Rylee Harrison Over a year ago

I wrote an interpreter database class that is crafts my query and I would like to save memory by only needing to call it once inserted of each function requiring the class

Rylee Harrison Over a year ago

Call to a member function Select() on a non-object

Cohan Over a year ago

Did you fix my typo $b-> should be $db->. I will edit that in my answer

Collectives™ on Stack Overflow

Using $this when not in object context. ERROR

2 Answers 2

Comments

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related