Some help

Tanax · 02-27-2009, 03:23 PM

Hi!

I'm wondering what the error with the following code is:

php Code:

public function loadQuery($query)
        {
           
            $this->query_sql = $query;
            
            return $this;
           
        }

        public function exeQuery($sql = NULL)
        {
           
            if($sql == NULL)
            {
               
                if(isset($this->query_sql))
                {
               
                    $this->query_result = mysql_query($this->query_sql, $this->conn);
                   
                    if($this->query_result) 
                    {
                        
                        return $this;
                        
                    }
                    
                    return 'Query did not work';
                   
                }
               
                return 'Query sql not provided';
               
            }
            
            else
            {
                
                $this->loadQuery($sql)->exeQuery();
            
            }
           
           
        }

I've just done like this:

PHP Code:


			
$sql = "SELECT * FROM `test` WHERE `id` = '4'";
$fetch = $db->exeQuery($sql);
    
echo $fetch;

This should actually give me an error: "Fatal blabla.. cannot convert object to string... blabla".. because it returns $this.. but it doesn't. It just displays blank.

Oh, and btw. It works if I do like:

PHP Code:


			
$sql = "SELECT * FROM `test` WHERE `id` = '4'";
$fetch = $db->loadQuery($sql)->exeQuery();
    
echo $fetch;

Then it gives me:

Code:

Catchable fatal error: Object of class DBmysql could not be converted to string in C:\wamp\www\Eldobado\index.php on line 9

which is correct.
But it doesn't seem like I can link them like I do, and skip the loadQuery.

Anyone see the problem?

allworknoplay · 02-27-2009, 04:04 PM

I don't see anywhere in your code where it should say "cannot convert object to string..."

Is that part somewhere else?

allworknoplay · 02-27-2009, 04:10 PM

Also can we see the rest of your class or is it too big? Like do you have any other properties declared etc...

sketchMedia · 02-27-2009, 05:50 PM

Try putting a return on the line:

PHP Code:


			
$this->loadQuery($sql)->exeQuery();

in your class.

Salathe · 02-27-2009, 07:10 PM

Quote:

Originally Posted by Tanax

This should actually give me an error: "Fatal blabla.. cannot convert object to string... blabla".. because it returns $this.. but it doesn't. It just displays blank.

Anyone see the problem?

That's because the return value (which is assigned to $fetch) isn't an object instance, it's NULL.

If some SQL is passed through to exeQuery, the else block is followed. This block just calls $this->loadQuery(SQL)->exeQuery(); then exits the if/else block. Following that route, nothing is returned from exeQuery so $fetch will be NULL which will print nothing when echoed.

As sketchMedia said, just put return before that loadQuery/exeQuery line.

Tanax · 02-27-2009, 09:53 PM

That's wicked. It worked though! Thanks!

The thing is that:

PHP Code:


			
$this->loadQuery($sql)->exeQuery();

this line, should execute this part of the code in that function:

PHP Code:


			
if($sql == NULL)
            {
               
                if(isset($this->query_sql))
                {
               
                    $this->query_result = mysql_query($this->query_sql, $this->conn);
                   
                    if($this->query_result) 
                    {
                        
                        return $this;
                        
                    }
                    
                    return 'Query did not work';
                   
                }
               
                return 'Query sql not provided';
               
            }

because no SQL value is passed.
And in THAT code, I return a value. I didn't think I'd need to return the function when I'm returning values IN that function :O

Tricky..
But it's working, thanks!

Tanax · 02-27-2009, 11:19 PM

Btw!

Quote:

Originally Posted by allworknoplay

I don't see anywhere in your code where it should say "cannot convert object to string..."

Is that part somewhere else?

No, it's a PHP-generated error, fatal errors are caused by PHP, and not really "outputted" anywhere in the code.
It's like.. when you try to connect to a db with the wrong settings, it gives an errormsg that it can't connect to the database, even if you haven't said anywhere that it should output that specific message.

Quote:

Originally Posted by allworknoplay

Also can we see the rest of your class or is it too big? Like do you have any other properties declared etc...

It's quite big, but since it's solved now it doesn't really matter