Charles Kline wrote:
On Sep 27, 2005, at 3:42 PM, Andy Pieters wrote:
Hi
Without you actually showing us these class files we can only guess
but a
common mistake is this:
mysql_open(connection details)
mysql_query(query)
In those cases the last opened handle is used. To prevent this, use
this
syntax
$db1=mysql_open(connection for db1);
$db2=mysql_open(connection for db2);
mysql_query($db1,$query_db_1);
mysql_query($db2,$query_db_2);
If you have used this syntax then check your class if it is using a
global
variable to hold the database handle and if it does make it a class
variable
instead
Instead of
$db=null
class db
{function db()
{$GLOBALS['db']=mysql_open(...
do instead
class db
{var $db=null;
function db()
{$this->db=mysql_open
That way you can instanciate as many instances of the class as you
like and
each will have its own database handle.
HTH
Andy
What I have is more like this:
class db {
var $dbpath = "localhost";
var $dbname = "testdb";
var $dblogin = "test";
var $dbpass = "test";
function db() {
$link = mysql_connect($this->dbpath, $this->dblogin, $this- >dbpass)
or die ('Not Connected: ' . mysql_error());
mysql_select_db($this->dbname, $link) or die ('Can\'t use this
database: ' . mysql_error());
}
function retrieveData( $sql ) {
$rs = mysql_query( $sql ) or die("Invalid query: " . mysql_error ());
// if no result, return null
if (($rs == null) || (mysql_num_rows($rs) == 0)) {
return null;
} else {
return ( $rs );
}
}
function insertData( $sql ) {
mysql_query( $sql );
// return new id if insert is successful
if ( mysql_affected_rows() > 0 ) {
return ( mysql_insert_id() );
} else {
return null;
}
}
function updateData( $sql ) {
$rs = mysql_query( $sql );
if (mysql_affected_rows() > 0) {
return ( $rs );
} else {
return null; // no changes were made
}
}
}
I then have another class with a different name and I changed the names
of the functions as well. I have another set of class files that
contain my various queries etc.
Thanks for any help.
Charles
that's where it's going wrong. You're not linking db::db()'s link in the
db::insertData() mysql_query() function. Thus, the mysql_query function
defaults to the "last opened mysql connection" regardless of object. A
way to fix this is to store the link in $this->link instead, and
changing your mysql_query()'s to mysql_query($sql, $this->link);
same goes for your mysql_affected_rows() and mysql_insert_id() functions
-tul
--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
