Brian Seymour wrote:
I am super stumped. This works fine separately but when I put everything
together it breaks. I has an authenticate class and a sql class. However I
always get the same error.
SQL class.
<?php
class SQL {
public $host;
public $user;
public $pass;
public $conx;
public $db;
public $dbname;
public $query;
public $result;
public $fetchedArray;
public $nRows;
public function __construct($host,$user,$pass,$dbname = null){
$this->host=$host;
$this->user=$user;
$this->pass=$pass;
$this->conx=$this->connection($host,$user,$pass);
if (!is_null($dbname)){ $this->selectDb($dbname); }
}
final public function connection($host,$user,$pass){
$this->conx=mysql_connect($host,$user,$pass) or
die(mysql_error());
}
final public function selectDb($db){
$this->db=mysql_select_db($db);
}
final public function query($query){
$this->result=mysql_query($query, $this->conx);
echo mysql_error();
echo $query;
return $this->result;
}
final public function fetchArray($query){
$this->result=$this->query($query);
$this->fetchedArray=mysql_fetch_array($this->result,MYSQL_ASSOC);
return $this->fetchedArray;
}
final public function makeArray($query){
$this->curArray=mysql_fetch_array($query,MYSQL_ASSOC);
return $this->curArray;
}
final public function numRows($result)
{
$this->nRows=mysql_num_rows($result);
return $this->nRows;
}
public function __destruct(){
if (isset($this->connection)){
mysql_close($this->connection); }
}
}
?>
Authenticate class
<?php
class Authentication extends SQL {
public $errorMsg;
public function __construct(){echo "Auth constructed";}
You didn't call the __construct() method of your parent.
The above code, should be like this
public function __construct($host,$user,$pass,$dbname = null) {
parent::__construct($host,$user,$pass,$dbname);
echo "Auth constructed";
}
You were forgetting to call to the parent and have it initialize the DB
connection.
In the second part, the $auth->verifyCreds() call, it didn't create a
valid db connection to pass as the second arg to the mysql_query() call.
And by not passing the $this->conx as the second arg, you are telling it
to "use the most recently opened mysql connection.
Hope this clears up why it was failing on the latter mysql_query() calls.
final public function verifyCreds ($user, $pass, $table)
{
$result = $this->query("SELECT * FROM $table where
$user='$pass'");
if ($this->numRows($this->result) == 0)
{
$this->errorMsg = "Incorrect Username/Password
Combo";
return false;
}
else
{
// debugging lines \/
echo "login good!";
// debugging lines /\
return true;
}
}
public function __destruct(){}
}
?>
Normal page.
<?php
/************************************************************
* common.php
*
* project: Renegades Revenge
* programmer: Brian Seymour
************************************************************/
// autoload classes
function __autoload($class_name) {
require_once 'includes/classes/class_' . strtolower($class_name)
. '.php';
}
// initialize Renegades Revenge database
$database = new SQL($host,$user,$pass,"aerocor_renegade");
// login
if (isset($_GET['login']))
{
$auth = new
Authentication($host,$user,$pass,"aerocor_renegade");
if
($auth->verifyCreds($_POST['username'],$_POST['password'],"players"))
{
echo "logged in good!";
}
}
?>
The form is just 2 fields. Username and password. I get this error.
Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource
in /home/aerocor/public_html/rr/includes/classes/class_sql.php on line 52
Warning: mysql_num_rows(): supplied argument is not a valid MySQL result
resource in /home/aerocor/public_html/rr/includes/classes/class_sql.php on
line 71
I put some simple query and display code in the constructor for the SQL
class and it outputted from the database with no problem, that's whats
weird.
Any help would be great, thanks.
Brian
--
Jim Lucas
"Some men are born to greatness, some achieve greatness,
and some have greatness thrust upon them."
Unknown
--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
