Fatal error: Call to a member function query() on a non-object

[dottedquad]

I'm getting the following error:
Fatal error: Call to a member function query() on a non-object

My class code:

<?php

    require_once '/home/dottedqu/php/MDB2.php';

class mdb2cls {

	function _construct () {

	}

	public function dataSourceName($dbtype, $user_name, $password, $host, $database, $dsnopts, $connectfunct) {

		//set dns string
		switch ($dbtype) {
			case "mysqli":
				$dsn = "mysqli://$user_name:$password@$host/$database";
				break;

		}

		$options = $dsnopts;

		switch ($connectfunct) {
			case "factory":
				$mdb2 =& MDB2::factory($dsn, $options);
				break;
			case "connect":
				$mdb2 =& MDB2::connect($dsn, $options);
				break;
			case "singleton":
				$mdb2 =& MDB2::singleton($dsn, $options);
				break;
			default:
				$mdb2 =& MDB2::factory($dsn, $options);
				break;
		}

		//check for errors
		if (PEAR::isError($mdb2)) {

			die($mdb2->getMessage() . ', ' . $mdb2->getDebugInfo());

		}

		//$mdb2->disconnect();

	}

	public function runQuery($type, $sql) {

		$query = $type . ' ' . $sql;

		switch ($type) {
			case "select";
				// Proceed with a select query
				$res =& $mdb2->query($query); //Fatal Error produced here!!!
				break;
			case "insert";
				// Proceed with insert query
				$affected =& $mdb2->exec($query);
			default:
				echo "runquery error";
		}

		// Always check that result is not an error
		if (PEAR::isError($affected)) {
				die($affected->getMessage());
		}

		// Disconnect
		//$mdb2->disconnect();

	}

	public function fetchRes() {

		// Get each row of data on each iteration until
		// there are no more rows
		while (($row = $res->fetchRow())) {
				// Assuming MDB2's default fetchmode is MDB2_FETCHMODE_ORDERED
				echo $row[0] . "\n";
		}
	}


}
?>

login.php code:

			if ($form->validate())
			{

			$mysql = new mdb2cls;
			$mysql->dataSourceName(
						"mysqli",
						"*****",
						"*****",
						"localhost",
						"*****",
						array(
							'debug' => 2,
							'result_buffering' => false
						),
						"factory"
					       );
			$mysql->runQuery ('select', 'SELECT * FROM `webpanel_accounts`');
			//$mysql->connectobj();
			//Out put the results
			$mysql->fetchRes();

		}
		else
		{
		$form->display();
		}

I'm brand new to OOP programing. I'm pretty sure I know what the issue is but have no idea how to fix it. I think the reason why i'm receiving that error is, there is no initiated object by the name $MDB2 within runQuery function? Like I said that just a guess. Any help/suggestions on what I should do to solve this error is greatly appreciated.

-Thanks,
Rich

[planetsim]

$mdb2 is a local variable in the method dataSourceName it is not accessible outside this method e.g. in runQuery. So when used in runQuery it is a NULL (in this case unintiated) variable. What you will need to do is make mdb2 a property of the class mdb2cls so its accessible to all its methods.

[dottedquad]

planetsim;10894829 wrote:

$mdb2 is a local variable in the method dataSourceName it is not accessible outside this method e.g. in runQuery. So when used in runQuery it is a NULL (in this case unintiated) variable. What you will need to do is make mdb2 a property of the class mdb2cls so its accessible to all its methods.

mdb2 is already a property of dataSourceName. Can I some how point to that function and use the object that is already initated?

From what you explained, do i need to place: $mdb2 =& MDB2::factory($dsn, $options); here:

class mdb2cls {
      $mdb2 =& MDB2::factory($dsn, $options);

If that's the case, what i'm attempting to accomplish isn't going to work. Or atleast I wouldn't know how to do.

I'm attempting to create another class to break of the database steps along side of '/home/dottedqu/php/MDB2.php to make my code reusable.

code that i'm trying to break up using a class:

<?php
// Create a valid MDB2 object named $mdb2
// at the beginning of your program...
require_once 'MDB2.php';

$mdb2 =& MDB2::connect('mysqli://usr:pw@localhost/dbnam');
if (PEAR::isError($mdb2)) {
    die($mdb2->getMessage());
}

// Proceed with getting some data...
$res =& $mdb2->query('SELECT * FROM mytable');

// Get each row of data on each iteration until
// there are no more rows
while (($row = $res->fetchRow())) {
    // Assuming MDB2's default fetchmode is MDB2_FETCHMODE_ORDERED
    echo $row[0] . "\n";
}

// while (($one = $res->fetchOne())) {
//     echo $one . "\n";
// }

?>

-Rich

[laserlight]

dottedquad wrote:

mdb2 is already a property of dataSourceName. Can I some how point to that function and use the object that is already initated?

A local variable is not a "property" (member variable). planetsim suggested that you make that variable a member variable instead, which would allow you to use the object that is already instantiated.